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

Fixed var #49

Closed
wants to merge 1 commit into from
Closed

Fixed var #49

wants to merge 1 commit into from

Conversation

JaroslavHerber
Copy link
Contributor

protected $thisTemplate is wrong. Should be protected $_sThisTemplate.

This caused IPN-Handler to spam warnings in apaches error.log:
[Thu Feb 07 15:10:47.010789 2019] [php7:warn] [pid 4446] [client XXX] PHP Warning: Smarty error: unable to read resource: "" in XXX/vendor/smarty/smarty/libs/Smarty.class.php on line 1100

protected $thisTemplate is wrong. Should be protected $_sThisTemplate.
@Sieg
Copy link
Member

Sieg commented Mar 12, 2019

Hi @JaroslavHerber, thanks for the fix! Have merged to b-5.x and master recently. Will be available in next release.

@Sieg Sieg closed this Mar 12, 2019
@Sieg Sieg removed the in progress label Mar 12, 2019
@Sieg Sieg self-assigned this Mar 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants