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

The tokens not working here #21

Closed
jnajajreh opened this issue Feb 24, 2022 · 7 comments
Closed

The tokens not working here #21

jnajajreh opened this issue Feb 24, 2022 · 7 comments

Comments

@jnajajreh
Copy link

Dears
I am using civicrm from one year and I am using Related token extension, we used to use it to send thank you letter for the head of organization, so I am using the relationship type ("Head of") in my template, but unfortunately I have issue in displaying the address of the head and other information, the email token is working well, would you please help us to solve this issue? I appreciate your help
Civicrm 5.46.2

1
1-1
2

@twomice
Copy link
Owner

twomice commented Apr 14, 2022

Thank you for the report.

Here's what I'm seeing:

  • This extension will exhibit such problems (some related tokens simply won't display their values) in some newer versions of civicrm (not sure what's the latest compatible version, but perhaps as early as 5.43.x), because it relies on a deprecated utiltity function in CiviCRM core (CRM_Utils_Token::getTokenDetails())
  • To be compatible with newer versions of CiviCRM core, the extension should be modified to use the Token Processor approach described at https://docs.civicrm.org/dev/en/latest/framework/token/

@MegaphoneJon
Copy link
Contributor

@jnajajreh - I think my new version of this extension (https://github.com/MegaphoneJon/com.joineryhq.reltoken/archive/refs/heads/token-processor.zip) fixes your problem. Could you please try it and let us know?

@jnajajreh
Copy link
Author

jnajajreh commented Jun 28, 2022 via email

@twomice
Copy link
Owner

twomice commented Jul 11, 2022

Thanks for responding @MegaphoneJon . Curious, does your linked version (this comment) contain anything other than PR #22 (now merged into this repo as of v0.4.0)? I'd be glad to get a PR on any additional improvements you might have made.

@twomice
Copy link
Owner

twomice commented Jul 29, 2022

@jnajajreh and @MegaphoneJon: As of version 0.4.0, I think this repo now contains all the fixes from Jon's github fork repo (unless Jon can point out others). @jnajajreh I'm inclined to close this on the assumption that this version will address your issue. Please comment here if that's not so. Thanks!

@MegaphoneJon
Copy link
Contributor

This looks good to close.

@twomice
Copy link
Owner

twomice commented Apr 28, 2023

Thanks @MegaphoneJon .

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

No branches or pull requests

3 participants