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

Smarty 3.1.21: Issue when PHP tags are allowed, asp tags are off but asp tag <% is encoutered #17

Closed
m038 opened this issue Feb 18, 2015 · 6 comments

Comments

@m038
Copy link

m038 commented Feb 18, 2015

On line 3456 of vendor/smarty/smarty/libs/sysplugins/smarty_internal_templateparser.php there is a syntax error $this->_retvalue = new _smarty_text($this, st);, which gets triggered in the scenario when PHP tags are allowed, asp tags are off, but an asp-tag <% is found. (We use it for backbone templtes.)

The error message: Use of undefined constant st - assumed 'st'

@uwetews
Copy link
Contributor

uwetews commented Feb 19, 2015

This is already fixed in the master branch and will be included in the upcomming 3.1.22

@uwetews uwetews closed this as completed Feb 19, 2015
@Artistan
Copy link

i see this issue in 3.1.21
[Thu Apr 30 06:37:41.557259 2015] [:error] [pid 8687] [client 10.128.1.1:34038] PHP Notice: Use of undefined constant st - assumed 'st' in /usr/local/lib/php/Smarty/sysplugins/smarty_internal_templateparser.php on line 3456,

Server version: Apache/2.4.6 (CentOS)
PHP 5.4.16 (cli) (built: Oct 31 2014 12:59:36)
short tags enabled

anything else???

@uwetews
Copy link
Contributor

uwetews commented Apr 30, 2015

This bug is already fixed in the dev-master version.
The 3.1.22 version including this fix will be released in a few days.

@spaceone
Copy link

spaceone commented May 5, 2015

can you please refer to a commit so that i can cherry-pick the fix?

@uwetews
Copy link
Contributor

uwetews commented May 6, 2015

It was fixed with commit a22ee9f from oct 23rd, 2014

@spaceone
Copy link

spaceone commented May 6, 2015

thank you!

think-mcunanan pushed a commit to think-mcunanan/smarty that referenced this issue Mar 22, 2023
…)-Allow-setting-of-business-hours-and-number-of-reservations-(by-day)-to-be-configured-per-kanzashi-account

Ticket #5017 (kirei) allow setting of business hours and number of reservations (by day) to be configured per kanzashi account
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants