search

twomice / com.joineryhq.jsumfields

Provides additional fields under the Summary Fields framework.
Other
1 stars 10 forks source link

Smarty 3+ compatibility fix #29

Closed MegaphoneJon closed 1 month ago

MegaphoneJon commented 1 month ago

This code fails in Smarty 3+. See https://github.com/civicrm/civicrm-core/pull/27589 for an example of this change in core.

Once Smarty 2 is dropped, we can change these again to getTemplateVars() but for now we use get_template_vars() and rely on the Civi Smarty compatibility layer.

twomice commented 1 month ago

Merged -- thanks @MegaphoneJon !