Carnet Wiki

PSR_SPIP

Version 9 — 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

Ce sont des avis personnels.

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();
}

(confirmé par mm)

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);
}