Changeset 93651 in spip-zone for _core_/plugins/revisions/inc/diff.php


Ignore:
Timestamp:
Dec 13, 2015, 2:51:19 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

File:
1 edited

Legend:

Unmodified
Added
Removed
  • _core_/plugins/revisions/inc/diff.php

    r93622 r93651  
    5353                for ($len = $max_len; $len > 0; $len--) {
    5454                        if ($paths_ymin[$len] < $y) {
    55                                 $paths_ymin[$len+1] = $y;
    56                                 $paths[$len+1] = $paths[$len];
    57                                 $paths[$len+1][$y] = $c;
     55                                $paths_ymin[$len + 1] = $y;
     56                                $paths[$len + 1] = $paths[$len];
     57                                $paths[$len + 1][$y] = $c;
    5858                                break;
    5959                        }
     
    6363                        $paths[1] = array($y => $c);
    6464                }
    65                 if ($len+1 > $max_len) {
    66                         $max_len = $len+1;
     65                if ($len + 1 > $max_len) {
     66                        $max_len = $len + 1;
    6767                }
    6868        }
     
    106106                        for ($len = $max_len; $len > 0; $len--) {
    107107                                if ($paths_ymin[$len] < $y) {
    108                                         $paths_ymin[$len+1] = $y;
     108                                        $paths_ymin[$len + 1] = $y;
    109109                                        // On construit le resultat sous forme de chaine d'abord,
    110110                                        // car les tableaux de PHP sont dispendieux en taille memoire
    111                                         $paths[$len+1] = $paths[$len] . " $x,$y";
     111                                        $paths[$len + 1] = $paths[$len] . " $x,$y";
    112112                                        break;
    113113                                }
    114114                        }
    115                         if ($len+1 > $max_len) {
    116                                 $max_len = $len+1;
     115                        if ($len + 1 > $max_len) {
     116                                $max_len = $len + 1;
    117117                        }
    118118                        if ($len == 0) {
     
    122122                }
    123123        }
    124         if (isset($paths[$max_len]) AND $paths[$max_len]) {
     124        if (isset($paths[$max_len]) and $paths[$max_len]) {
    125125                $path = explode(" ", $paths[$max_len]);
    126126                $u = $v = array();
     
    148148         * @var Object Objet Diff* (DiffTexte, DiffPara, DiffPhrase)
    149149         */
    150         var $diff;
    151         var $fuzzy;
     150        public $diff;
     151        public $fuzzy;
    152152
    153153        /**
     
    156156         * @param Object $diff Objet Diff* d'un texte ou morceau de texte
    157157         **/
    158         function __construct($diff) {
     158        public function __construct($diff) {
    159159                $this->diff = $diff;
    160160                $this->fuzzy = true;
     
    162162
    163163// http://code.spip.net/@comparer
    164         function comparer($new, $old) {
     164        public function comparer($new, $old) {
    165165                $paras = $this->diff->segmenter($new);
    166166                $paras_old = $this->diff->segmenter($old);
     
    248248 **/
    249249class DiffTexte {
    250         var $r;
     250        public $r;
    251251
    252252        /**
    253253         * Constructeur
    254254         **/
    255         function __construct() {
     255        public function __construct() {
    256256                $this->r = "";
    257257        }
    258258
    259259// http://code.spip.net/@_diff
    260         function _diff($p, $p_old) {
     260        public function _diff($p, $p_old) {
    261261                $diff = new Diff(new DiffPara);
    262262
     
    265265
    266266// http://code.spip.net/@fuzzy
    267         function fuzzy() {
     267        public function fuzzy() {
    268268                return true;
    269269        }
     
    275275         * @return string[]       Tableau de fragments (paragraphes)
    276276         **/
    277         function segmenter($texte) {
     277        public function segmenter($texte) {
    278278                return separer_paras($texte);
    279279        }
     
    281281        // NB :  rem=\"diff-\" est un signal pour la fonction "afficher_para_modifies"
    282282// http://code.spip.net/@ajouter
    283         function ajouter($p) {
     283        public function ajouter($p) {
    284284                $p = trim($p);
    285285                $this->r .= "\n\n\n<span class=\"diff-para-ajoute\" title=\"" . _T('revisions:diff_para_ajoute') . "\">" . $p . "</span rem=\"diff-\">";
     
    287287
    288288// http://code.spip.net/@supprimer
    289         function supprimer($p_old) {
     289        public function supprimer($p_old) {
    290290                $p_old = trim($p_old);
    291291                $this->r .= "\n\n\n<span class=\"diff-para-supprime\" title=\"" . _T('revisions:diff_para_supprime') . "\">" . $p_old . "</span rem=\"diff-\">";
     
    293293
    294294// http://code.spip.net/@deplacer
    295         function deplacer($p, $p_old) {
     295        public function deplacer($p, $p_old) {
    296296                $this->r .= "\n\n\n<span class=\"diff-para-deplace\" title=\"" . _T('revisions:diff_para_deplace') . "\">";
    297297                $this->r .= trim($this->_diff($p, $p_old));
     
    300300
    301301// http://code.spip.net/@comparer
    302         function comparer($p, $p_old) {
     302        public function comparer($p, $p_old) {
    303303                $this->r .= "\n\n\n" . $this->_diff($p, $p_old);
    304304        }
    305305
    306306// http://code.spip.net/@resultat
    307         function resultat() {
     307        public function resultat() {
    308308                return $this->r;
    309309        }
     
    316316 **/
    317317class DiffPara {
    318         var $r;
     318        public $r;
    319319
    320320        /** Constructeur */
    321         function __construct() {
     321        public function __construct() {
    322322                $this->r = "";
    323323        }
    324324
    325325// http://code.spip.net/@_diff
    326         function _diff($p, $p_old) {
     326        public function _diff($p, $p_old) {
    327327                $diff = new Diff(new DiffPhrase);
    328328
     
    331331
    332332// http://code.spip.net/@fuzzy
    333         function fuzzy() {
     333        public function fuzzy() {
    334334                return true;
    335335        }
    336336
    337337// http://code.spip.net/@segmenter
    338         function segmenter($texte) {
     338        public function segmenter($texte) {
    339339                $paras = array();
    340340                $texte = trim($texte);
    341341                while (preg_match('/[\.!\?\]]+\s*/u', $texte, $regs)) {
    342                         $p = strpos($texte, $regs[0])+strlen($regs[0]);
     342                        $p = strpos($texte, $regs[0]) + strlen($regs[0]);
    343343                        $paras[] = substr($texte, 0, $p);
    344344                        $texte = substr($texte, $p);
     
    352352
    353353// http://code.spip.net/@ajouter
    354         function ajouter($p) {
     354        public function ajouter($p) {
    355355                $this->r .= "<span class=\"diff-ajoute\" title=\"" . _T('revisions:diff_texte_ajoute') . "\">" . $p . "</span rem=\"diff-\">";
    356356        }
    357357
    358358// http://code.spip.net/@supprimer
    359         function supprimer($p_old) {
     359        public function supprimer($p_old) {
    360360                $this->r .= "<span class=\"diff-supprime\" title=\"" . _T('revisions:diff_texte_supprime') . "\">" . $p_old . "</span rem=\"diff-\">";
    361361        }
    362362
    363363// http://code.spip.net/@deplacer
    364         function deplacer($p, $p_old) {
     364        public function deplacer($p, $p_old) {
    365365                $this->r .= "<span class=\"diff-deplace\" title=\"" . _T('revisions:diff_texte_deplace') . "\">" . $this->_diff($p,
    366366                                $p_old) . "</span rem=\"diff-\">";
     
    368368
    369369// http://code.spip.net/@comparer
    370         function comparer($p, $p_old) {
     370        public function comparer($p, $p_old) {
    371371                $this->r .= $this->_diff($p, $p_old);
    372372        }
    373373
    374374// http://code.spip.net/@resultat
    375         function resultat() {
     375        public function resultat() {
    376376                return $this->r;
    377377        }
     
    384384 **/
    385385class DiffPhrase {
    386         var $r;
     386        public $r;
    387387
    388388        /** Constructeur */
    389         function __construct() {
     389        public function __construct() {
    390390                $this->r = "";
    391391        }
    392392
    393393// http://code.spip.net/@fuzzy
    394         function fuzzy() {
     394        public function fuzzy() {
    395395                return false;
    396396        }
    397397
    398398// http://code.spip.net/@segmenter
    399         function segmenter($texte) {
     399        public function segmenter($texte) {
    400400                $paras = array();
    401401                if (test_pcre_unicode()) {
     
    417417                                if ($punct == '[[') {
    418418                                        $avant = substr($texte, 0, $p) . $regs[5] . $punct;
    419                                         $texte = $regs[4] . substr($texte, $p+$l);
     419                                        $texte = $regs[4] . substr($texte, $p + $l);
    420420                                } else {
    421421                                        if ($punct == ']]') {
    422422                                                $avant = substr($texte, 0, $p) . $regs[5] . $punct;
    423                                                 $texte = substr($texte, $p+$l);
     423                                                $texte = substr($texte, $p + $l);
    424424                                        } // Attacher les raccourcis fermants au mot precedent
    425425                                        else {
    426426                                                if (preg_match(',^[\]}]+$,', $punct)) {
    427427                                                        $avant = substr($texte, 0, $p) . (isset($regs[5]) ? $regs[5] : '') . $punct;
    428                                                         $texte = $regs[4] . substr($texte, $p+$l);
     428                                                        $texte = $regs[4] . substr($texte, $p + $l);
    429429                                                } // Attacher les raccourcis ouvrants au mot suivant
    430430                                                else {
    431431                                                        if (isset($regs[5]) && $regs[5] && preg_match(',^[\[{]+$,', $punct)) {
    432432                                                                $avant = substr($texte, 0, $p) . $regs[5];
    433                                                                 $texte = $punct . substr($texte, $p+$l);
     433                                                                $texte = $punct . substr($texte, $p + $l);
    434434                                                        } // Les autres signes de ponctuation sont des mots a part entiere
    435435                                                        else {
    436436                                                                $avant = substr($texte, 0, $p);
    437437                                                                $milieu = $regs[0];
    438                                                                 $texte = substr($texte, $p+$l);
     438                                                                $texte = substr($texte, $p + $l);
    439439                                                        }
    440440                                                }
     
    442442                                }
    443443                        } else {
    444                                 $avant = substr($texte, 0, $p+$l);
    445                                 $texte = substr($texte, $p+$l);
     444                                $avant = substr($texte, 0, $p + $l);
     445                                $texte = substr($texte, $p + $l);
    446446                        }
    447447                        if ($avant) {
     
    460460
    461461// http://code.spip.net/@ajouter
    462         function ajouter($p) {
     462        public function ajouter($p) {
    463463                $this->r .= "<span class=\"diff-ajoute\" title=\"" . _T('revisions:diff_texte_ajoute') . "\">" . $p . "</span rem=\"diff-\"> ";
    464464        }
    465465
    466466// http://code.spip.net/@supprimer
    467         function supprimer($p_old) {
     467        public function supprimer($p_old) {
    468468                $this->r .= "<span class=\"diff-supprime\" title=\"" . _T('revisions:diff_texte_supprime') . "\">" . $p_old . "</span rem=\"diff-\"> ";
    469469        }
    470470
    471471// http://code.spip.net/@comparer
    472         function comparer($p, $p_old) {
     472        public function comparer($p, $p_old) {
    473473                $this->r .= $p;
    474474        }
    475475
    476476// http://code.spip.net/@resultat
    477         function resultat() {
     477        public function resultat() {
    478478                return $this->r;
    479479        }
     
    502502        return charset2unicode($texte, 'utf-8');
    503503}
    504 
    505 
    506 ?>
Note: See TracChangeset for help on using the changeset viewer.