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/templating is deprecated #437

Closed
ndench opened this issue Dec 28, 2023 · 9 comments
Closed

symfony/templating is deprecated #437

ndench opened this issue Dec 28, 2023 · 9 comments
Assignees
Labels
Milestone

Comments

@ndench
Copy link
Contributor

ndench commented Dec 28, 2023

Bug Report

Q A
BC Break yes
Version all

Summary

Symfony has deprecated the Templating component in 6.x and it's been removed in 7.0. In order to support Symfony 7 we'll need to update the usages of the Templating component to Twig.

@knallcharge
Copy link
Contributor

knallcharge commented Jan 11, 2024

I have tried and removed "implements Helper" from UploadHelper altogether (as the Helper only has a getter and setter for Charset which is not used in this bundle), and everything seems to work fine.

@ndench
Copy link
Contributor Author

ndench commented Jan 11, 2024

That's great @knallcharge! Would you be able to submit a PR with the change?

@knallcharge
Copy link
Contributor

I have already included all the other sf7-fixes from this fork (https://github.com/evertharmeling/OneupUploaderBundle/tree/sf-7) into mine, so I'll have to setup a new one for it. Basically it's just removing the interface-implementation, but I can look into probably sometime on Monday.

@knallcharge
Copy link
Contributor

I just created a PR: #438
Note that all sf7-fixes from evertharmeling are also included so credit for that goes to him. I can't create another fork with just the templating-change without deleting my old one which is currently in use, therefore the combined PR.
Not using all featues of the bundle this works for me but a close look is recommended. Also this may break custom filesystem-implementation that may have used the get/setCharset-methods.

@bytehead
Copy link
Member

What if I merge the changes from @evertharmeling and you can create a follow up PR? I like to keep thing simple and keep the credits.

@bytehead bytehead added this to the 5.x milestone Jan 15, 2024
@knallcharge
Copy link
Contributor

Sure, good idea.

@bytehead
Copy link
Member

bytehead commented Feb 5, 2024

@knallcharge I've merged the PR from @evertharmeling 👍

@knallcharge
Copy link
Contributor

@bytehead
check it out: #440

Reminder :)
"Not using all featues of the bundle this works for me but a close look is recommended. Also this may break custom filesystem-implementation that may have used the get/setCharset-methods."

@bytehead
Copy link
Member

Fixed in #440

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

No branches or pull requests

3 participants