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

Force le getProjectDir() #394

Open
wants to merge 25 commits into
base: develop
Choose a base branch
from

Conversation

pengu-fr
Copy link

Fixes #393

pengu-fr and others added 7 commits July 13, 2020 19:48
…rs le chemin %kernel.project_dir% n'est pas correct (il renvoit le répertoire app au lieu de renvoyer la racine du projet).
…rs le chemin %kernel.project_dir% n'est pas correct (il renvoit le répertoire app au lieu de renvoyer la racine du projet).
…rs le chemin %kernel.project_dir% n'est pas correct (il renvoit le répertoire app au lieu de renvoyer la racine du projet).
Modificaiton des services docker d'initialisation.
@Arakmar
Copy link
Member

Arakmar commented Jul 20, 2020

Merci pour la correction ! Est-ce que tu peux squasher tes commits puis te rebaser sur develop avant que j'y merge ?

DILL44 and others added 6 commits July 21, 2020 02:50
…rs le chemin %kernel.project_dir% n'est pas correct (il renvoit le répertoire app au lieu de renvoyer la racine du projet).

Revert suppresion composer.json
This reverts commit 2c6c0b2.

Correction du Si composer.json n'est pas présent (en production), alors le chemin %kernel.project_dir% n'est pas correct (il renvoit le répertoire app au lieu de renvoyer la racine du projet).
@pengu-fr
Copy link
Author

Est-ce que c'est OK ? J'ai fais un squash, un rebase, mais sans trop comprendre... (j'ai jamais utilisé github avant, ni ces fonctions de git). J'avoue que je suis un peu perdu sur les opérations à faire, sur quel repo, quelle branche... etc...
Sinon, la modification faite est assez simple. Voici le patch :

Index: app/AppKernel.php
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
--- app/AppKernel.php	(revision baf74885dcb8298d98b425672c2ba57cc3f8ba0f)
+++ app/AppKernel.php	(revision 58f47cd715cc8740ff6739685d9d0894692bb600)
@@ -46,6 +46,11 @@
         return __DIR__;
     }
 
+    public function getProjectDir()
+    {
+        return \realpath($this->getRootDir() . "/../");
+    }
+
     public function getCacheDir()
     {
         return dirname(__DIR__).'/var/cache/'.$this->getEnvironment();

@Arakmar
Copy link
Member

Arakmar commented Nov 15, 2020

Je crois que je vais juste cherry-picker ton commit, y'a eu un petit soucis au rebasage je crois, il y a plein d'autres commit.
Je veux bien y merger vu que ça n'a pas d'impact mais honnêtement, je ne suis pas sûr de bien comprendre l'utilité, même en production.

Quelle est l'utilité de supprimer le composer.json en production si le cache Symfony n'est pas généré en amont ? (pour une image docker j'imagine ?)
Je m'explique : Les endroits ou on fait appel au project_dir c'est uniquement dans de la conf yaml. Une fois le cache généré, on n'y fait plus aucun appel et il ne devrait plus y avoir de problèmes même sans le composer.json. Je viens de tester ça fonctionne parfaitement sans le composer.json si le cache est généré avant (en env prod, pas dev mais c'est tout à fait normal).

Pour te donner un exemple, sur certains environnements de prod, j'ai mon cache généré lors du déploiement uniquement (avec quelques ajustements au niveau du warmup) et il reste ensuite en lecture seule par l'utilisateur de PHP pour l'exécution de l'application.

@raphodn
Copy link
Member

raphodn commented Sep 3, 2022

le commit à isoler ? 2c6c0b2

@raphodn raphodn requested a review from petitalb September 3, 2022 14:15
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.

Mauvais chemin %kernel.project_dir% lorsque composer.json n'est pas présent.
6 participants