Skip to content

Upgrade to Symfony 7.4#171

Open
swinkelmann wants to merge 16 commits into
developfrom
feature/symfony7-upgrade
Open

Upgrade to Symfony 7.4#171
swinkelmann wants to merge 16 commits into
developfrom
feature/symfony7-upgrade

Conversation

@swinkelmann
Copy link
Copy Markdown
Contributor

No description provided.

@swinkelmann swinkelmann requested a review from Phocacius May 12, 2026 09:22
Copy link
Copy Markdown
Member

@Phocacius Phocacius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Please update all symfony flex recipes, there are 13 pending changes (see https://symfony.com/doc/current/setup/upgrade_major.html#3-updating-recipes)
  • For doctrine, there are also major updates (collections: 2.6->3.1, doctrine-bundle: 2.18->3.2, orm: 2.20->3.6, persistence: 3.4->4.2). Did you check those?
  • There is the warning message Package doctrine/cache is abandoned, you should avoid using it. No replacement was suggested.. Did you check if doctrine/cache is still needed and if there is a replacement?

For the warning message PearInstaller is not available (removed in Composer 2.x); PearPlugin skipping registration. I opened a ticket in mnsami/composer-custom-directory-installer#52

Copy link
Copy Markdown
Member

@astroidex astroidex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@astroidex astroidex added this to the 5.0.0 milestone May 15, 2026
@swinkelmann swinkelmann force-pushed the feature/symfony7-upgrade branch from 6f0c90a to 9816772 Compare May 20, 2026 09:59
@swinkelmann
Copy link
Copy Markdown
Contributor Author

  • Updated all symfony recipes
  • Updated the doctrine/dbal dependency to ensure compatibility with current mapbender requirements

Important:
You must run
bin/console mapbender:database:upgrade
directly after deploying this branch to ensure a successful migration of all database values.
Make a full backup of your database before running the migration command!!

Copy link
Copy Markdown
Member

@Phocacius Phocacius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

Merge this last and change the branch references in mapbender/mapbender and mapbender/digitizer back to develop and composer update once more before merging this.

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.

3 participants