search

civicrm / org.civicrm.flexmailer

FlexMailer is now part of civicrm-core. Please see the link below for how to upgrade.
https://civicrm.org/extensions/flexmailer
Other
5 stars 22 forks source link

Fixes #61: partial revert the old autoloader to avoid cache problems #62

Closed mlutfy closed 4 years ago

mlutfy commented 4 years ago

See #61

Upgrading flexmailer on an existing site causes cache problems. Deleting the cache manually fixes the issue, but only if we delete the cache files manually (with "rm"), because CiviCRM won't boot.

To reproduce:

Until we find a better fix, this PR reverts the removal of the legacy autoload code.

Once the cache is flushed (with the new version of flexmailer), the old autoload is not necessary anymore.

mattwire commented 4 years ago

@mlutfy This seems reasonable. It would be nice to get to the bottom of the cache clear issue but should not hold up the merge of this PR once it's tested.

agileware-justin commented 4 years ago

@mlutfy we used the same fix as this PR. So good to merge from Agileware's perspective.

mlutfy commented 4 years ago

Thanks @agileware-justin @mattwire . I took the liberty to self-merge this PR based on reviews.