Changeset 93650 in spip-zone


Ignore:
Timestamp:
Dec 13, 2015, 2:50:39 PM (5 years ago)
Author:
marcimat@…
Message:

Meilleure compatibilité avec PSR-2 et nos règles d'écriture, en appliquant
différents fix avec php-cs-fixers. Fixers appliqués ici :

'encoding', utf8
'eof_ending', un saut de ligne en fin de fichier
'elseif', elseif plutôt que else if
'function_call_space', espaces sur fonctions
'function_declaration', espaces sur fonctions
'function_typehint_space', espaces sur fonctions
'linefeed', sauts de ligne \n uniquement
'lowercase_constants', true, false, null en munuscule
'lowercase_keywords', mots clés PHP en lowercase
'method_argument_space', espaces sur appels de fonctions
'multiple_use', use unique sur fonctions anonymes
'newline_after_open_tag', ouverture de php… et c'est tout sur cette ligne
'operators_spaces', espaces de part et d'autres des opérateurs binaires
'parenthesis', pas d'espace juste après parenthèse ouvrante, ou avant parenthèse fermante
'php_closing_tag', pas de fermeture de php
'short_tag', tag PHP corrects
'trailing_spaces', pas d'espace qui traîne en fin de ligne
'visibility', déclarer 'public / private / protected' sur les méthodes

