Grsp0008 php #110
Grsp0008 php #110
Conversation
| } | ||
|
|
||
| public void checkIssue(FunctionCallTree functionCallTree) { | ||
| if(functionCallTree.callee().toString().equals("include") || functionCallTree.callee().toString().equals("require")){ |
There was a problem hiding this comment.
Il manque également les méthodes d'import include_once et require_once
| public void checkIssue(FunctionCallTree functionCallTree) { | ||
| if(functionCallTree.callee().toString().equals("include") || functionCallTree.callee().toString().equals("require")){ | ||
| ParenthesisedExpressionTree expressionTree = (ParenthesisedExpressionTree) functionCallTree.arguments().get(0); | ||
| if(!expressionTree.expression().toString().startsWith("\"/")){ |
There was a problem hiding this comment.
Cela ne fonctionnera malheureusement pas dans la plupart des cas. L'exemple de bonne solution de greenspector par exemple :
include __DIR__ . 'foo.php';Va générer une erreur avec votre check
|
SonarCloud Quality Gate failed. |
|
Bonjour @mouradmamassi , merci de prendre en comptes les retours de @KuSh ainsi que de corriger les pbs cités ci-dessus, SVP. merci à vous. (for information, we are migrating this repo to a new one : https://github.com/green-code-initiative/ecocode-mobile, I will report your modifications manually) |
|
Hi @mouradmamassi , |
dedece35
added
the
migrated_to_new_standrad_repo
To review GRSP0008 - [PHP] - Utiliser un chemin absolu pour un import de fichier (include)