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

Replace isset with token for pay_later_receipt #26299

Merged
merged 1 commit into from
May 22, 2023

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Replace isset with token for pay_later_receipt

Before

Use of isset in templates causes hard-errors with smarty-secure mode enabled

After

Replaced with a token, this is also previewable

Technical Details

As tested in https://github.com/civicrm/civicrm-core/pull/26294/files#diff-a5d237ba4e392d0e8ad764a535315f0b1101cc5ca61c39b4e572de42f5c6b4bcR676 this token works & removes one of the breaky issets

Comments

@civibot
Copy link

civibot bot commented May 22, 2023

(Standard links)

@civibot civibot bot added the master label May 22, 2023
@yashodha
Copy link
Contributor

@eileenmcnaughton
looks good, merging this PR

@yashodha yashodha merged commit a157b35 into civicrm:master May 22, 2023
@yashodha
Copy link
Contributor

@eileenmcnaughton
You might want to add to the pre-upgrade message that the said templates have been changed.

@eileenmcnaughton eileenmcnaughton deleted the isset branch May 22, 2023 19:40
@eileenmcnaughton
Copy link
Contributor Author

@yashodha yeah - we have been waiting until there is a critical mass of changes to push-upgrade people

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