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

Can't access the SEPA Dashboard and Contact summary page - Template error #691

Open
allinappliadmin opened this issue Dec 13, 2023 · 7 comments
Labels
bug Bug in the code

Comments

@allinappliadmin
Copy link

I am testing the 1.10-dev+4ad4b0cbb5336161 version on CiviCRM 5.70.alpha1 and get this errors when:

Access Sepa Dashboard: Unable to load template 'file:CRM/Sepa/Page/DashBoard.tpl' in 'file:CRM/common/CMSPrint.tpl'
Try to see any Contact: Unable to load template 'file:CRM/Contact/Page/View/Summary.sepa.tpl' and I can't access the contact summary

In the terminal I can see these notices: [PHP Notice] Indirect modification of overloaded property CRM_Core_Smarty::$template_dir has no effect at ./httpdocs/wp-content/uploads/civicrm/ext/org.project60.sepa/sepa.civix.php:107

I am using the smarty 3 on this instance.

@bjendres bjendres added the bug Bug in the code label Dec 13, 2023
@bjendres
Copy link
Member

@allinappliadmin thanks for reporting.

@jensschuppe Does this ring a bell? Do we now about anything in 5.70.alpha1+ that could cause this?

@allinappliadmin
Copy link
Author

I tested on a 5.67.2/PHP 7.4 instance and I don't have the error for the Sepa Dashboard anymore but I still can't access/search ANY contacts. All my displays are empty and I'm having a DB error pop-up

@jensschuppe
Copy link
Collaborator

jensschuppe commented Dec 13, 2023

@jensschuppe Does this ring a bell? Do we now about anything in 5.70.alpha1+ that could cause this?

Not me, not yet at least … Maybe that's connected to Smarty upgrade in Core, see dev/core#4146, especially

** Challenges that we are doing OK on **

  • Direct access of class properties - we have a couple of places that do things like $smarty->_tpl_vars - those need replacing with get_template_vars or, if we add compatibility getTemplateVars

As this is in the Civix file, regenerating it with civix upgrade might help - given that Civix has been updated already.

@jensschuppe jensschuppe removed their assignment Dec 13, 2023
@allinappliadmin
Copy link
Author

@jensschuppe Does this ring a bell? Do we now about anything in 5.70.alpha1+ that could cause this?

Not me, not yet at least … Maybe that's connected to Smarty upgrade in Core, see dev/core#4146, especially

** Challenges that we are doing OK on **

  • Direct access of class properties - we have a couple of places that do things like $smarty->_tpl_vars - those need replacing with get_template_vars or, if we add compatibility getTemplateVars

As this is in the Civix file, regenerating it with civix upgrade might help - given that Civix has been updated already.

as said, I have a bit of the same issue on a 5.67.2 instance at least for contacts. it breaks everything.

@bjendres
Copy link
Member

Hm, we have a bunch of CivSEPA-1.9.1s running on 5.64.4, but not many reports from higher versions.

@allinappliadmin Could you please try two things:

  1. Extract and attach the DB error
  2. Run civix upgrade on the extension in your CiviCRM and see if that changes anything
  3. Test if you can reproduce the error on 5.64.4

@allinappliadmin
Copy link
Author

my test instance on 5.67.2 was actually already broken :-/ and on a 5.65.1 instance it seems to work. So I guess the initial issue stays for a recent version

@allinappliadmin
Copy link
Author

I confirm that it's working on CiviCRM 5.67.3. I've just installed the 1.10-dev+4ad4b0cbb5336161 version for the client that had this issue #678

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

No branches or pull requests

3 participants