Carnet Wiki

PSR_SPIP

Version 8 — May 2019 JLuc

Mise en oeuvre

Doc : Outils pour tester la conformité d’un code php avec les PSR-12

Jeu de règles pour SPIP
La version connue la plus aboutie est : https://gist.github.com/marcimat/0e83a42334ecdb0b9e31d4dd7cb97963

Évolutions envisagées


-  améliorer le jeu de règles si nécessaire
-  améliorer le fixer
-  le mettre en hook post commit d’une forge
* avec mantis sur github
* avec drone sur le gitea de git.spip.net

Corrections

- les différentes sortes de tabulations du codestyle de cerdic font toutes 2 espaces de large
-  > 4 c’est mieux

Règles à ajouter

Instruction après accolade ouvrante

Le fixer laisse les instructions après une accolade ouvrante et génère parfois corrige un code formant cela
Par exemple typiquement il corrige

if ($test) doit();

en

if ($test) { doit();
}

Il devrait plutôt fournir :

if ($test) { 
    doit();
}

elseif

Les elseif sont forcément sur la ligne de l’accolade fermante du if précédent

if ($untest) {
        $index = $donnees['index'];
        include_spip('about_truc')
        $truc = get_truc($index);
} elseif ($autrecas) {
        $index = $donnees['index'];
        include_spip('about_truc')
        $truc = get_truc($index);
}

mais il serait agréable de reporter le elseif à la ligne suivant l’accolade fermante

if ($untest) {
        $index = $donnees['index'];
        include_spip('about_truc')
        $truc = get_truc($index);
}
elseif ($autrecas) {
        $index = $donnees['index'];
        include_spip('about_truc')
        $truc = get_truc($index);
}