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

Unexpected "apply" tag #15983

Closed
garak opened this issue Feb 22, 2020 · 9 comments · Fixed by #16040
Closed

Unexpected "apply" tag #15983

garak opened this issue Feb 22, 2020 · 9 comments · Fixed by #16040
Assignees
Labels
Bug A problem or regression with an existing feature
Projects
Milestone

Comments

@garak
Copy link

garak commented Feb 22, 2020

Describe the bug

Uncaught Twig_Error_Syntax: Unexpected "apply" tag (expecting closing tag for the "set" tag defined near line 27). in /usr/share/phpmyadmin/templates/display/import/javascript.twig

To Reproduce

Steps to reproduce the behavior:

  1. Go to import
  2. See a 500 error popup
  3. Find above error in log

Expected behavior

No error

Server configuration

  • Operating system: Ubuntu 18.04
  • Web server: nginx
  • Database version: mysql 5.7.29-0ubuntu0.18.04.1
  • PHP version: 7.4.3
  • phpMyAdmin version: 4.9.4deb4

Client configuration

  • Browser: Firefox 73.0
  • Operating system: Ubuntu 18.04
@williamdes
Copy link
Member

@williamdes williamdes added the Bug A problem or regression with an existing feature label Feb 22, 2020
@williamdes williamdes added this to Needs triage in issues via automation Feb 22, 2020
@williamdes williamdes moved this from Needs triage to High priority in issues Feb 22, 2020
@williamdes
Copy link
Member

I need to investigate if it is a solution to upgrade twig on the PPA

@garak are you using the PPA ?

@garak
Copy link
Author

garak commented Feb 22, 2020

I do

@williamdes
Copy link
Member

williamdes commented Feb 22, 2020

So my recent upload did break things..
I will try to fix this ASAP

b475cfd
bd33d92
99250a8
eb8472b

Ubuntu bionic is at twig 2.4.6-1
Ubuntu eoan is at twig 2.11.3-2

@williamdes williamdes added this to the 4.9.5 milestone Feb 22, 2020
@williamdes
Copy link
Member

works for Twig 1.40+ and 2.9+: https://twigfiddle.com/jhf5f0

@williamdes
Copy link
Member

williamdes commented Feb 23, 2020

One thought I had is WTF why did you remove the tag between 1.40 and 2.9 🤔
@fabpot do you have an idea ?

@MauricioFauth
Copy link
Member

MauricioFauth commented Feb 23, 2020

One thought I had is WTF why did you remove the tag between 1.40 and 2.9 🤔

It was not removed, it was added in 2.9 and backported to 1.40. Both versions were released at the same time, because Twig develops 1.x, 2.x and 3.x in parallel.

@williamdes
Copy link
Member

One thought I had is WTF why did you remove the tag between 1.40 and 2.9 thinking

It was not removed, it was added in 2.9 and backported to 1.40. Both versions were released at the same time, because Twig develops 1.x, 2.x and 3.x in parallel.

Oh !
Maybe the documentation should be more clear about that
Eg: display the versions it is available on each tag
One line with versions restrictions in the composer format could be easyly readable

@garak
Copy link
Author

garak commented Feb 27, 2020

It seems to be fixed with latest release of php-twig package
Thanks!

@garak garak closed this as completed Feb 27, 2020
issues automation moved this from High priority to Closed Feb 27, 2020
@williamdes williamdes self-assigned this Mar 24, 2020
@williamdes williamdes reopened this Mar 24, 2020
issues automation moved this from Closed to Needs triage Mar 24, 2020
williamdes added a commit that referenced this issue Mar 24, 2020
Pull-request: #16040
Fixes: #15983

Signed-off-by: William Desportes <williamdes@wdes.fr>
williamdes added a commit that referenced this issue Mar 24, 2020
Signed-off-by: William Desportes <williamdes@wdes.fr>
issues automation moved this from Needs triage to Closed Mar 24, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature
Projects
issues
  
Closed
Development

Successfully merging a pull request may close this issue.

3 participants