Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev/core#2817 towards deprecating legacy replaceCaseTokens #21380

Merged
merged 1 commit into from
Sep 7, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

dev/core#2817 towards deprecating legacy

This adds support for the syntax

{case.status_id:label} which what we agreed as a preferred syntax
to disambiguate the demands for labels vs ids vs machine names in
various implementations.

The new (preferred) label is advertised - but the old
one still works

Before

{case.status_id} renders (e.g.) 'Ongoing'
{case.case_type_id} renders (e.g 'Housing Support')

After

The unambiguous versions work & are supported. The ambiguous ones also still work - but in the token processor variant they will render the DB value

{case.status_id} renders (e.g.) 'Ongoing'
{case.status_id:label} renders (e.g.) 'Ongoing'
{case.case_type_id} renders (e.g 'Housing Support')
{case.case_type_id:label} renders (e.g 'Housing Support')

Technical Details

See https://lab.civicrm.org/dev/core/-/issues/2817 for next steps

Comments

This adds support for the syntax

{case.status_id:label} which what we agreed as a preferred syntax
to disambiguate the demands for labels vs ids vs machine names in
various implementations.

The new (preferred) label is advertised - but the old
one still works

See https://lab.civicrm.org/dev/core/-/issues/2817 for next steps
@civibot
Copy link

civibot bot commented Sep 6, 2021

(Standard links)

@civibot civibot bot added the master label Sep 6, 2021
@eileenmcnaughton eileenmcnaughton changed the title dev/core#2817 towards deprecating legacy dev/core#2817 towards deprecating legacy replaceCaseTokens Sep 6, 2021
@mattwire mattwire merged commit be16c5e into civicrm:master Sep 7, 2021
@seamuslee001 seamuslee001 deleted the case_opts branch September 7, 2021 10:30
@eileenmcnaughton
Copy link
Contributor Author

thanks @mattwire

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants