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

corrections mineures - version 5.3.2 #28

Open
ytennevin opened this issue May 2, 2023 · 1 comment
Open

corrections mineures - version 5.3.2 #28

ytennevin opened this issue May 2, 2023 · 1 comment

Comments

@ytennevin
Copy link
Collaborator

Je mets trois corrections mineures à apporter (c'est des diff générés avec git diff ):

1 - il y a deux erreurs de sql sur le code présent dans le fichier de la v5.2.11.md

Sinon Il me semble que tu (=bertrand) m'avais dit de ne pas prendre en compte le v5.2.X.md mais il est nécessaire pour poursuivre les patchs a priori.

2- si je met pas un
if ( !defined('APPLICATION_DIR') ) dans global.php devant le define,

j'ai un
Notice: Constant APPLICATION_DIR already defined in /data/app-pg/config/autoload/global.php on line 12
Il me semble que ça affecte l'application en CLI de pas mettre le define, donc il est nécessaire mais pas dans tous les contextes.

3- On a des pages en phtml qui gènère des notices sur le code php pour des variables non définies.

1-v5_2_11-patch.txt

2-application-dir-patch.txt

3-isset_variables-patch.txt

@ytennevin ytennevin changed the title corrections mineures corrections mineures - version 5.3.2 May 2, 2023
@bertrandgauthier
Copy link
Collaborator

Salut Yves,
merci pour ces retours ! :-)
Pourrais-tu carrément faire des pull requests dans ce genre de pb ? Je pense que ça serait plus simple/clair pour nous deux...

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

2 participants