Location:
_core_/plugins/porte_plume
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • _core_/plugins/porte_plume/porte_plume_fonctions.php

    r93621 r93650  
    184184         * @return void
    185185         */
    186         function __construct($params = array()) {
     186        public function __construct($params = array()) {
    187187                foreach ($params as $p => $v) {
    188188                        if (isset($this->$p)) {
    … …  
    208208         *     Paramètres, soustrait de ceux qui ne sont pas valides
    209209         */
    210         function verif_params($nom, $params = array()) {
     210        public function verif_params($nom, $params = array()) {
    211211                // si markupset, on boucle sur les items
    212212                if (stripos($nom, 'markupSet') !== false) {
    … …  
    256256         *     False si l'identifiant cherché n'est pas trouvé
    257257         */
    258         function affecter(&$tableau, $identifiant, $params = array(), $lieu = 'dedans', $plusieurs = false) {
     258        public function affecter(&$tableau, $identifiant, $params = array(), $lieu = 'dedans', $plusieurs = false) {
    259259                static $cle_de_recherche = 'id'; // ou className ?
    260260
    … …  
    294294                                        return $params;
    295295                                } elseif ($lieu == 'apres') {
    296                                         array_splice($tableau, $trouve+1, 0, $les_params);
     296                                        array_splice($tableau, $trouve + 1, 0, $les_params);
    297297
    298298                                        return $params;
    … …  
    307307                        if (is_array($v)) {
    308308                                foreach ($v as $m => $n) {
    309                                         if (is_array($n) AND ($r = $this->affecter($tableau[$i][$m], $identifiant, $params, $lieu, $plusieurs))) {
     309                                        if (is_array($n) and ($r = $this->affecter($tableau[$i][$m], $identifiant, $params, $lieu, $plusieurs))) {
    310310                                                return $r;
    311311                                        }
    … …  
    332332         *     false si aucun paramètre à affecter, true sinon.
    333333         */
    334         function affecter_a_tous(&$tableau, $params = array(), $ids = array()) {
     334        public function affecter_a_tous(&$tableau, $params = array(), $ids = array()) {
    335335                if (!$params) {
    336336                        return false;
    … …  
    345345                // merge de premiere ligne
    346346                foreach ($tableau as $i => &$v) {
    347                         if (!$ids OR in_array($v['id'], $ids)) {
     347                        if (!$ids or in_array($v['id'], $ids)) {
    348348                                $tableau[$i] = array_merge($tableau[$i], $params);
    349349                        }
    … …  
    372372         *     array sinon : description de l'identifiant cherché.
    373373         */
    374         function set($identifiant, $params = array()) {
     374        public function set($identifiant, $params = array()) {
    375375                // prudence tout de meme a pas tout modifier involontairement (si array)
    376376                if (!$identifiant) {
    … …  
    397397         *     array sinon : Description de l'identifiant cherché.
    398398         */
    399         function get($identifiant) {
     399        public function get($identifiant) {
    400400                if ($a = $this->affecter($this->markupSet, $identifiant)) {
    401401                        return $a;
    … …  
    417417         *     array sinon : description de l'identifiant cherché.
    418418         */
    419         function afficher($identifiant) {
     419        public function afficher($identifiant) {
    420420                return $this->set($identifiant, array('display' => true));
    421421        }
    … …  
    433433         *     array sinon : description de l'identifiant cherché.
    434434         */
    435         function cacher($identifiant) {
     435        public function cacher($identifiant) {
    436436                return $this->set($identifiant, array('display' => false));
    437437        }
    … …  
    445445         *     false si aucun paramètre à affecter, true sinon.
    446446         */
    447         function afficherTout() {
     447        public function afficherTout() {
    448448                return $this->affecter_a_tous($this->markupSet, array('display' => true));
    449449        }
    … …  
    456456         *     false si aucun paramètre à affecter, true sinon.
    457457         */
    458         function cacherTout() {
     458        public function cacherTout() {
    459459                return $this->affecter_a_tous($this->markupSet, array('display' => false));
    460460        }
    … …  
    474474         *     False si l'identifiant cherché n'est pas trouvé
    475475         */
    476         function ajouterAvant($identifiant, $params) {
     476        public function ajouterAvant($identifiant, $params) {
    477477                return $this->affecter($this->markupSet, $identifiant, $params, 'avant');
    478478        }
    … …  
    491491         *     False si l'identifiant cherché n'est pas trouvé
    492492         */
    493         function ajouterPlusieursAvant($identifiant, $tableau_params) {
     493        public function ajouterPlusieursAvant($identifiant, $tableau_params) {
    494494                return $this->affecter($this->markupSet, $identifiant, $tableau_params, 'avant', true);
    495495        }
    … …  
    508508         *     False si l'identifiant cherché n'est pas trouvé
    509509         */
    510         function ajouterApres($identifiant, $params) {
     510        public function ajouterApres($identifiant, $params) {
    511511                return $this->affecter($this->markupSet, $identifiant, $params, 'apres');
    512512        }
    … …  
    525525         *     False si l'identifiant cherché n'est pas trouvé
    526526         */
    527         function ajouterPlusieursApres($identifiant, $tableau_params) {
     527        public function ajouterPlusieursApres($identifiant, $tableau_params) {
    528528                return $this->affecter($this->markupSet, $identifiant, $tableau_params, 'apres', true);
    529529        }
    … …  
    536536         * @return void
    537537         */
    538         function ajouterFonction($fonction) {
     538        public function ajouterFonction($fonction) {
    539539                if (false === strpos($this->functions, $fonction)) {
    540540                        $this->functions .= "\n" . $fonction . "\n";
    … …  
    549549         * @return void
    550550         */
    551         function enlever_elements_non_affiches(&$tableau) {
     551        public function enlever_elements_non_affiches(&$tableau) {
    552552                if ($tableau === null) { // utile ?
    553553                        $tableau = &$this->markupSet;
    … …  
    555555
    556556                foreach ($tableau as $p => &$v) {
    557                         if (isset($v['display']) AND !$v['display']) {
     557                        if (isset($v['display']) and !$v['display']) {
    558558                                unset($tableau[$p]);
    559559                                $tableau = array_values($tableau); // remettre les cles automatiques sinon json les affiche et ça plante.
    … …  
    584584         * @return void
    585585         **/
    586         function enlever_separateurs(&$tableau) {
     586        public function enlever_separateurs(&$tableau) {
    587587                if ($tableau === null) { // utile ?
    588588                        $tableau = &$this->markupSet;
    … …  
    592592                foreach ($tableau as $p => &$v) {
    593593                        if (isset($v['separator']) and $v['separator']) {
    594                                 if (isset($tableau[$p-1])) {
    595                                         if (!isset($tableau[$p-1]['className'])) {
    596                                                 $tableau[$p-1]['className'] = "";
     594                                if (isset($tableau[$p - 1])) {
     595                                        if (!isset($tableau[$p - 1]['className'])) {
     596                                                $tableau[$p - 1]['className'] = "";
    597597                                        }
    598                                         $tableau[$p-1]['className'] .= " separateur_avant";
    599                                 }
    600                                 if (isset($tableau[$p+1])) {
    601                                         if (!isset($tableau[$p+1]['className'])) {
    602                                                 $tableau[$p+1]['className'] = "";
     598                                        $tableau[$p - 1]['className'] .= " separateur_avant";
     599                                }
     600                                if (isset($tableau[$p + 1])) {
     601                                        if (!isset($tableau[$p + 1]['className'])) {
     602                                                $tableau[$p + 1]['className'] = "";
    603603                                        }
    604                                         $tableau[$p+1]['className'] .= " separateur separateur_apres $v[id]";
     604                                        $tableau[$p + 1]['className'] .= " separateur separateur_apres $v[id]";
    605605                                }
    606606                                unset($tableau[$p]);
    … …  
    623623         * (id, display, icone)
    624624         */
    625         function enlever_parametres_inutiles() {
     625        public function enlever_parametres_inutiles() {
    626626                foreach ($this as $p => $v) {
    627627                        if (!$v) {
    … …  
    640640                        }
    641641                }
    642                 unset ($this->_liste_params_autorises);
     642                unset($this->_liste_params_autorises);
    643643        }
    644644
    … …  
    649649         * @return string Déclaration json de la barre
    650650         */
    651         function creer_json() {
     651        public function creer_json() {
    652652                $barre = $this;
    653653                $type = $barre->nameSpace;
    … …  
    681681         *     - boolean false if error
    682682         */
    683         function json_export($var) {
     683        public function json_export($var) {
    684684                $asso = false;
    685685                switch (true) {

  • _core_/plugins/porte_plume/porte_plume_pipelines.php

    r93621 r93650  
    3535function autoriser_porteplume_previsualiser_dist($faire, $type, $id, $qui, $opt) {
    3636        return
    37                 (test_espace_prive() AND autoriser('ecrire'))
    38                 OR (!test_espace_prive() AND autoriser('afficher_public', 'porteplume'));
     37                (test_espace_prive() and autoriser('ecrire'))
     38                or (!test_espace_prive() and autoriser('afficher_public', 'porteplume'));
    3939}
    4040
    … …  
    107107        $js_previsu = find_in_path('javascript/jquery.previsu_spip.js');
    108108        $js_start = parametre_url(generer_url_public('porte_plume_start.js'), 'lang', $lang);
    109         if (defined('_VAR_MODE') AND _VAR_MODE == "recalcul") {
     109        if (defined('_VAR_MODE') and _VAR_MODE == "recalcul") {
    110110                $js_start = parametre_url($js_start, 'var_mode', 'recalcul');
    111111        }
    … …  
    150150                $css = direction_css(find_in_path('css/barre_outils.css'), lang_dir());
    151151                $css_icones = generer_url_public('barre_outils_icones.css');
    152                 if (defined('_VAR_MODE') AND _VAR_MODE == "recalcul") {
     152                if (defined('_VAR_MODE') and _VAR_MODE == "recalcul") {
    153153                        $css_icones = parametre_url($css_icones, 'var_mode', 'recalcul');
    154154                }

  • _core_/plugins/porte_plume/porte_plume_start.js_fonctions.php

    r93621 r93650  
    3737        $barres = array();
    3838        foreach ($sets as $set) {
    39                 if (($barre = barre_outils_initialiser($set)) AND is_object($barre)) {
     39                if (($barre = barre_outils_initialiser($set)) and is_object($barre)) {
    4040                        $barres[$set] = $barre;
    4141                }
Note: See TracChangeset for help on using the changeset viewer.