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

Préparation pour mise à jour Symfony 5.4 #1669

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

stakovicz
Copy link
Contributor

Dernière MR avant la migration vers Symfony 5.4 :

  • Correction plus propre de la récupération des $_GET['sens'] Smarty
  • Correction de la validation du CSRF token qui pose problème en 5.4
  • Correction de la notice de PROD :

    2025-03-04T10:00:32.278Z [Tue Mar 04 10:00:32.278279 2025] [proxy_fcgi:error] [pid 2443:tid 140045705004736] [client 91.208.207.215:50776] [185.31.151.110] AH01071: Got error 'PHP message: PHP Notice: Constant AFUP_CHEMIN_RACINE already defined in /home/bas/app_ee619ef6-05d2-4b4c-978f-d8f85e5a49da/sources/Afup/Bootstrap/_Common.php on line 31PHP message: PHP Notice: Constant AFUP_PERSONNES_PHYSIQUES already defined in /home/bas/app_ee619ef6-05d2-4b4c-978f-d8f85e5a49da/sources/Afup/Bootstrap/_Common.php on line 34PHP message: PHP Notice: Constant AFUP_COTISATION_PERSONNE_PHYSIQUE already defined in /home/bas/app_ee619ef6-05d2-4b4c-978f-d8f85e5a49da/sources/Afup/Bootstrap/_Common.php on line 35', referer https://afup.org/pages/administration/index.php?page=forum_inscriptions&id_forum=49

Il restera la mise à jour de ekino/newrelic-bundle#1645
Et le découpage de symfony/symfony en multiple packages.

On sera en Symfony 5.4

@stakovicz stakovicz requested review from agallou and Mopolo March 4, 2025 12:48
@stakovicz stakovicz self-assigned this Mar 4, 2025
@stakovicz stakovicz merged commit 25da0b0 into afup:master Mar 4, 2025
4 checks passed
@stakovicz stakovicz deleted the prepare-upgrade-symfony-5-4 branch March 4, 2025 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants