Civix upgrade required for Smarty 3 support

systopia / de.systopia.eventmessages

CiviCRM Custom Event Communication Extension

Other

4 stars 6 forks source link

Civix upgrade required for Smarty 3 support #48

Open nickperkins opened 8 months ago

nickperkins commented 8 months ago

When running with Smarty 3 enabled on recent versions of Civicrm, the following notice is shown on my developer site:

Notice: Indirect modification of overloaded property CRM_Core_Smarty::$template_dir has no effect in _eventmessages_civix_civicrm_config() (line 107 of sites/default/files/civicrm/ext/de.systopia.eventmessages/eventmessages.civix.php).

Visiting an event page shows broken functionality. This is fixed if I revert to Smarty 2, but core is moving to Smarty 3 shortly.

jensschuppe commented 7 months ago

I'm pretty sure that a civix upgrade would be enough to fix that. Would you be able to provide a PR for this, @nickperkins?