Skip to content

Add support for Symfony 8#431

Open
relthyg wants to merge 12 commits into
craue:masterfrom
relthyg:symfony8
Open

Add support for Symfony 8#431
relthyg wants to merge 12 commits into
craue:masterfrom
relthyg:symfony8

Conversation

@relthyg

@relthyg relthyg commented Dec 18, 2025

Copy link
Copy Markdown

No description provided.

@relthyg relthyg marked this pull request as draft December 18, 2025 13:43
@romainnorberg

Copy link
Copy Markdown

+1 🙏

@relthyg If you're interested, I can help.

@relthyg

relthyg commented Jan 5, 2026

Copy link
Copy Markdown
Author

Hey @romainnorberg,

I'm still on winter break and don't really have an overview of the situation at the moment. I'll probably get back to it next week. Feel free to get started if you feel like and have time.

@knallcharge

knallcharge commented Jan 9, 2026

Copy link
Copy Markdown

I have tried to upgrade my project to Symfony 8 using this PR but found some issues:

  1. There are some "Fatal error: Declaration of x must be compatible with y"-errors displayed.
  2. Accessing the get()-method of an request was deprecated in Symfony 7.4 and will throw an "Attempted to call an undefined method named "get" of class "Symfony\Component\HttpFoundation\Request"."-error in Symfony 8.
  3. XML-configs are deprecated since Symfony 7.4.

I have tried to fix the issues, you may want to compare my fork:
master...knallcharge:CraueFormFlowBundle:master

One thing I could not quite figure out: line 628 in Form/FormFlow.php where the instanceId is fetched from the request. Not being familiar with the internals of this bundle I didn't know if this is expected from the GET- or the POST-variables so I implemented a switch. Don't know if this is necessary, maybe someone can advice?

@romainnorberg

Copy link
Copy Markdown

Nice job @knallcharge 👍

@relthyg

relthyg commented Jan 19, 2026

Copy link
Copy Markdown
Author

Thanks for your contribution, @knallcharge! Currenty I'm working on making the tests green.

@relthyg relthyg marked this pull request as ready for review February 10, 2026 12:01
@relthyg

relthyg commented Feb 18, 2026

Copy link
Copy Markdown
Author

This PR branch has been in productive use with us for a week and could be merged.

@tjveldhuizen

Copy link
Copy Markdown

Can we do something to help moving this forward?

@romainnorberg

Copy link
Copy Markdown

This PR branch has been in productive use with us for a week and could be merged.

+1 🙏

@Florian-B

Copy link
Copy Markdown

Thx @relthyg !

@craue Any update on this PR ? It's currently blocking Symfony 8 upgrades

🙏

@romainnorberg

Copy link
Copy Markdown

Thx @relthyg !

@craue Any update on this PR ? It's currently blocking Symfony 8 upgrades

🙏

I'm using this fork for now: https://github.com/relthyg/CraueFormFlowBundle/tree/symfony8

{
            "type": "vcs",
            "url": "https://github.com/relthyg/CraueFormFlowBundle"
        },
        
        
...

        "craue/formflow-bundle": "dev-symfony8 as 3.0.99",

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

Successfully merging this pull request may close these issues.

5 participants