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

Symfony 5 support #119

Open
Zausenec opened this issue Dec 11, 2019 · 9 comments
Open

Symfony 5 support #119

Zausenec opened this issue Dec 11, 2019 · 9 comments

Comments

@Zausenec
Copy link
Contributor

Need Sf5 support please!
I found only the issue with this line

->scalarNode('web_dir')->defaultValue('%kernel.root_dir%/../web')->end()

kernel.root_dir is no longer supported, and it breaks container build

@sergioplaza
Copy link

Change
-> scalarNode ('web_dir') -> defaultValue ('% kernel.project_root% / .. / web') -> end ()

by
-> scalarNode ('web_dir') -> defaultValue ('% kernel.project_dir% / .. / web') -> end ()

@Zausenec
Copy link
Contributor Author

I now, but this is not generic and may cause BC break, so I can't contribute PR.

@lordjancso
Copy link

@Gregwar Are you planning to release an SF5 compatible version of this bundle?

@aykonsvk
Copy link

Can't you ask for SF version? then call method with corresponding data. It will be still generic

@aykonsvk
Copy link

I have fixed issues with SF 5 on my fork would you merge it? Probably not into the master because of BC

@Gregwar
Copy link
Owner

Gregwar commented Feb 17, 2020

Hi,
It should be OK now, I published a v3 that should be compliant to Sf5

BC is broken because we need Twig 2, but this is the work of composer, so I also bumped twig dependency to twig 2

Bundle worked for me on a simple test (image resize), ping me if any issue appear

@azhararmar
Copy link

It cannot be installed in Symfony 5 since it is asking Twig3 dependency.

@pwnyprod
Copy link

The only Problem is the dependency "twig/twig": "^2.0" which limit to twig2 but symfony requires twig3. We need just the twig3 dependency in composer

@Gregwar
Copy link
Owner

Gregwar commented Apr 20, 2020

Ok, is it safe to bump

"twig/twig": "^1.12 || ^2.0"

To

"twig/twig": "^1.12 || ^2.0 || ^3.0"

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

7 participants