Changeset 93622 in spip-zone

Timestamp:

Dec 13, 2015, 11:52:51 AM (4 years ago)

Author:

cedric@…

Message:

Indentation et regles de codage selon ​http://www.spip.net/fr_article3497.html#regles_codage

Location:

_core_/plugins/revisions

Files:

• afficher_diff/champ.php (modified) (4 diffs)
• afficher_diff/id_rubrique.php (modified) (2 diffs)
• afficher_diff/jointure.php (modified) (3 diffs)
• base/revisions.php (modified) (6 diffs)
• formulaires/configurer_revisions_objets.php (modified) (3 diffs)
• formulaires/reviser.php (modified) (2 diffs)
• genie/optimiser_revisions.php (modified) (5 diffs)
• inc/diff.php (modified) (30 diffs)
• inc/revisions.php (modified) (49 diffs)
• inc/revisions_autoriser.php (modified) (5 diffs)
• inc/revisions_pipeline.php (modified) (16 diffs)
• inc/suivi_versions.php (modified) (12 diffs)
• prive/objets/contenu/revision_fonctions.php (modified) (2 diffs)
• prive/objets/liste/versions_fonctions.php (modified) (1 diff)
• prive/rss/revisions_fonctions.php (modified) (1 diff)
• prive/squelettes/contenu/revision_fonctions.php (modified) (1 diff)
• revisions_administrations.php (modified) (10 diffs)
• revisions_ieconfig.php (modified) (2 diffs)

_core_/plugins/revisions/afficher_diff/champ.php

@@ -11 +11 @@
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 include_spip('inc/diff');
@@ -17 +19 @@
 /**
  * Afficher le diff d'un champ texte generique
+ *
  * @param string $champ
  * @param string $old
@@ -24 +27 @@
  * @return string
  */
-function afficher_diff_champ_dist($champ, $old, $new, $format = 'diff'){
+function afficher_diff_champ_dist($champ, $old, $new, $format = 'diff') {
         // ne pas se compliquer la vie !
-        if ($old==$new)
-                $out = ($format!='complet'?'':$new);
-        else {
-                if($f = charger_fonction($champ,'afficher_diff', true)){
+        if ($old == $new) {
+                $out = ($format != 'complet' ? '' : $new);
+        } else {
+                if ($f = charger_fonction($champ, 'afficher_diff', true)) {
                         return $f($champ, $old, $new, $format);
                 }
-                
+
                 $diff = new Diff(new DiffTexte);
                 $n = preparer_diff($new);
                 $o = preparer_diff($old);
 
-                $out = afficher_diff($diff->comparer($n,$o));
-                if ($format == 'diff' OR $format == 'apercu')
+                $out = afficher_diff($diff->comparer($n, $o));
+                if ($format == 'diff' OR $format == 'apercu') {
                         $out = afficher_para_modifies($out, ($format == 'apercu'));
+                }
         }
+
         return $out;
 }
@@ -52 +57 @@
  * @return string
  */
-function afficher_para_modifies ($texte, $court = false) {
+function afficher_para_modifies($texte, $court = false) {
         // Limiter la taille de l'affichage
-        if ($court) $max = 200;
-        else $max = 2000;
+        if ($court) {
+                $max = 200;
+        } else {
+                $max = 2000;
+        }
 
         $texte_ret = "";
-        $paras = explode ("\n",$texte);
+        $paras = explode("\n", $texte);
         for ($i = 0; $i < count($paras) AND strlen($texte_ret) < $max; $i++) {
-                if (strpos($paras[$i], '"diff-')) $texte_ret .= $paras[$i]."\n\n";
+                if (strpos($paras[$i], '"diff-')) {
+                        $texte_ret .= $paras[$i] . "\n\n";
+                }
 #               if (strlen($texte_ret) > $max) $texte_ret .= '(...)';
         }
         $texte = $texte_ret;
+
         return $texte;
 }

_core_/plugins/revisions/afficher_diff/id_rubrique.php

@@ -11 +11 @@
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 include_spip('inc/diff');
@@ -26 +28 @@
  * @return string
  */
-function afficher_diff_id_rubrique_dist($champ, $old, $new, $format = 'diff'){
+function afficher_diff_id_rubrique_dist($champ, $old, $new, $format = 'diff') {
         // ne pas se compliquer la vie !
-        if ($old==$new)
+        if ($old == $new) {
                 $out = _T('info_dans_rubrique')
-                       . " <b>&#171;&nbsp;".generer_info_entite($new,'rubrique','titre')."&nbsp;&#187;</b>";
-        else
+                        . " <b>&#171;&nbsp;" . generer_info_entite($new, 'rubrique', 'titre') . "&nbsp;&#187;</b>";
+        } else {
                 $out = _T('revisions:version_deplace_rubrique',
-                                                         array('from'=> generer_info_entite($old,'rubrique','titre')
-                                                                         ,'to'=>generer_info_entite($new,'rubrique','titre')
-                                                         )
-                         );
+                        array(
+                                'from' => generer_info_entite($old, 'rubrique', 'titre')
+                        ,
+                                'to' => generer_info_entite($new, 'rubrique', 'titre')
+                        )
+                );
+        }
+
         return $out;
 }

_core_/plugins/revisions/afficher_diff/jointure.php

@@ -11 +11 @@
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 include_spip('inc/diff');
@@ -17 +19 @@
 /**
  * Afficher le diff d'un champ texte generique
+ *
  * @param string $champ
  * @param string $old
@@ -24 +27 @@
  * @return string
  */
-function afficher_diff_jointure_dist($champ, $old, $new, $format = 'diff'){
-        $join = substr($champ,9);
+function afficher_diff_jointure_dist($champ, $old, $new, $format = 'diff') {
+        $join = substr($champ, 9);
         $objet = objet_type($join);
 
-        $old = explode(',',$old);
-        $new = explode(',',$new);
+        $old = explode(',', $old);
+        $new = explode(',', $new);
 
         $liste = array();
 
         // les communs
-        $intersection = array_intersect($new,$old);
-        foreach($intersection as $id)
-                if ($id=intval(trim($id)))
-                        $liste[$id] = "<a href='".generer_url_entite($id,$objet)."' title='"._T(objet_info($objet,'texte_objet'))." $id'>".generer_info_entite($id,$objet,'titre')."</a>";
+        $intersection = array_intersect($new, $old);
+        foreach ($intersection as $id) {
+                if ($id = intval(trim($id))) {
+                        $liste[$id] = "<a href='" . generer_url_entite($id, $objet) . "' title='" . _T(objet_info($objet,
+                                        'texte_objet')) . " $id'>" . generer_info_entite($id, $objet, 'titre') . "</a>";
+                }
+        }
 
         // les supprimes
-        $old = array_diff($old,$intersection);
-        foreach($old as $id)
-                if ($id=intval(trim($id)))
+        $old = array_diff($old, $intersection);
+        foreach ($old as $id) {
+                if ($id = intval(trim($id))) {
                         $liste[$id] = "<span class='diff-supprime'>"
-                                      . "<a href='".generer_url_entite($id,$objet)."' title='"._T(objet_info($objet,'texte_objet'))." $id'>".generer_info_entite($id,$objet,'titre')."</a>"
-                                      . "</span>";
+                                . "<a href='" . generer_url_entite($id, $objet) . "' title='" . _T(objet_info($objet,
+                                        'texte_objet')) . " $id'>" . generer_info_entite($id, $objet, 'titre') . "</a>"
+                                . "</span>";
+                }
+        }
 
         // les ajoutes
-        $new = array_diff($new,$intersection);
-        foreach($new as $id)
-                if ($id=intval(trim($id)))
+        $new = array_diff($new, $intersection);
+        foreach ($new as $id) {
+                if ($id = intval(trim($id))) {
                         $liste[$id] = "<span class='diff-ajoute'>"
-                                      . "<a href='".generer_url_entite($id,$objet)."' title='"._T(objet_info($objet,'texte_objet'))." $id'>".generer_info_entite($id,$objet,'titre')."</a>"
-                                      . "</span>";
+                                . "<a href='" . generer_url_entite($id, $objet) . "' title='" . _T(objet_info($objet,
+                                        'texte_objet')) . " $id'>" . generer_info_entite($id, $objet, 'titre') . "</a>"
+                                . "</span>";
+                }
+        }
 
         ksort($liste);
-        $liste = implode(', ',$liste);
+        $liste = implode(', ', $liste);
+
         return $liste;
 }

_core_/plugins/revisions/base/revisions.php

@@ -13 +13 @@
 /**
  * Déclarations relatives à la base de données
- * 
+ *
  * @package SPIP\Revisions\Pipelines
-**/
+ **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 /**
@@ -28 +30 @@
  *     Déclarations d'interface pour le compilateur
  */
-function revisions_declarer_tables_interfaces($interface){
+function revisions_declarer_tables_interfaces($interface) {
 
-        $interface['table_des_tables']['versions']='versions';
+        $interface['table_des_tables']['versions'] = 'versions';
 
         return $interface;
@@ -37 +39 @@
 /**
  * Déclaration des jointures génériques
- * 
+ *
  * @pipeline declarer_tables_objets_sql
  * @param array $tables
@@ -44 +46 @@
  *     Description complétée des tables
  */
-function revisions_declarer_tables_objets_sql($tables){
+function revisions_declarer_tables_objets_sql($tables) {
 
         // jointures sur les mots pour tous les objets
-        $tables[]['tables_jointures'][]= 'versions';
+        $tables[]['tables_jointures'][] = 'versions';
 
         return $tables;
@@ -62 +64 @@
  *     Description complétée des tables
  */
-function revisions_declarer_tables_auxiliaires($tables_auxiliaires){
+function revisions_declarer_tables_auxiliaires($tables_auxiliaires) {
 
-        $spip_versions = array (
-                "id_version"    => "bigint(21) DEFAULT 0 NOT NULL",
-                "id_objet"              => "bigint(21) DEFAULT 0 NOT NULL",
-                "objet"                 => "VARCHAR (25) DEFAULT '' NOT NULL",
-                "date"  => "datetime DEFAULT '0000-00-00 00:00:00' NOT NULL",
-                "id_auteur"     => "VARCHAR(23) DEFAULT '' NOT NULL", # stocke aussi IP(v6)
-                "titre_version" => "text DEFAULT '' NOT NULL",
-                "permanent"     => "char(3) DEFAULT '' NOT NULL",
-                "champs"        => "text DEFAULT '' NOT NULL"
+        $spip_versions = array(
+                "id_version" => "bigint(21) DEFAULT 0 NOT NULL",
+                "id_objet" => "bigint(21) DEFAULT 0 NOT NULL",
+                "objet" => "VARCHAR (25) DEFAULT '' NOT NULL",
+                "date" => "datetime DEFAULT '0000-00-00 00:00:00' NOT NULL",
+                "id_auteur" => "VARCHAR(23) DEFAULT '' NOT NULL", # stocke aussi IP(v6)
+                "titre_version" => "text DEFAULT '' NOT NULL",
+                "permanent" => "char(3) DEFAULT '' NOT NULL",
+                "champs" => "text DEFAULT '' NOT NULL"
         );
 
-        $spip_versions_key = array (
-                "PRIMARY KEY"   => "id_version, id_objet, objet",
+        $spip_versions_key = array(
+                "PRIMARY KEY" => "id_version, id_objet, objet",
                 "KEY id_version" => "id_version",
                 "KEY id_objet" => "id_objet",
-                "KEY objet" => "objet");
+                "KEY objet" => "objet"
+        );
 
         $spip_versions_fragments = array(
-                "id_fragment"   => "int unsigned DEFAULT '0' NOT NULL",
-                "version_min"   => "int unsigned DEFAULT '0' NOT NULL",
-                "version_max"   => "int unsigned DEFAULT '0' NOT NULL",
-                "id_objet"      => "bigint(21) NOT NULL",
-                "objet"                 => "VARCHAR (25) DEFAULT '' NOT NULL",
-                "compress"      => "tinyint NOT NULL",
-                "fragment"      => "longblob"  # ici c'est VRAIMENT un blob (on y stocke du gzip)
+                "id_fragment" => "int unsigned DEFAULT '0' NOT NULL",
+                "version_min" => "int unsigned DEFAULT '0' NOT NULL",
+                "version_max" => "int unsigned DEFAULT '0' NOT NULL",
+                "id_objet" => "bigint(21) NOT NULL",
+                "objet" => "VARCHAR (25) DEFAULT '' NOT NULL",
+                "compress" => "tinyint NOT NULL",
+                "fragment" => "longblob"  # ici c'est VRAIMENT un blob (on y stocke du gzip)
         );
 
         $spip_versions_fragments_key = array(
-                "PRIMARY KEY"   => "id_objet, objet, id_fragment, version_min"
+                "PRIMARY KEY" => "id_objet, objet, id_fragment, version_min"
         );
 
@@ -98 +101 @@
         $tables_auxiliaires['spip_versions'] = array(
                 'field' => &$spip_versions,
-                'key' => &$spip_versions_key);
+                'key' => &$spip_versions_key
+        );
 
         $tables_auxiliaires['spip_versions_fragments'] = array(
                 'field' => &$spip_versions_fragments,
-                'key' => &$spip_versions_fragments_key);
+                'key' => &$spip_versions_fragments_key
+        );
 
         return $tables_auxiliaires;

_core_/plugins/revisions/formulaires/configurer_revisions_objets.php

@@ -11 +11 @@
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
-function formulaires_configurer_revisions_objets_charger_dist(){
-        if (!$objets = unserialize($GLOBALS['meta']['objets_versions']))
+function formulaires_configurer_revisions_objets_charger_dist() {
+        if (!$objets = unserialize($GLOBALS['meta']['objets_versions'])) {
                 $objets = array();
+        }
         $valeurs = array(
-                'objets_versions'=>$objets,
+                'objets_versions' => $objets,
         );
 
@@ -23 +26 @@
 }
 
-function formulaires_configurer_revisions_objets_traiter_dist(){
+function formulaires_configurer_revisions_objets_traiter_dist() {
 
         include_spip('inc/meta');
         $tables = serialize(_request('objets_versions'));
-        ecrire_meta('objets_versions',$tables);
+        ecrire_meta('objets_versions', $tables);
 
-        return array('message_ok'=>_T('config_info_enregistree'));
+        return array('message_ok' => _T('config_info_enregistree'));
 }
 
-function test_objet_versionable($desc){
+function test_objet_versionable($desc) {
         if (!$desc['editable']
                 OR !isset($desc['champs_versionnes'])
                 OR !count($desc['champs_versionnes'])
-        )
+        ) {
                 return '';
+        }
 
         // regarder si il y a un vrai champ versionne, pas seulement une jointure
-        foreach ($desc['champs_versionnes'] as $c){
-                if (strncmp($c,'jointure_',9)!=0)
+        foreach ($desc['champs_versionnes'] as $c) {
+                if (strncmp($c, 'jointure_', 9) != 0) {
                         return ' ';
+                }
         }
 
@@ -48 +53 @@
         return '';
 }
+
 ?>

_core_/plugins/revisions/formulaires/reviser.php

@@ -11 +11 @@
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 include_spip('base/abstract_sql');
 
-function trouver_version_inf($objet, $id_objet, $cond = ""){
+function trouver_version_inf($objet, $id_objet, $cond = "") {
         return sql_getfetsel(
-                                'id_version',
-                                'spip_versions',
-                                ($cond?"$cond AND ":"")."objet=".sql_quote($objet)." AND id_objet=".intval($id_objet),
-                                '',
-                                'id_version DESC',
-                                '0,1'
-                          );
+                'id_version',
+                'spip_versions',
+                ($cond ? "$cond AND " : "") . "objet=" . sql_quote($objet) . " AND id_objet=" . intval($id_objet),
+                '',
+                'id_version DESC',
+                '0,1'
+        );
 }
-function check_version_diff($objet, $id_objet, $id_version, $id_diff, $last_version = 0){
-        if (!$last_version)
-                $last_version = trouver_version_inf($objet,$id_objet);
+
+function check_version_diff($objet, $id_objet, $id_version, $id_diff, $last_version = 0) {
+        if (!$last_version) {
+                $last_version = trouver_version_inf($objet, $id_objet);
+        }
 
         // si pas de diff possible, on renvoi 0,0
-        if (!$last_version)
-                return array(0,0);
+        if (!$last_version) {
+                return array(0, 0);
+        }
 
-        if ($last_version==1)
-                return array(1,0);
+        if ($last_version == 1) {
+                return array(1, 0);
+        }
 
-        $id_version = max($id_version,2);
-        $id_version = min($id_version,$last_version);
+        $id_version = max($id_version, 2);
+        $id_version = min($id_version, $last_version);
 
         // verifier id_version
-        $id_version = trouver_version_inf($objet,$id_objet,"id_version<=".intval($id_version));
+        $id_version = trouver_version_inf($objet, $id_objet, "id_version<=" . intval($id_version));
 
         // si rien trouve on prend la derniere
-        if (!$id_version)
+        if (!$id_version) {
                 $id_version = $last_version;
+        }
 
         // minorer id_diff en fonction de id_version
-        $id_diff = min($id_diff,$id_version-1);
+        $id_diff = min($id_diff, $id_version-1);
         // verifier id_diff
-        $id_diff = trouver_version_inf($objet,$id_objet,"id_version<=".intval($id_diff));
+        $id_diff = trouver_version_inf($objet, $id_objet, "id_version<=" . intval($id_diff));
 
-        if (!$id_diff)
-                $id_diff = trouver_version_inf($objet,$id_objet,"id_version<".intval($id_version));
+        if (!$id_diff) {
+                $id_diff = trouver_version_inf($objet, $id_objet, "id_version<" . intval($id_version));
+        }
 
         // echec, on renvoi ce qu'on peut
-        if (!$id_diff)
+        if (!$id_diff) {
                 $id_diff = $id_version-1;
+        }
 
         return array($id_version, $id_diff);
 }
 
-function formulaires_reviser_charger_dist($objet, $id_objet, $id_version, $id_diff){
-        if (!$objets = unserialize($GLOBALS['meta']['objets_versions']))
+function formulaires_reviser_charger_dist($objet, $id_objet, $id_version, $id_diff) {
+        if (!$objets = unserialize($GLOBALS['meta']['objets_versions'])) {
                 $objets = array();
+        }
 
-        if (!in_array(table_objet_sql($objet),$objets))
+        if (!in_array(table_objet_sql($objet), $objets)) {
                 return false;
+        }
 
-        $last_version = trouver_version_inf($objet,$id_objet);
-        list($id_version,$id_diff) = check_version_diff($objet,$id_objet,$id_version,$id_diff,$last_version);
-        if (!$id_version)
+        $last_version = trouver_version_inf($objet, $id_objet);
+        list($id_version, $id_diff) = check_version_diff($objet, $id_objet, $id_version, $id_diff, $last_version);
+        if (!$id_version) {
                 return false;
-        
+        }
+
         $valeurs = array(
-                '_last_version'=>$last_version,
-                '_objet'=>$objet,
-                '_id_objet'=>$id_objet,
-                'id_version'=>$id_version,
-                'id_diff'=>$id_diff,
+                '_last_version' => $last_version,
+                '_objet' => $objet,
+                '_id_objet' => $id_objet,
+                'id_version' => $id_version,
+                'id_diff' => $id_diff,
         );
 
@@ -84 +96 @@
 }
 
-function formulaires_reviser_verifier_dist($objet, $id_objet, $id_version, $id_diff){
+function formulaires_reviser_verifier_dist($objet, $id_objet, $id_version, $id_diff) {
         $erreurs = array();
-        list($id_version,$id_diff) = check_version_diff($objet,$id_objet,_request('id_version'),_request('id_diff'));
-        set_request('id_version',$id_version);
-        set_request('id_diff',$id_diff);
+        list($id_version, $id_diff) = check_version_diff($objet, $id_objet, _request('id_version'), _request('id_diff'));
+        set_request('id_version', $id_version);
+        set_request('id_diff', $id_diff);
+
         return $erreurs;
 }
 
-function formulaires_reviser_traiter_dist($objet, $id_objet, $id_version, $id_diff){
-        $res = array('message_ok'=>'','editable'=>true);
+function formulaires_reviser_traiter_dist($objet, $id_objet, $id_version, $id_diff) {
+        $res = array('message_ok' => '', 'editable' => true);
 
         $id_version = _request('id_version');
         $id_diff = _request('id_diff');
 
-        if (_AJAX)
-                $res['message_ok'].="<script type='text/javascript'>if (window.jQuery) jQuery('#wysiwyg.revision').ajaxReload({args:{id_version:$id_version,id_diff:$id_diff},history:true});</script>";
-        else
-                $res['redirect'] = parametre_url(parametre_url(self(),'id_version',$id_version),'id_diff',$id_diff,'&');
+        if (_AJAX) {
+                $res['message_ok'] .= "<script type='text/javascript'>if (window.jQuery) jQuery('#wysiwyg.revision').ajaxReload({args:{id_version:$id_version,id_diff:$id_diff},history:true});</script>";
+        } else {
+                $res['redirect'] = parametre_url(parametre_url(self(), 'id_version', $id_version), 'id_diff', $id_diff, '&');
+        }
+
         return $res;
 }

_core_/plugins/revisions/genie/optimiser_revisions.php

@@ -16 +16 @@
  */
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 include_spip('base/abstract_sql');
@@ -22 +24 @@
 /**
  * Tâche Cron d'optimisation des révisions
- * 
+ *
  * @param int $last
  *     Timestamp de la dernière exécution de cette tâche
@@ -32 +34 @@
         optimiser_base_revisions();
         optimiser_tables_revision();
-        
+
         return 1;
 }
 
 /**
- * Supprimer les révisions des objets disparus 
+ * Supprimer les révisions des objets disparus
  */
-function optimiser_base_revisions(){
+function optimiser_base_revisions() {
         /**
          * On commence par récupérer la liste des types d'objet ayant au moins une révision
-         */ 
-        $objets_revises = sql_select('objet','spip_versions','id_version=1','objet');
-        
+         */
+        $objets_revises = sql_select('objet', 'spip_versions', 'id_version=1', 'objet');
+
         /**
          * Pour chaque objet, on va contruire un tableau des identifiants disparus
          * On supprimera ensuite les occurences dans spip_versions et spip_versions_fragments
          */
-        while ($objet = sql_fetch($objets_revises)){
+        while ($objet = sql_fetch($objets_revises)) {
                 $in = array();
                 $table = table_objet_sql($objet['objet']);
                 $id_table_objet = id_table_objet($objet['objet']);
                 $res = sql_select("A.id_objet AS id_objet, A.objet AS objet",
-                                                        "spip_versions AS A LEFT JOIN $table AS R
-                                                        ON R.$id_table_objet=A.id_objet AND A.objet=".sql_quote($objet['objet']),
-                                                        "R.$id_table_objet IS NULL AND A.objet=".sql_quote($objet['objet'])." AND A.id_objet > 0",
-                                                        "A.id_objet",
-                                                        "A.id_objet");
+                        "spip_versions AS A LEFT JOIN $table AS R
+                                                        ON R.$id_table_objet=A.id_objet AND A.objet=" . sql_quote($objet['objet']),
+                        "R.$id_table_objet IS NULL AND A.objet=" . sql_quote($objet['objet']) . " AND A.id_objet > 0",
+                        "A.id_objet",
+                        "A.id_objet");
 
                 while ($row = sql_fetch($res)) {
@@ -67 +69 @@
                 /**
                  * Si on a un array
-                 * On supprime toute occurence des objets disparus dans : 
+                 * On supprime toute occurence des objets disparus dans :
                  * -* spip_versions
                  * -* spip_versions_fragments
                  */
                 if ($in) {
-                        foreach(array('spip_versions','spip_versions_fragments') as $table){
-                                sql_delete($table,  sql_in('id_objet',array_keys($in)) . " AND objet=".sql_quote($objet['objet']));
+                        foreach (array('spip_versions', 'spip_versions_fragments') as $table) {
+                                sql_delete($table, sql_in('id_objet', array_keys($in)) . " AND objet=" . sql_quote($objet['objet']));
                         }
                 }
@@ -82 +84 @@
  * Optimisation des tables spip_versions et spip_versions_fragments
  */
-function optimiser_tables_revision(){
-        foreach(array('spip_versions','spip_versions_fragments') as $table){
+function optimiser_tables_revision() {
+        foreach (array('spip_versions', 'spip_versions_fragments') as $table) {
                 spip_log("debut d'optimisation de la table $table");
-                if (sql_optimize($table))
+                if (sql_optimize($table)) {
                         spip_log("fin d'optimisation de la table $table");
-                else
+                } else {
                         spip_log("Pas d'optimiseur necessaire pour $table");
+                }
         }
 }
+
 ?>

_core_/plugins/revisions/inc/diff.php

@@ -15 +15 @@
  *
  * @package SPIP\Revisions\Diff
-**/
-
-if (!defined("_ECRIRE_INC_VERSION")) return;
+ **/
+
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 
@@ -24 +26 @@
 
 /**
- * Calcule un LCS (Longest Common Subsequence) simplifié 
+ * Calcule un LCS (Longest Common Subsequence) simplifié
  *
  * Chaque chaîne est une permutation de l'autre et on passe en paramètre
@@ -32 +34 @@
  * @param array $s
  * @return array
-**/
+ **/
 function lcs_opt($s) {
         $n = count($s);
-        if (!$n) return array();
+        if (!$n) {
+                return array();
+        }
         $paths = array();
         $paths_ymin = array();
@@ -44 +48 @@
         $max = 400;
         foreach ($s as $y => $c) {
-                if ($max-- < 0) break;  # eviter l'explosion memoire des tres gros diff
+                if ($max-- < 0) {
+                        break;
+                }  # eviter l'explosion memoire des tres gros diff
                 for ($len = $max_len; $len > 0; $len--) {
                         if ($paths_ymin[$len] < $y) {
-                                $paths_ymin[$len + 1] = $y;
-                                $paths[$len + 1] = $paths[$len];
-                                $paths[$len + 1][$y] = $c;
+                                $paths_ymin[$len+1] = $y;
+                                $paths[$len+1] = $paths[$len];
+                                $paths[$len+1][$y] = $c;
                                 break;
                         }
@@ -57 +63 @@
                         $paths[1] = array($y => $c);
                 }
-                if ($len + 1 > $max_len) $max_len = $len + 1;
-        }
+                if ($len+1 > $max_len) {
+                        $max_len = $len+1;
+                }
+        }
+
         return $paths[$max_len];
 }
@@ -71 +80 @@
  * @param array $t
  * @return array
-**/
+ **/
 function lcs($s, $t) {
         $n = count($s);
         $p = count($t);
-        if (!$n || !$p) return array(0 => array(), 1 => array());
+        if (!$n || !$p) {
+                return array(0 => array(), 1 => array());
+        }
         $paths = array();
         $paths_ymin = array();
@@ -82 +93 @@
 
         // Insertion des points
-        foreach ($t as $y => $c) $t_pos[trim($c)][] = $y;
+        foreach ($t as $y => $c) {
+                $t_pos[trim($c)][] = $y;
+        }
 
         foreach ($s as $x => $c) {
                 $c = trim($c);
-                if (!isset($t_pos[$c])) continue;
+                if (!isset($t_pos[$c])) {
+                        continue;
+                }
                 krsort($t_pos[$c]);
                 foreach ($t_pos[$c] as $y) {
                         for ($len = $max_len; $len > 0; $len--) {
                                 if ($paths_ymin[$len] < $y) {
-                                        $paths_ymin[$len + 1] = $y;
+                                        $paths_ymin[$len+1] = $y;
                                         // On construit le resultat sous forme de chaine d'abord,
                                         // car les tableaux de PHP sont dispendieux en taille memoire
-                                        $paths[$len + 1] = $paths[$len]." $x,$y";
+                                        $paths[$len+1] = $paths[$len] . " $x,$y";
                                         break;
                                 }
                         }
-                        if ($len + 1 > $max_len) $max_len = $len + 1;
+                        if ($len+1 > $max_len) {
+                                $max_len = $len+1;
+                        }
                         if ($len == 0) {
                                 $paths_ymin[1] = $y;
@@ -113 +130 @@
                         $v[$y] = $x;
                 }
+
                 return array($u, $v);
         }
+
         return array(0 => array(), 1 => array());
 }
@@ -122 +141 @@
  *
  * @package SPIP\Revisions\Diff
-**/
+ **/
 class Diff {
         /**
          * Objet DiffX d'un texte ou partie de texte
-         * 
-         * @var Object Objet Diff* (DiffTexte, DiffPara, DiffPhrase) */
+         *
+         * @var Object Objet Diff* (DiffTexte, DiffPara, DiffPhrase)
+         */
         var $diff;
         var $fuzzy;
 
         /**
-         * Constructeur 
+         * Constructeur
          *
-         * @param Object $diff    Objet Diff* d'un texte ou morceau de texte
-        **/
+         * @param Object $diff Objet Diff* d'un texte ou morceau de texte
+         **/
         function __construct($diff) {
                 $this->diff = $diff;
@@ -149 +169 @@
                         $lcs = lcs_opt($trans);
                         $lcs_rev = array_flip($lcs);
-                }
-                else {
+                } else {
                         list($trans_rev, $trans) = lcs($paras_old, $paras);
                         $lcs = $trans;
                         $lcs_rev = $trans_rev;
                 }
-        
+
                 reset($paras_old);
                 reset($paras);
@@ -177 +196 @@
                                 if (!isset($i_old)) {
                                         list($i_old, $p_old) = each($paras_old);
-                                        if (!$p_old) $fin_old = true;
+                                        if (!$p_old) {
+                                                $fin_old = true;
+                                        }
                                 }
                                 while (!$fin_old && $i_old < $j) {
@@ -185 +206 @@
                                         unset($i_old);
                                         list($i_old, $p_old) = each($paras_old);
-                                        if (!$p_old) $fin_old = true;
+                                        if (!$p_old) {
+                                                $fin_old = true;
+                                        }
                                 }
                         }
@@ -195 +218 @@
                         if (!isset($i_old)) {
                                 list($i_old, $p_old) = each($paras_old);
-                                if (!strlen($p_old)) $fin_old = true;
+                                if (!strlen($p_old)) {
+                                        $fin_old = true;
+                                }
                         }
                         while (!$fin_old) {
@@ -202 +227 @@
                                 }
                                 list($i_old, $p_old) = each($paras_old);
-                                if (!$p_old) $fin_old = true;
+                                if (!$p_old) {
+                                        $fin_old = true;
+                                }
                         }
                 }
@@ -210 +237 @@
                         }
                 }
+
                 return $this->diff->resultat();
         }
@@ -218 +246 @@
  *
  * @package SPIP\Revisions\Diff
-**/
+ **/
 class DiffTexte {
         var $r;
 
         /**
-         * Constructeur 
-        **/
+         * Constructeur
+         **/
         function __construct() {
                 $this->r = "";
@@ -232 +260 @@
         function _diff($p, $p_old) {
                 $diff = new Diff(new DiffPara);
+
                 return $diff->comparer($p, $p_old);
         }
@@ -241 +270 @@
 
         /**
-         * Découper les paragraphes d'un texte en fragments 
+         * Découper les paragraphes d'un texte en fragments
          *
-         * @param string $texte   Texte à fragmenter
+         * @param string $texte Texte à fragmenter
          * @return string[]       Tableau de fragments (paragraphes)
-        **/
+         **/
         function segmenter($texte) {
                 return separer_paras($texte);
@@ -254 +283 @@
         function ajouter($p) {
                 $p = trim($p);
-                $this->r .= "\n\n\n<span class=\"diff-para-ajoute\" title=\""._T('revisions:diff_para_ajoute')."\">".$p."</span rem=\"diff-\">";
-        }
+                $this->r .= "\n\n\n<span class=\"diff-para-ajoute\" title=\"" . _T('revisions:diff_para_ajoute') . "\">" . $p . "</span rem=\"diff-\">";
+        }
+
 // http://code.spip.net/@supprimer
         function supprimer($p_old) {
                 $p_old = trim($p_old);
-                $this->r .= "\n\n\n<span class=\"diff-para-supprime\" title=\""._T('revisions:diff_para_supprime')."\">".$p_old."</span rem=\"diff-\">";
-        }
+                $this->r .= "\n\n\n<span class=\"diff-para-supprime\" title=\"" . _T('revisions:diff_para_supprime') . "\">" . $p_old . "</span rem=\"diff-\">";
+        }
+
 // http://code.spip.net/@deplacer
         function deplacer($p, $p_old) {
-                $this->r .= "\n\n\n<span class=\"diff-para-deplace\" title=\""._T('revisions:diff_para_deplace')."\">";
+                $this->r .= "\n\n\n<span class=\"diff-para-deplace\" title=\"" . _T('revisions:diff_para_deplace') . "\">";
                 $this->r .= trim($this->_diff($p, $p_old));
                 $this->r .= "</span rem=\"diff-\">";
         }
+
 // http://code.spip.net/@comparer
         function comparer($p, $p_old) {
-                $this->r .= "\n\n\n".$this->_diff($p, $p_old);
-        }
-        
+                $this->r .= "\n\n\n" . $this->_diff($p, $p_old);
+        }
+
 // http://code.spip.net/@resultat
         function resultat() {
@@ -282 +314 @@
  *
  * @package SPIP\Revisions\Diff
-**/
+ **/
 class DiffPara {
         var $r;
@@ -294 +326 @@
         function _diff($p, $p_old) {
                 $diff = new Diff(new DiffPhrase);
+
                 return $diff->comparer($p, $p_old);
         }
@@ -301 +334 @@
                 return true;
         }
+
 // http://code.spip.net/@segmenter
         function segmenter($texte) {
@@ -306 +340 @@
                 $texte = trim($texte);
                 while (preg_match('/[\.!\?\]]+\s*/u', $texte, $regs)) {
-                        $p = strpos($texte, $regs[0]) + strlen($regs[0]);
+                        $p = strpos($texte, $regs[0])+strlen($regs[0]);
                         $paras[] = substr($texte, 0, $p);
                         $texte = substr($texte, $p);
                 }
-                if ($texte) $paras[] = $texte;
+                if ($texte) {
+                        $paras[] = $texte;
+                }
+
                 return $paras;
         }
@@ -316 +353 @@
 // http://code.spip.net/@ajouter
         function ajouter($p) {
-                $this->r .= "<span class=\"diff-ajoute\" title=\""._T('revisions:diff_texte_ajoute')."\">".$p."</span rem=\"diff-\">";
-        }
+                $this->r .= "<span class=\"diff-ajoute\" title=\"" . _T('revisions:diff_texte_ajoute') . "\">" . $p . "</span rem=\"diff-\">";
+        }
+
 // http://code.spip.net/@supprimer
         function supprimer($p_old) {
-                $this->r .= "<span class=\"diff-supprime\" title=\""._T('revisions:diff_texte_supprime')."\">".$p_old."</span rem=\"diff-\">";
-        }
+                $this->r .= "<span class=\"diff-supprime\" title=\"" . _T('revisions:diff_texte_supprime') . "\">" . $p_old . "</span rem=\"diff-\">";
+        }
+
 // http://code.spip.net/@deplacer
         function deplacer($p, $p_old) {
-                $this->r .= "<span class=\"diff-deplace\" title=\""._T('revisions:diff_texte_deplace')."\">".$this->_diff($p, $p_old)."</span rem=\"diff-\">";
-        }
+                $this->r .= "<span class=\"diff-deplace\" title=\"" . _T('revisions:diff_texte_deplace') . "\">" . $this->_diff($p,
+                                $p_old) . "</span rem=\"diff-\">";
+        }
+
 // http://code.spip.net/@comparer
         function comparer($p, $p_old) {
                 $this->r .= $this->_diff($p, $p_old);
         }
-        
+
 // http://code.spip.net/@resultat
         function resultat() {
@@ -341 +382 @@
  *
  * @package SPIP\Revisions\Diff
-**/
+ **/
 class DiffPhrase {
         var $r;
@@ -354 +395 @@
                 return false;
         }
+
 // http://code.spip.net/@segmenter
         function segmenter($texte) {
                 $paras = array();
                 if (test_pcre_unicode()) {
-                        $punct = '([[:punct:]]|'.plage_punct_unicode().')';
+                        $punct = '([[:punct:]]|' . plage_punct_unicode() . ')';
                         $mode = 'u';
-                }
-                else {
+                } else {
                         // Plages de poncutation pour preg_match bugge (ha ha)
-                        $punct = '([^\w\s\x80-\xFF]|'.plage_punct_unicode().')';
+                        $punct = '([^\w\s\x80-\xFF]|' . plage_punct_unicode() . ')';
                         $mode = '';
                 }
-                $preg = '/('.$punct.'+)(\s+|$)|(\s+)('.$punct.'*)/'.$mode;
+                $preg = '/(' . $punct . '+)(\s+|$)|(\s+)(' . $punct . '*)/' . $mode;
                 while (preg_match($preg, $texte, $regs)) {
                         $p = strpos($texte, $regs[0]);
@@ -376 +417 @@
                                 if ($punct == '[[') {
                                         $avant = substr($texte, 0, $p) . $regs[5] . $punct;
-                                        $texte = $regs[4] . substr($texte, $p + $l);
-                                }
-                                else
-                                if ($punct == ']]') {
-                                        $avant = substr($texte, 0, $p) . $regs[5] . $punct;
-                                        $texte = substr($texte, $p + $l);
-                                }
-                                // Attacher les raccourcis fermants au mot precedent
-                                else
-                                if (preg_match(',^[\]}]+$,', $punct)) {
-                                        $avant = substr($texte, 0, $p) . (isset($regs[5])?$regs[5]:'') . $punct;
-                                        $texte = $regs[4] . substr($texte, $p + $l);
-                                }
-                                // Attacher les raccourcis ouvrants au mot suivant
-                                else if (isset($regs[5]) && $regs[5] && preg_match(',^[\[{]+$,', $punct)) {
-                                        $avant = substr($texte, 0, $p) . $regs[5];
-                                        $texte = $punct . substr($texte, $p + $l);
-                                }
-                                // Les autres signes de ponctuation sont des mots a part entiere
-                                else {
-                                        $avant = substr($texte, 0, $p);
-                                        $milieu = $regs[0];
-                                        $texte = substr($texte, $p + $l);
-                                }
-                        }
-                        else {
-                                $avant = substr($texte, 0, $p + $l);
-                                $texte = substr($texte, $p + $l);
-                        }
-                        if ($avant) $paras[] = $avant;
-                        if ($milieu) $paras[] = $milieu;
-                }
-                if ($texte) $paras[] = $texte;
+                                        $texte = $regs[4] . substr($texte, $p+$l);
+                                } else {
+                                        if ($punct == ']]') {
+                                                $avant = substr($texte, 0, $p) . $regs[5] . $punct;
+                                                $texte = substr($texte, $p+$l);
+                                        } // Attacher les raccourcis fermants au mot precedent
+                                        else {
+                                                if (preg_match(',^[\]}]+$,', $punct)) {
+                                                        $avant = substr($texte, 0, $p) . (isset($regs[5]) ? $regs[5] : '') . $punct;
+                                                        $texte = $regs[4] . substr($texte, $p+$l);
+                                                } // Attacher les raccourcis ouvrants au mot suivant
+                                                else {
+                                                        if (isset($regs[5]) && $regs[5] && preg_match(',^[\[{]+$,', $punct)) {
+                                                                $avant = substr($texte, 0, $p) . $regs[5];
+                                                                $texte = $punct . substr($texte, $p+$l);
+                                                        } // Les autres signes de ponctuation sont des mots a part entiere
+                                                        else {
+                                                                $avant = substr($texte, 0, $p);
+                                                                $milieu = $regs[0];
+                                                                $texte = substr($texte, $p+$l);
+                                                        }
+                                                }
+                                        }
+                                }
+                        } else {
+                                $avant = substr($texte, 0, $p+$l);
+                                $texte = substr($texte, $p+$l);
+                        }
+                        if ($avant) {
+                                $paras[] = $avant;
+                        }
+                        if ($milieu) {
+                                $paras[] = $milieu;
+                        }
+                }
+                if ($texte) {
+                        $paras[] = $texte;
+                }
+
                 return $paras;
         }
@@ -414 +461 @@
 // http://code.spip.net/@ajouter
         function ajouter($p) {
-                $this->r .= "<span class=\"diff-ajoute\" title=\""._T('revisions:diff_texte_ajoute')."\">".$p."</span rem=\"diff-\"> ";
-        }
+                $this->r .= "<span class=\"diff-ajoute\" title=\"" . _T('revisions:diff_texte_ajoute') . "\">" . $p . "</span rem=\"diff-\"> ";
+        }
+
 // http://code.spip.net/@supprimer
         function supprimer($p_old) {
-                $this->r .= "<span class=\"diff-supprime\" title=\""._T('revisions:diff_texte_supprime')."\">".$p_old."</span rem=\"diff-\"> ";
-        }
+                $this->r .= "<span class=\"diff-supprime\" title=\"" . _T('revisions:diff_texte_supprime') . "\">" . $p_old . "</span rem=\"diff-\"> ";
+        }
+
 // http://code.spip.net/@comparer
         function comparer($p, $p_old) {
@@ -437 +486 @@
 
         $charset = $GLOBALS['meta']['charset'];
-        if ($charset == 'utf-8')
+        if ($charset == 'utf-8') {
                 return unicode_to_utf_8(html2unicode($texte));
+        }
+
         return unicode_to_utf_8(html2unicode(charset2unicode($texte, $charset, true)));
 }
@@ -445 +496 @@
 function afficher_diff($texte) {
         $charset = $GLOBALS['meta']['charset'];
-        if ($charset == 'utf-8') return $texte;
+        if ($charset == 'utf-8') {
+                return $texte;
+        }
+
         return charset2unicode($texte, 'utf-8');
 }

_core_/plugins/revisions/inc/revisions.php

@@ -15 +15 @@
  *
  * @package SPIP\Revisions\Fonctions
-**/
-if (!defined("_ECRIRE_INC_VERSION")) return;
+ **/
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 $GLOBALS['agregation_versions'] = 10;
 
 /** Intervalle de temps (en seconde) separant deux révisions par un même auteur */
-define('_INTERVALLE_REVISIONS', 600); 
+define('_INTERVALLE_REVISIONS', 600);
 
 /**
- * Découper les paragraphes d'un texte en fragments 
- *
- * @param string $texte  Texte à fragmenter
- * @param array $paras   Tableau de fragments déjà là
+ * Découper les paragraphes d'un texte en fragments
+ *
+ * @param string $texte Texte à fragmenter
+ * @param array $paras Tableau de fragments déjà là
  * @return string[]      Tableau de fragments (paragraphes)
-**/
+ **/
 function separer_paras($texte, $paras = array()) {
-        if (!$paras) $paras = array();
+        if (!$paras) {
+                $paras = array();
+        }
         while (preg_match("/(\r\n?){2,}|\n{2,}/", $texte, $regs)) {
-                $p = strpos($texte, $regs[0]) + strlen($regs[0]);
+                $p = strpos($texte, $regs[0])+strlen($regs[0]);
                 $paras[] = substr($texte, 0, $p);
                 $texte = substr($texte, $p);
         }
-        if ($texte) $paras[] = $texte;
+        if ($texte) {
+                $paras[] = $texte;
+        }
+
         return $paras;
 }
@@ -64 +71 @@
         // Attention a echapper $fragment, binaire potentiellement gz
         return array(
-                     'id_objet' => intval($id_objet),
-                         'objet' => $objet,
-                     'id_fragment' => intval($id_fragment),
-                     'version_min' => intval($version_min),
-                     'version_max' => intval($version_max),
-                     'compress' => $compress,
-                     'fragment' => $fragment);
+                'id_objet' => intval($id_objet),
+                'objet' => $objet,
+                'id_fragment' => intval($id_fragment),
+                'version_min' => intval($version_min),
+                'version_max' => intval($version_max),
+                'compress' => $compress,
+                'fragment' => $fragment
+        );
 }
 
@@ -76 +84 @@
 function envoi_replace_fragments($replaces) {
         $desc = $GLOBALS['tables_auxiliaires']['spip_versions_fragments'];
-        foreach($replaces as $r)
+        foreach ($replaces as $r) {
                 sql_replace('spip_versions_fragments', $r, $desc);
+        }
 }
 
@@ -84 +93 @@
 function envoi_delete_fragments($id_objet, $objet, $deletes) {
         if (count($deletes)) {
-                sql_delete("spip_versions_fragments", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND ((".  join(") OR (", $deletes)."))");
+                sql_delete("spip_versions_fragments",
+                        "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND ((" . join(") OR (",
+                                $deletes) . "))");
         }
 }
@@ -100 +111 @@
                 $nouveau = true;
                 // Recuperer la version la plus recente
-                $row = sql_fetsel("compress, fragment, version_min, version_max", "spip_versions_fragments", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_fragment=$id_fragment AND version_min<=$id_version", "", "version_min DESC", "1");
+                $row = sql_fetsel("compress, fragment, version_min, version_max", "spip_versions_fragments",
+                        "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_fragment=$id_fragment AND version_min<=$id_version",
+                        "", "version_min DESC", "1");
 
                 if ($row) {
                         $fragment = $row['fragment'];
                         $version_min = $row['version_min'];
-                        if ($row['compress'] > 0) $fragment = @gzuncompress($fragment);
+                        if ($row['compress'] > 0) {
+                                $fragment = @gzuncompress($fragment);
+                        }
                         $fragment = unserialize($fragment);
                         if (is_array($fragment)) {
@@ -117 +132 @@
                         $fragment = array($id_version => $texte);
                         $version_min = $id_version;
-                }
-                else {
+                } else {
                         // Ne pas dupliquer les fragments non modifies
                         $modif = true;
-                        for ($i = $id_version - 1; $i >= $version_min; $i--) {
+                        for ($i = $id_version-1; $i >= $version_min; $i--) {
                                 if (isset($fragment[$i])) {
                                         $modif = ($fragment[$i] != $texte);
@@ -127 +141 @@
                                 }
                         }
-                        if ($modif) $fragment[$id_version] = $texte;
+                        if ($modif) {
+                                $fragment[$id_version] = $texte;
+                        }
                 }
 
                 // Preparer l'enregistrement du fragment
-                $replaces[] = replace_fragment($id_objet,$objet, $version_min, $id_version, $id_fragment, $fragment);
+                $replaces[] = replace_fragment($id_objet, $objet, $version_min, $id_version, $id_fragment, $fragment);
         }
 
@@ -149 +165 @@
 
         // D'abord, vider les fragments inutiles
-        sql_delete("spip_versions_fragments", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND version_min>=$version_debut AND version_max<=$version_fin");
+        sql_delete("spip_versions_fragments",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND version_min>=$version_debut AND version_max<=$version_fin");
 
         // Fragments chevauchant l'ensemble de l'intervalle, s'ils existent
-        $result = sql_select("id_fragment, compress, fragment, version_min, version_max", "spip_versions_fragments", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND version_min<$version_debut AND version_max>$version_fin");
+        $result = sql_select("id_fragment, compress, fragment, version_min, version_max", "spip_versions_fragments",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND version_min<$version_debut AND version_max>$version_fin");
 
         while ($row = sql_fetch($result)) {
                 $id_fragment = $row['id_fragment'];
                 $fragment = $row['fragment'];
-                if ($row['compress'] > 0) $fragment = gzuncompress($fragment);
+                if ($row['compress'] > 0) {
+                        $fragment = gzuncompress($fragment);
+                }
                 $fragment = unserialize($fragment);
                 for ($i = $version_fin; $i >= $version_debut; $i--) {
                         if (isset($fragment[$i])) {
                                 // Recopier le dernier fragment si implicite
-                                if (!isset($fragment[$version_fin + 1]))
-                                        $fragment[$version_fin + 1] = $fragment[$i];
+                                if (!isset($fragment[$version_fin+1])) {
+                                        $fragment[$version_fin+1] = $fragment[$i];
+                                }
                                 unset($fragment[$i]);
                         }
                 }
 
-                $replaces[] = replace_fragment($id_objet,$objet,
+                $replaces[] = replace_fragment($id_objet, $objet,
                         $row['version_min'], $row['version_max'], $id_fragment, $fragment);
         }
 
         // Fragments chevauchant le debut de l'intervalle, s'ils existent
-        $result = sql_select("id_fragment, compress, fragment, version_min, version_max", "spip_versions_fragments", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND version_min<$version_debut AND version_max>=$version_debut AND version_max<=$version_fin");
+        $result = sql_select("id_fragment, compress, fragment, version_min, version_max", "spip_versions_fragments",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND version_min<$version_debut AND version_max>=$version_debut AND version_max<=$version_fin");
 
         $deb_fragment = array();
@@ -181 +203 @@
                 $version_min = $row['version_min'];
                 $version_max = $row['version_max'];
-                if ($row['compress'] > 0) $fragment = gzuncompress($fragment);
+                if ($row['compress'] > 0) {
+                        $fragment = gzuncompress($fragment);
+                }
                 $fragment = unserialize($fragment);
                 for ($i = $version_debut; $i <= $version_max; $i++) {
-                        if (isset($fragment[$i])) unset($fragment[$i]);
+                        if (isset($fragment[$i])) {
+                                unset($fragment[$i]);
+                        }
                 }
 
@@ -191 +217 @@
                 // Ajuster l'intervalle des versions
                 $deb_version_min[$id_fragment] = $version_min;
-                $deb_version_max[$id_fragment] = $version_debut - 1;
+                $deb_version_max[$id_fragment] = $version_debut-1;
         }
 
         // Fragments chevauchant la fin de l'intervalle, s'ils existent
-        $result = sql_select("id_fragment, compress, fragment, version_min, version_max", "spip_versions_fragments", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND version_max>$version_fin AND version_min>=$version_debut AND version_min<=$version_fin");
+        $result = sql_select("id_fragment, compress, fragment, version_min, version_max", "spip_versions_fragments",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND version_max>$version_fin AND version_min>=$version_debut AND version_min<=$version_fin");
 
         while ($row = sql_fetch($result)) {
@@ -202 +229 @@
                 $version_min = $row['version_min'];
                 $version_max = $row['version_max'];
-                if ($row['compress'] > 0) $fragment = gzuncompress($fragment);
+                if ($row['compress'] > 0) {
+                        $fragment = gzuncompress($fragment);
+                }
                 $fragment = unserialize($fragment);
                 for ($i = $version_fin; $i >= $version_min; $i--) {
                         if (isset($fragment[$i])) {
                                 // Recopier le dernier fragment si implicite
-                                if (!isset($fragment[$version_fin + 1]))
-                                        $fragment[$version_fin + 1] = $fragment[$i];
+                                if (!isset($fragment[$version_fin+1])) {
+                                        $fragment[$version_fin+1] = $fragment[$i];
+                                }
                                 unset($fragment[$i]);
                         }
@@ -218 +248 @@
                 $agreger = false;
                 if (isset($deb_fragment[$id_fragment])) {
-                        $agreger = (count($deb_fragment[$id_fragment]) + count($fragment) <= $agregation_versions);
+                        $agreger = (count($deb_fragment[$id_fragment])+count($fragment) <= $agregation_versions);
                         if ($agreger) {
-                                $fragment = $deb_fragment[$id_fragment] + $fragment;
+                                $fragment = $deb_fragment[$id_fragment]+$fragment;
                                 $version_min = $deb_version_min[$id_fragment];
-                        }
-                        else {
-                                $replaces[] = replace_fragment($id_objet,$objet,
+                        } else {
+                                $replaces[] = replace_fragment($id_objet, $objet,
                                         $deb_version_min[$id_fragment], $deb_version_max[$id_fragment],
                                         $id_fragment, $deb_fragment[$id_fragment]);
@@ -232 +261 @@
                 if (!$agreger) {
                         // Ajuster l'intervalle des versions
-                        $version_min = $version_fin + 1;
+                        $version_min = $version_fin+1;
                 }
                 $replaces[] = replace_fragment($id_objet, $objet, $version_min, $version_max, $id_fragment, $fragment);
@@ -240 +269 @@
         if (is_array($deb_fragment) && count($deb_fragment) > 0) {
                 foreach ($deb_fragment as $id_fragment => $fragment) {
-                        $replaces[] = replace_fragment($id_objet,$objet,
+                        $replaces[] = replace_fragment($id_objet, $objet,
                                 $deb_version_min[$id_fragment], $deb_version_max[$id_fragment],
                                 $id_fragment, $deb_fragment[$id_fragment]);
@@ -247 +276 @@
 
         envoi_replace_fragments($replaces);
-        envoi_delete_fragments($id_objet,$objet, $deletes);
+        envoi_delete_fragments($id_objet, $objet, $deletes);
 }
 
@@ -254 +283 @@
  * Récupérer les fragments d'un objet pour une version demandée
  *
- * @param int $id_objet    Identifiant de l'objet
- * @param string $objet    Objet
- * @param int $id_version  Identifiant de la version
+ * @param int $id_objet Identifiant de l'objet
+ * @param string $objet Objet
+ * @param int $id_version Identifiant de la version
  * @return array           Couples id_fragment => texte
  */
@@ -262 +291 @@
         $fragments = array();
 
-        if ($id_version == 0) return array();
+        if ($id_version == 0) {
+                return array();
+        }
 
         $result = sql_select(
                 "id_fragment, version_min, version_max, compress, fragment",
                 "spip_versions_fragments",
-                "id_objet=" . intval($id_objet) ." AND objet=".sql_quote($objet)
-                        ." AND version_min<=$id_version AND version_max>=$id_version");
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet)
+                . " AND version_min<=$id_version AND version_max>=$id_version");
 
         while ($row = sql_fetch($result)) {
@@ -275 +306 @@
                 $fragment = $row['fragment'];
                 // si le fragment est compressé, tenter de le décompresser, sinon écrire une erreur
-                if ($row['compress'] > 0){
+                if ($row['compress'] > 0) {
                         $fragment_ = @gzuncompress($fragment);
-                        if (strlen($fragment) && $fragment_===false){
+                        if (strlen($fragment) && $fragment_ === false) {
                                 $fragment = serialize(array($row['version_max'] => "[" . _T('forum_titre_erreur') . $id_fragment . "]"));
-                        }
-                        else
-                         $fragment = $fragment_;
+                        } else {
+                                $fragment = $fragment_;
+                        }
                 }
                 // tenter dedésérialiser le fragment, sinon écrire une erreur
                 $fragment_ = unserialize($fragment);
-                if (strlen($fragment) && $fragment_===false){
-                        $fragment=array($row['version_max']=>"["._T('forum_titre_erreur').$id_fragment."]");
-                }
-                else
-                 $fragment = $fragment_;
+                if (strlen($fragment) && $fragment_ === false) {
+                        $fragment = array($row['version_max'] => "[" . _T('forum_titre_erreur') . $id_fragment . "]");
+                } else {
+                        $fragment = $fragment_;
+                }
                 // on retrouve le fragment le plus près de notre version
                 for ($i = $id_version; $i >= $version_min; $i--) {
@@ -299 +330 @@
                                 ## la situation n'est pas meilleure ni pire qu'avant)
                                 if ($GLOBALS['meta']['charset'] == 'utf-8'
-                                AND include_spip('inc/charsets')
-                                AND !is_utf8($fragment[$i])) {
+                                        AND include_spip('inc/charsets')
+                                        AND !is_utf8($fragment[$i])
+                                ) {
                                         $fragment[$i] = importer_charset($fragment[$i], 'iso-8859-1');
                                 }
@@ -310 +342 @@
                 }
         }
+
         return $fragments;
 }
@@ -331 +364 @@
 
         // Nettoyage de la ponctuation pour faciliter l'appariement
-        foreach($src as $key => $val) {
+        foreach ($src as $key => $val) {
                 $t1[$key] = strval(preg_replace("/[[:punct:][:space:]]+/", " ", $val));
         }
-        foreach($dest as $key => $val) {
+        foreach ($dest as $key => $val) {
                 $t2[$key] = strval(preg_replace("/[[:punct:][:space:]]+/", " ", $val));
         }
 
         // Premiere passe : chercher les correspondance exactes
-        foreach($t1 as $key => $val) $md1[$key] = md5($val);
-        foreach($t2 as $key => $val) $md2[md5($val)][$key] = $key;
-        foreach($md1 as $key1 => $h) {
+        foreach ($t1 as $key => $val) {
+                $md1[$key] = md5($val);
+        }
+        foreach ($t2 as $key => $val) {
+                $md2[md5($val)][$key] = $key;
+        }
+        foreach ($md1 as $key1 => $h) {
                 if (isset($md2[$h])) {
                         $key2 = reset($md2[$h]);
@@ -356 +393 @@
         if ($flou) {
                 // Deuxieme passe : recherche de correlation par test de compressibilite
-                foreach($t1 as $key => $val) {
+                foreach ($t1 as $key => $val) {
                         $l1[$key] = strlen(gzcompress($val));
                 }
-                foreach($t2 as $key => $val) {
+                foreach ($t2 as $key => $val) {
                         $l2[$key] = strlen(gzcompress($val));
                 }
-                foreach($t1 as $key1 => $s1) {
-                        foreach($t2 as $key2 => $s2) {
-                                $r = strlen(gzcompress($s1.$s2));
-                                $taux = 1.0 * $r / ($l1[$key1] + $l2[$key2]);
+                foreach ($t1 as $key1 => $s1) {
+                        foreach ($t2 as $key2 => $s2) {
+                                $r = strlen(gzcompress($s1 . $s2));
+                                $taux = 1.0*$r/($l1[$key1]+$l2[$key2]);
                                 if (!isset($gz_min1[$key1]) || !$gz_min1[$key1] || $gz_min1[$key1] > $taux) {
                                         $gz_min1[$key1] = $taux;
                                         $gz_trans1[$key1] = $key2;
                                 }
-                                if (!isset($gz_min2[$key2]) ||!$gz_min2[$key2] || $gz_min2[$key2] > $taux) {
+                                if (!isset($gz_min2[$key2]) || !$gz_min2[$key2] || $gz_min2[$key2] > $taux) {
                                         $gz_min2[$key2] = $taux;
                                         $gz_trans2[$key2] = $key1;
@@ -379 +416 @@
                 // Depouiller les resultats de la deuxieme passe :
                 // ne retenir que les correlations reciproques
-                foreach($gz_trans1 as $key1 => $key2) {
+                foreach ($gz_trans1 as $key1 => $key2) {
                         if ($gz_trans2[$key2] == $key1 && $gz_min1[$key1] < 0.9) {
                                 $src_dest[$key1] = $key2;
@@ -395 +432 @@
  * Récupérer les champs d'un objet, pour une version demandée
  *
- * @param int $id_objet    Identifiant de l'objet
- * @param string $objet    Objet
- * @param int $id_version  Identifiant de la version
+ * @param int $id_objet Identifiant de l'objet
+ * @param string $objet Objet
+ * @param int $id_version Identifiant de la version
  * @return array           Couples champs => textes
  */
 function recuperer_version($id_objet, $objet, $id_version) {
 
-        $champs = sql_getfetsel("champs", "spip_versions", "id_objet=" . intval($id_objet) . " AND objet=".sql_quote($objet)." AND id_version=" . intval($id_version));
-        if (!$champs OR !is_array($champs = unserialize($champs)))
+        $champs = sql_getfetsel("champs", "spip_versions",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version=" . intval($id_version));
+        if (!$champs OR !is_array($champs = unserialize($champs))) {
                 return array();
-        else return reconstuire_version($champs,
-                         recuperer_fragments($id_objet,$objet, $id_version));
+        } else {
+                return reconstuire_version($champs,
+                        recuperer_fragments($id_objet, $objet, $id_version));
+        }
 }
 
@@ -415 +455 @@
  * retourne le texte de chaque champ.
  *
- * @param array $champs     Couples (champ => liste d'id_fragment).
+ * @param array $champs Couples (champ => liste d'id_fragment).
  *                          La liste est de la forme "5 32 7 16 8 2"
- * @param array $fragments  Couples (id_fragment => texte)
- * @param array $res        Couples (champ => texte) déjà connus
+ * @param array $fragments Couples (id_fragment => texte)
+ * @param array $res Couples (champ => texte) déjà connus
  * @return array            Couples (champ => texte)
  */
@@ -424 +464 @@
 
         static $msg;
-        if (!$msg) $msg = _T('forum_titre_erreur');
+        if (!$msg) {
+                $msg = _T('forum_titre_erreur');
+        }
 
         foreach ($champs as $nom => $code) {
@@ -437 +479 @@
                 }
         }
+
         return $res;
 }
@@ -442 +485 @@
 // http://code.spip.net/@supprimer_versions
 function supprimer_versions($id_objet, $objet, $version_min, $version_max) {
-        sql_delete("spip_versions", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_version>=$version_min AND id_version<=$version_max");
-
-        supprimer_fragments($id_objet,$objet, $version_min, $version_max);
+        sql_delete("spip_versions",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version>=$version_min AND id_version<=$version_max");
+
+        supprimer_fragments($id_objet, $objet, $version_min, $version_max);
 }
 
@@ -459 +503 @@
  *     Auteur apportant les modifications. En absence (session anonyme), utilisera l'IP pour garder une trace.
  * @return int
- *     id_version : identifiant de la version 
-**/
+ *     id_version : identifiant de la version
+ **/
 function ajouter_version($id_objet, $objet, $champs, $titre_version = "", $id_auteur = null) {
         $paras = $paras_old = $paras_champ = $fragments = array();
@@ -475 +519 @@
         // Detruire les tentatives d'archivages non abouties en 1 heure
         sql_delete('spip_versions', array(
-                "id_objet=" . intval($id_objet),
-                "objet=" . sql_quote($objet),
-                "id_version <= 0",
-                "date < DATE_SUB(" . sql_quote(date('Y-m-d H:i:s')) . ", INTERVAL " . _INTERVALLE_REVISIONS . " SECOND)")
+                        "id_objet=" . intval($id_objet),
+                        "objet=" . sql_quote($objet),
+                        "id_version <= 0",
+                        "date < DATE_SUB(" . sql_quote(date('Y-m-d H:i:s')) . ", INTERVAL " . _INTERVALLE_REVISIONS . " SECOND)"
+                )
         );
 
@@ -485 +530 @@
         // et un titre contenant en fait le moment de l'insertion
         list($ms, $sec) = explode(' ', microtime());
-        $date = $sec . substr($ms,1,4)-20; // SQL ne ramene que 4 chiffres significatifs apres la virgule pour 0.0+titre_version
-        $datediff = ($sec - mktime(0,0,0,9,1,2007)) * 1000000 + substr($ms,2, strlen($ms)-4);
-
-        $valeurs = array('id_objet' => $id_objet,
-                         'objet' => $objet,
-                         'id_version' => (0 - $datediff),
-                         'date' => date('Y-m-d H:i:s'),
-                         'id_auteur' => $str_auteur, //  varchar ici!
-                         'titre_version' => $date);
-        sql_insertq('spip_versions',  $valeurs);
+        $date = $sec . substr($ms, 1,
+                        4)-20; // SQL ne ramene que 4 chiffres significatifs apres la virgule pour 0.0+titre_version
+        $datediff = ($sec-mktime(0, 0, 0, 9, 1, 2007))*1000000+substr($ms, 2, strlen($ms)-4);
+
+        $valeurs = array(
+                'id_objet' => $id_objet,
+                'objet' => $objet,
+                'id_version' => (0-$datediff),
+                'date' => date('Y-m-d H:i:s'),
+                'id_auteur' => $str_auteur, //  varchar ici!
+                'titre_version' => $date
+        );
+        sql_insertq('spip_versions', $valeurs);
 
         // Eviter les validations entremelees en s'endormant s'il existe
@@ -508 +556 @@
 #         sleep(15);
         $delai = $sec-10;
-        while (sql_countsel('spip_versions', "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_version < 0 AND 0.0+titre_version < $date AND titre_version<>".sql_quote($date,'','text')." AND 0.0+titre_version > $delai")) {
-                spip_log("version $objet $id_objet :insertion en cours avant $date ($delai)",'revisions');
+        while (sql_countsel('spip_versions',
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version < 0 AND 0.0+titre_version < $date AND titre_version<>" . sql_quote($date,
+                        '', 'text') . " AND 0.0+titre_version > $delai")) {
+                spip_log("version $objet $id_objet :insertion en cours avant $date ($delai)", 'revisions');
                 sleep(1);
                 $delai++;
@@ -515 +565 @@
 #   sleep(15);  spip_log("sortie $sec $delai");
         // Determiner le numero du prochain fragment
-        $next = sql_fetsel("id_fragment", "spip_versions_fragments", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet), "", "id_fragment DESC", "1");
+        $next = sql_fetsel("id_fragment", "spip_versions_fragments",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet), "", "id_fragment DESC", "1");
 
         $onlylock = '';
 
         // Examiner la derniere version
-        $row = sql_fetsel("id_version, champs, id_auteur, date, permanent", "spip_versions", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_version > 0", '', "id_version DESC", "1"); // le champ id_auteur est un varchar dans cette table
+        $row = sql_fetsel("id_version, champs, id_auteur, date, permanent", "spip_versions",
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version > 0", '', "id_version DESC",
+                "1"); // le champ id_auteur est un varchar dans cette table
 
         if ($row) {
                 $id_version = $row['id_version'];
-                $paras_old = recuperer_fragments($id_objet,$objet, $id_version);
+                $paras_old = recuperer_fragments($id_objet, $objet, $id_version);
                 $champs_old = $row['champs'];
-                if ($row['id_auteur']!= $str_auteur
-                  OR $row['permanent']!='non'
-                  OR strtotime($row['date']) < (time()-_INTERVALLE_REVISIONS)) {
+                if ($row['id_auteur'] != $str_auteur
+                        OR $row['permanent'] != 'non'
+                        OR strtotime($row['date']) < (time()-_INTERVALLE_REVISIONS)
+                ) {
                         spip_log(strtotime($row['date']), 'revisions');
                         spip_log(time(), 'revisions');
@@ -540 +594 @@
                         $onlylock = 're';
                 }
-        } else
+        } else {
                 $id_version = 1;
-
-        $next = !$next ? 1 : ($next['id_fragment'] + 1);
+        }
+
+        $next = !$next ? 1 : ($next['id_fragment']+1);
 
         // Generer les nouveaux fragments
@@ -557 +612 @@
         if ($n) {
                 // Tables d'appariement dans les deux sens
-                list(,$trans) = apparier_paras($paras_old, $paras);
+                list(, $trans) = apparier_paras($paras_old, $paras);
                 reset($champs);
                 $nom = '';
@@ -563 +618 @@
                 // eviter une notice PHP au tout debut de la boucle
                 // on ajoute ''=>0 en debut de tableau.
-                $paras_champ = array($nom=>0) + $paras_champ;
+                $paras_champ = array($nom => 0)+$paras_champ;
 
                 for ($i = 0; $i < $n; $i++) {
-                        while ($i >= $paras_champ[$nom]) list($nom, ) = each($champs);
+                        while ($i >= $paras_champ[$nom]) {
+                                list($nom,) = each($champs);
+                        }
                         // Lier au fragment existant si possible, sinon creer un nouveau fragment
                         $id_fragment = isset($trans[$i]) ? $trans[$i] : $next++;
@@ -580 +637 @@
 
         // Enregistrer les modifications
-        ajouter_fragments($id_objet,$objet, $id_version, $fragments);
+        ajouter_fragments($id_objet, $objet, $id_version, $fragments);
 
         // Si l'insertion ne servait que de verrou,
@@ -587 +644 @@
 
         if (!$onlylock) {
-                sql_updateq('spip_versions', array('id_version'=>$id_version, 'date'=>date('Y-m-d H:i:s'), 'champs'=> serialize($codes), 'permanent'=>$permanent, 'titre_version'=> $titre_version), "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_version < 0 AND titre_version='$date'");
+                sql_updateq('spip_versions', array(
+                        'id_version' => $id_version,
+                        'date' => date('Y-m-d H:i:s'),
+                        'champs' => serialize($codes),
+                        'permanent' => $permanent,
+                        'titre_version' => $titre_version
+                ),
+                        "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version < 0 AND titre_version='$date'");
         } else {
-                sql_updateq('spip_versions', array('date'=>date('Y-m-d H:i:s'), 'champs'=>serialize($codes), 'permanent'=>$permanent, 'titre_version'=> $titre_version), "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_version=$id_version");
-                sql_delete("spip_versions", "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_version < 0 AND titre_version ='$date'");
-        }
-        spip_log($onlylock . "memorise la version $id_version de l'objet $objet $id_objet $titre_version",'revisions');
+                sql_updateq('spip_versions', array(
+                        'date' => date('Y-m-d H:i:s'),
+                        'champs' => serialize($codes),
+                        'permanent' => $permanent,
+                        'titre_version' => $titre_version
+                ), "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version=$id_version");
+                sql_delete("spip_versions",
+                        "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version < 0 AND titre_version ='$date'");
+        }
+        spip_log($onlylock . "memorise la version $id_version de l'objet $objet $id_objet $titre_version", 'revisions');
 
         return $id_version;
@@ -604 +674 @@
         $span_diff = array();
         if (preg_match_all(',<(/)?(span|div) (class|rem)="diff-[^>]*>,', $texte, $regs, PREG_SET_ORDER)) {
-                $regs = array_slice($regs,0,500); #limiter la casse s'il y en a trop
+                $regs = array_slice($regs, 0, 500); #limiter la casse s'il y en a trop
                 foreach ($regs as $c => $reg) {
-                        $texte = str_replace($reg[0], '@@@SPIP_DIFF'.$c.'@@@', $texte);
+                        $texte = str_replace($reg[0], '@@@SPIP_DIFF' . $c . '@@@', $texte);
                 }
         }
@@ -639 +709 @@
                 foreach ($area as $reg) {
                         $remplace = preg_replace(',@@@SPIP_DIFF[0-9]+@@@,', '**', $reg[0]);
-                        if ($remplace <> $reg[0])
+                        if ($remplace <> $reg[0]) {
                                 $texte = str_replace($reg[0], $remplace, $texte);
+                        }
                 }
         }
 
         // replacer les valeurs des <span> et <div> diff-
-        if (is_array($regs))
-        foreach ($regs as $c => $reg) {
-                $bal = (!$reg[1]) ? $reg[0] : "</$reg[2]>";
-                $texte = str_replace('@@@SPIP_DIFF'.$c.'@@@', $bal, $texte);
-                $GLOBALS['les_notes'] = str_replace('@@@SPIP_DIFF'.$c.'@@@', $bal, $GLOBALS['les_notes']);
+        if (is_array($regs)) {
+                foreach ($regs as $c => $reg) {
+                        $bal = (!$reg[1]) ? $reg[0] : "</$reg[2]>";
+                        $texte = str_replace('@@@SPIP_DIFF' . $c . '@@@', $bal, $texte);
+                        $GLOBALS['les_notes'] = str_replace('@@@SPIP_DIFF' . $c . '@@@', $bal, $GLOBALS['les_notes']);
+                }
         }
 
@@ -655 +727 @@
         // quand le dernier tag est ouvrant le refermer ...
         $reg = end($regs);
-        if (!$reg[1] AND $reg[2]) $texte.="</$reg[2]>";
+        if (!$reg[1] AND $reg[2]) {
+                $texte .= "</$reg[2]>";
+        }
 
         // et interdire_scripts !
@@ -666 +740 @@
 /**
  * Liste les champs versionnés d'une table objet.
- * 
+ *
  * @param string $table
  *     Nom complet de sa table sql. Exemple 'spip_articles'
@@ -675 +749 @@
         $liste_objets_versionnees = is_array(unserialize($GLOBALS['meta']['objets_versions'])) ? unserialize($GLOBALS['meta']['objets_versions']) : array();
 
-        if (!in_array($table,$liste_objets_versionnees))
+        if (!in_array($table, $liste_objets_versionnees)) {
                 return array();
+        }
 
         include_spip('base/objets');
-        if ($infos=lister_tables_objets_sql($table)
-          AND isset($infos['champs_versionnes']))
+        if ($infos = lister_tables_objets_sql($table)
+                AND isset($infos['champs_versionnes'])
+        ) {
                 return $infos['champs_versionnes'];
+        }
 
         return array();
@@ -689 +766 @@
  * Lorsqu'un champ versionée est une jointure, récuperer tous les liens
  * et les mettre sous forme de liste énumérée
- * 
+ *
  * @param string $objet
  * @param string $id_objet
@@ -695 +772 @@
  * @return string
  */
-function recuperer_valeur_champ_jointure($objet, $id_objet, $jointure){
+function recuperer_valeur_champ_jointure($objet, $id_objet, $jointure) {
         $objet_joint = objet_type($jointure);
         include_spip('action/editer_liens');
         $v = array();
         if (objet_associable($objet_joint)) {
-                $liens = objet_trouver_liens(array($objet_joint=>'*'),array($objet=>$id_objet));
-                foreach($liens as $l)
+                $liens = objet_trouver_liens(array($objet_joint => '*'), array($objet => $id_objet));
+                foreach ($liens as $l) {
                         $v[] = $l[$objet_joint];
-        }
-        elseif(objet_associable($objet)) {
-                $liens = objet_trouver_liens(array($objet=>$id_objet),array($objet_joint=>'*'));
-                foreach($liens as $l)
+                }
+        } elseif (objet_associable($objet)) {
+                $liens = objet_trouver_liens(array($objet => $id_objet), array($objet_joint => '*'));
+                foreach ($liens as $l) {
                         $v[] = $l[$objet];
+                }
         }
         sort($v);
-        return implode(",",$v);
+
+        return implode(",", $v);
 }
 
 /**
  * Créer la première révision d'un objet si nécessaire
- * 
+ *
  * À faire notamment si on vient d'activer l'extension et qu'on fait une modif
  * sur un objet qui était déjà en base, mais non versionné
@@ -721 +800 @@
  * La fonction renvoie le numéro de la dernière version de l'objet,
  * et 0 si pas de version pour cet objet
- * 
+ *
  * @param string $table
  * @param string $objet
@@ -729 +808 @@
  * @return int
  */
-function verifier_premiere_revision($table, $objet, $id_objet, $champs = null, $id_auteur = 0){
+function verifier_premiere_revision($table, $objet, $id_objet, $champs = null, $id_auteur = 0) {
 
         $id_table_objet = id_table_objet($objet);
-        if (!$champs)
+        if (!$champs) {
                 $champs = liste_champs_versionnes($table);
-        if (!$champs)
+        }
+        if (!$champs) {
                 return false;
-
-        if (!$id_version = sql_getfetsel('id_version','spip_versions',"id_objet=".intval($id_objet)." AND objet=".sql_quote($objet),'','id_version DESC','0,1')) {
+        }
+
+        if (!$id_version = sql_getfetsel('id_version', 'spip_versions',
+                "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet), '', 'id_version DESC', '0,1')
+        ) {
                 // recuperer toutes les valeurs actuelles des champs
                 // pour l'objet
-                $originaux = sql_fetsel("*", $table, "$id_table_objet=".intval($id_objet));
+                $originaux = sql_fetsel("*", $table, "$id_table_objet=" . intval($id_objet));
                 $premiere = false;
                 $champs_originaux = array();
 
-                foreach($champs as $v){
-                        if (isset($originaux[$v])){
+                foreach ($champs as $v) {
+                        if (isset($originaux[$v])) {
                                 $champs_originaux[$v] = $originaux[$v];
-                        }
-                        elseif (strncmp($v, 'jointure_', 9) == 0) {
-                                $champs_originaux[$v] = recuperer_valeur_champ_jointure($objet,$id_objet,substr($v,9));
+                        } elseif (strncmp($v, 'jointure_', 9) == 0) {
+                                $champs_originaux[$v] = recuperer_valeur_champ_jointure($objet, $id_objet, substr($v, 9));
                         }
                         if (isset($champs_originaux[$v]) AND isset($originaux[$v]) AND strlen($originaux[$v])) {
@@ -759 +841 @@
                 // il faut creer une premiere revision
                 if ($premiere) {
-                        $trouver_table = charger_fonction('trouver_table','base');
+                        $trouver_table = charger_fonction('trouver_table', 'base');
                         $desc = $trouver_table($table);
 
@@ -765 +847 @@
 
                         $date_modif = "";
-                        foreach(array('date_modif','maj') as $d){
-                                if (!$date_modif AND isset($originaux[$d]) AND $t=strtotime($d))
+                        foreach (array('date_modif', 'maj') as $d) {
+                                if (!$date_modif AND isset($originaux[$d]) AND $t = strtotime($d)) {
                                         $date_modif = date("Y-m-d H:i:s", $t-20);
+                                }
                         }
                         if (!$date_modif
-                          AND isset($desc['date'])
-                          AND isset($originaux[$desc['date']])) {
+                                AND isset($desc['date'])
+                                AND isset($originaux[$desc['date']])
+                        ) {
                                 $date_modif = $originaux[$desc['date']];
-                        }
-                        elseif (!$date_modif)
+                        } elseif (!$date_modif) {
                                 $date_modif = date("Y-m-d H:i:s", time()-20);
-                                
-                        if ($id_version = ajouter_version($id_objet, $objet, $champs_originaux, _T('revisions:version_initiale'), $id_auteur))
-                                sql_updateq('spip_versions', array('date' => $date_modif), "id_objet=".intval($id_objet)." AND objet=".sql_quote($objet)." AND id_version=$id_version");
-                }
-        }
+                        }
+
+                        if ($id_version = ajouter_version($id_objet, $objet, $champs_originaux, _T('revisions:version_initiale'),
+                                $id_auteur)
+                        ) {
+                                sql_updateq('spip_versions', array('date' => $date_modif),
+                                        "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet) . " AND id_version=$id_version");
+                        }
+                }
+        }
+
         return $id_version;
 }
+
 ?>

_core_/plugins/revisions/inc/revisions_autoriser.php

@@ -16 +16 @@
  *
  * @package SPIP\Revisions\Autorisations
-**/
+ **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 /**
  * Fonction du pipeline autoriser. N'a rien à faire.
+ *
  * @pipeline autoriser
  */
-function revisions_autoriser(){}
+function revisions_autoriser() { }
 
 /**
  * Autorisation de voir les revisions ?
- * 
+ *
  * Il faut :
  * - des revisions définies pour cet objet
@@ -35 +38 @@
  *
  * @param  string $faire Action demandée
- * @param  string $type  Type d'objet sur lequel appliquer l'action
- * @param  int    $id    Identifiant de l'objet
- * @param  array  $qui   Description de l'auteur demandant l'autorisation
- * @param  array  $opt   Options de cette autorisation
+ * @param  string $type Type d'objet sur lequel appliquer l'action
+ * @param  int $id Identifiant de l'objet
+ * @param  array $qui Description de l'auteur demandant l'autorisation
+ * @param  array $opt Options de cette autorisation
  * @return bool          true s'il a le droit, false sinon
  */
@@ -46 +49 @@
 
         include_spip('inc/revisions');
-        if (!liste_champs_versionnes($table))
+        if (!liste_champs_versionnes($table)) {
                 return false;
+        }
 
-        if (!$row = sql_fetsel("*", $table, "$id_table_objet=".intval($id)))
+        if (!$row = sql_fetsel("*", $table, "$id_table_objet=" . intval($id))) {
                 return false;
+        }
 
         return
@@ -59 +64 @@
 /**
  * Autorisation de voir le menu révisions
- * 
+ *
  * Il faut des révisions activées et présentes.
  *
  * @param  string $faire Action demandée
- * @param  string $type  Type d'objet sur lequel appliquer l'action
- * @param  int    $id    Identifiant de l'objet
- * @param  array  $qui   Description de l'auteur demandant l'autorisation
- * @param  array  $opt   Options de cette autorisation
+ * @param  string $type Type d'objet sur lequel appliquer l'action
+ * @param  int $id Identifiant de l'objet
+ * @param  array $qui Description de l'auteur demandant l'autorisation
+ * @param  array $opt Options de cette autorisation
  * @return bool          true s'il a le droit, false sinon
  */
-function autoriser_revisions_menu_dist($faire, $type = '', $id = 0, $qui = NULL, $opt = NULL){
+function autoriser_revisions_menu_dist($faire, $type = '', $id = 0, $qui = null, $opt = null) {
         // SI pas de revisions sur un objet quelconque.
         // ET pas de version... pas de bouton, c'est inutile...
@@ -76 +81 @@
                 return false;
         }
+
         return true;
 }

_core_/plugins/revisions/inc/revisions_pipeline.php

@@ -15 +15 @@
  *
  * @package SPIP\Revisions\Pipelines
-**/
-if (!defined('_ECRIRE_INC_VERSION')) return;
+ **/
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 
@@ -22 +24 @@
  * Ajoute dans le bloc d'info d'un objet un bouton permettant d'aller voir
  * l'historique de ses révisions
- * 
- * @param array $flux   Données du pipeline
+ *
+ * @param array $flux Données du pipeline
  * @return array $flux  Données du pipeline
  */
-function revisions_boite_infos($flux){
+function revisions_boite_infos($flux) {
         $type = $flux['args']['type'];
         if ($id = intval($flux['args']['id'])
-          AND $tables = unserialize($GLOBALS['meta']['objets_versions'])
-                AND in_array(table_objet_sql($type),$tables)
-          AND autoriser('voirrevisions',$type,$id)
-          // regarder le numero de revision le plus eleve, et afficher le bouton
-          // si c'est interessant (id_version>1)
-          AND sql_countsel('spip_versions', 'id_objet='.intval($id).' AND objet = '.sql_quote($type)) > 1
+                AND $tables = unserialize($GLOBALS['meta']['objets_versions'])
+                AND in_array(table_objet_sql($type), $tables)
+                AND autoriser('voirrevisions', $type, $id)
+                // regarder le numero de revision le plus eleve, et afficher le bouton
+                // si c'est interessant (id_version>1)
+                AND sql_countsel('spip_versions', 'id_objet=' . intval($id) . ' AND objet = ' . sql_quote($type)) > 1
         ) {
                 include_spip('inc/presentation');
-                $flux['data'] .= icone_horizontale(_T('revisions:info_historique_lien'), generer_url_ecrire('revision',"id_objet=$id&objet=$type"), "revision-24.png");
+                $flux['data'] .= icone_horizontale(_T('revisions:info_historique_lien'),
+                        generer_url_ecrire('revision', "id_objet=$id&objet=$type"), "revision-24.png");
         }
 
@@ -49 +52 @@
  * et sur la page de suivi de l'activité du site
  *
- * @param array $flux   Données du pipeline
+ * @param array $flux Données du pipeline
  * @return array $flux  Données du pipeline
  */
@@ -55 +58 @@
         // la bonne page et des objets révisables cochées !
         if (in_array($flux['args']['exec'], array('accueil', 'suivi_edito'))
-          and unserialize($GLOBALS['meta']['objets_versions'])) {
+                and unserialize($GLOBALS['meta']['objets_versions'])
+        ) {
                 $contexte = array();
-                if ($GLOBALS['visiteur_session']['statut']!=='0minirezo') {
+                if ($GLOBALS['visiteur_session']['statut'] !== '0minirezo') {
                         $contexte['id_auteur'] = $GLOBALS['visiteur_session']['id_auteur'];
                 }
-                $flux['data'] .= recuperer_fond('prive/objets/liste/versions',$contexte,array('ajax'=>true));
-        }
+                $flux['data'] .= recuperer_fond('prive/objets/liste/versions', $contexte, array('ajax' => true));
+        }
+
         return $flux;
 }
@@ -67 +72 @@
 /**
  * Définir les metas de configuration liées aux révisions
- * 
+ *
  * Utilisé par inc/config
  *
- * @param array $metas  Liste des métas et leurs valeurs par défaut
+ * @param array $metas Liste des métas et leurs valeurs par défaut
  * @return array        Liste des métas et leurs valeurs par défaut
  */
-function revisions_configurer_liste_metas($metas){
+function revisions_configurer_liste_metas($metas) {
         // Dorénavant dans les metas on utilisera un array serialisé de types d'objets
         // qui correspondront aux objets versionnés
-        $metas['objets_versions'] =  '';
+        $metas['objets_versions'] = '';
 
         return $metas;
@@ -84 +89 @@
 /**
  * Charge les données d'une révision donnée dans le formulaire d'édition d'un objet
- * 
- * @param array $flux   Données du pipeline
+ *
+ * @param array $flux Données du pipeline
  * @return array $flux  Données du pipeline
  */
-function revisions_formulaire_charger($flux){
-        if (strncmp($flux['args']['form'],'editer_',7)==0
-          AND $id_version = _request('id_version')
-          AND $objet = substr($flux['args']['form'],7)
-          AND $id_table_objet = id_table_objet($objet)
-          AND isset($flux['data'][$id_table_objet])
+function revisions_formulaire_charger($flux) {
+        if (strncmp($flux['args']['form'], 'editer_', 7) == 0
+                AND $id_version = _request('id_version')
+                AND $objet = substr($flux['args']['form'], 7)
+                AND $id_table_objet = id_table_objet($objet)
+                AND isset($flux['data'][$id_table_objet])
                 AND $id = intval($flux['data'][$id_table_objet])
-          AND !$flux['args']['je_suis_poste']){
+                AND !$flux['args']['je_suis_poste']
+        ) {
                 // ajouter un message convival pour indiquer qu'on a restaure la version
-                $flux['data']['message_ok'] = _T('revisions:icone_restaurer_version',array('version'=>$id_version));
-                $flux['data']['message_ok'] .= "<br />"._T('revisions:message_valider_recuperer_version');
+                $flux['data']['message_ok'] = _T('revisions:icone_restaurer_version', array('version' => $id_version));
+                $flux['data']['message_ok'] .= "<br />" . _T('revisions:message_valider_recuperer_version');
                 // recuperer la version
                 include_spip('inc/revisions');
-                $champs = recuperer_version($id,$objet, $id_version);
-                foreach($champs as $champ=>$valeur){
-                        if (!strncmp($champ,'jointure_',9)==0){
-                                if ($champ=='id_rubrique'){
+                $champs = recuperer_version($id, $objet, $id_version);
+                foreach ($champs as $champ => $valeur) {
+                        if (!strncmp($champ, 'jointure_', 9) == 0) {
+                                if ($champ == 'id_rubrique') {
                                         $flux['data']['id_parent'] = $valeur;
+                                } else {
+                                        $flux['data'][$champ] = $valeur;
                                 }
-                                else
-                                        $flux['data'][$champ] = $valeur;
                         }
                 }
         }
+
         return $flux;
 }
-
 
 
@@ -120 +126 @@
  * Sur une insertion en base, lever un flag pour ne pas creer une premiere révision vide
  * dans pre_edition mais attendre la post_edition pour cela
- * 
- * @param array $x   Données du pipeline
- * @return array $x  Données du pipeline
- */
-function revisions_post_insertion($x){
+ *
+ * @param array $x Données du pipeline
+ * @return array $x  Données du pipeline
+ */
+function revisions_post_insertion($x) {
         $table = $x['args']['table'];
         include_spip('inc/revisions');
-        if  ($champs = liste_champs_versionnes($table)) {
-                $GLOBALS['premiere_revision']["$table:".$x['args']['id_objet']] = true;
-        }
+        if ($champs = liste_champs_versionnes($table)) {
+                $GLOBALS['premiere_revision']["$table:" . $x['args']['id_objet']] = true;
+        }
+
         return $x;
 }
@@ -138 +145 @@
  * et la creer sinon avec l'etat actuel de l'objet
  *
- * @param array $x   Données du pipeline
+ * @param array $x Données du pipeline
  * @return array $x  Données du pipeline
  */
@@ -144 +151 @@
         // ne rien faire quand on passe ici en controle md5
         if (!isset($x['args']['action'])
-          OR $x['args']['action']!=='controler'){
+                OR $x['args']['action'] !== 'controler'
+        ) {
                 $table = $x['args']['table'];
                 include_spip('inc/revisions');
                 // si flag leve passer son chemin, post_edition le fera (mais baisser le flag en le gardant en memoire tout de meme)
-                if (isset($GLOBALS['premiere_revision']["$table:".$x['args']['id_objet']])){
-                        $GLOBALS['premiere_revision']["$table:".$x['args']['id_objet']] = 0;
-                }
-                // sinon creer une premiere revision qui date et dont on ne connait pas l'auteur
-                elseif  ($versionnes = liste_champs_versionnes($table)) {
+                if (isset($GLOBALS['premiere_revision']["$table:" . $x['args']['id_objet']])) {
+                        $GLOBALS['premiere_revision']["$table:" . $x['args']['id_objet']] = 0;
+                } // sinon creer une premiere revision qui date et dont on ne connait pas l'auteur
+                elseif ($versionnes = liste_champs_versionnes($table)) {
                         $objet = isset($x['args']['type']) ? $x['args']['type'] : objet_type($table);
                         verifier_premiere_revision($table, $objet, $x['args']['id_objet'], $versionnes, -1);
                 }
         }
+
         return $x;
 }
@@ -164 +172 @@
  * enregistrer une première révision de l'objet si nécessaire
  *
- * @param array $x   Données du pipeline
+ * @param array $x Données du pipeline
  * @return array $x  Données du pipeline
  */
 function revisions_pre_edition_lien($x) {
-        if (intval($x['args']['id_objet_source'])>0
-            AND intval($x['args']['id_objet'])>0) {
+        if (intval($x['args']['id_objet_source']) > 0
+                AND intval($x['args']['id_objet']) > 0
+        ) {
                 $table = table_objet_sql($x['args']['objet']);
                 $id_objet = intval($x['args']['id_objet']);
                 include_spip('inc/revisions');
-                if (isset($GLOBALS['premiere_revision']["$table:".$id_objet])){
-                        $GLOBALS['premiere_revision']["$table:".$id_objet] = 0;
-                }
-                // ex : si le champ jointure_mots est versionnable sur les articles
+                if (isset($GLOBALS['premiere_revision']["$table:" . $id_objet])) {
+                        $GLOBALS['premiere_revision']["$table:" . $id_objet] = 0;
+                } // ex : si le champ jointure_mots est versionnable sur les articles
                 elseif ($versionnes = liste_champs_versionnes($table)
-                        AND in_array($j='jointure_'.table_objet($x['args']['objet_source']),$versionnes)){
-                        verifier_premiere_revision($table,$x['args']['objet'],$id_objet,$versionnes,-1);
+                        AND in_array($j = 'jointure_' . table_objet($x['args']['objet_source']), $versionnes)
+                ) {
+                        verifier_premiere_revision($table, $x['args']['objet'], $id_objet, $versionnes, -1);
                 }
 
                 $table = table_objet_sql($x['args']['objet_source']);
                 $id_objet = $x['args']['id_objet_source'];
-                if (isset($GLOBALS['premiere_revision']["$table:".$id_objet])){
-                        $GLOBALS['premiere_revision']["$table:".$id_objet] = 0;
-                }
-                // ex : si le champ jointure_articles est versionnable sur les mots
+                if (isset($GLOBALS['premiere_revision']["$table:" . $id_objet])) {
+                        $GLOBALS['premiere_revision']["$table:" . $id_objet] = 0;
+                } // ex : si le champ jointure_articles est versionnable sur les mots
                 elseif ($versionnes = liste_champs_versionnes($table)
-                        AND in_array($j='jointure_'.table_objet($x['args']['objet']),$versionnes)){
-                        verifier_premiere_revision($table,$x['args']['objet_source'],$id_objet,$versionnes,-1);
+                        AND in_array($j = 'jointure_' . table_objet($x['args']['objet']), $versionnes)
+                ) {
+                        verifier_premiere_revision($table, $x['args']['objet_source'], $id_objet, $versionnes, -1);
                 }
         }
@@ -200 +209 @@
  * Après modification en base, versionner l'objet
  *
- * @param array $x   Données du pipeline
+ * @param array $x Données du pipeline
  * @return array $x  Données du pipeline
  */
 function revisions_post_edition($x) {
         include_spip('inc/revisions');
-        if (isset($x['args']['table']) and $versionnes = liste_champs_versionnes($x['args']['table'])){
+        if (isset($x['args']['table']) and $versionnes = liste_champs_versionnes($x['args']['table'])) {
                 // Regarder si au moins une des modifs est versionnable
                 $champs = array();
@@ -212 +221 @@
                 include_spip('inc/session');
 
-                if (isset($GLOBALS['premiere_revision']["$table:".$x['args']['id_objet']])){
-                        unset($GLOBALS['premiere_revision']["$table:".$x['args']['id_objet']]);
+                if (isset($GLOBALS['premiere_revision']["$table:" . $x['args']['id_objet']])) {
+                        unset($GLOBALS['premiere_revision']["$table:" . $x['args']['id_objet']]);
                         // verifier la premiere version : sur une version initiale on attend ici pour la creer
                         // plutot que de creer une version vide+un diff
                         verifier_premiere_revision($table, $objet, $x['args']['id_objet'], $versionnes, session_get('id_auteur'));
-                }
-                else {
+                } else {
                         // on versionne les differences
                         foreach ($versionnes as $key) {
@@ -227 +235 @@
 
                         if (count($champs)) {
-                                ajouter_version($x['args']['id_objet'],$objet, $champs, '', session_get('id_auteur'));
+                                ajouter_version($x['args']['id_objet'], $objet, $champs, '', session_get('id_auteur'));
                         }
                 }
@@ -239 +247 @@
  * Après modification en base d'un lien, versionner l'objet si nécessaire
  *
- * @param array $x   Données du pipeline
+ * @param array $x Données du pipeline
  * @return array $x  Données du pipeline
  */
@@ -256 +264 @@
                 )
         */
-        if (intval($x['args']['id_objet_source'])>0
-            AND intval($x['args']['id_objet'])>0) {
+        if (intval($x['args']['id_objet_source']) > 0
+                AND intval($x['args']['id_objet']) > 0
+        ) {
 
                 $table = table_objet_sql($x['args']['objet']);
                 $id_objet = $x['args']['id_objet'];
                 include_spip('inc/revisions');
-                if (isset($GLOBALS['premiere_revision']["$table:".$id_objet])){
-                        $GLOBALS['premiere_revision']["$table:".$id_objet] = 0;
-                }
-                // ex : si le champ jointure_mots est versionnable sur les articles
+                if (isset($GLOBALS['premiere_revision']["$table:" . $id_objet])) {
+                        $GLOBALS['premiere_revision']["$table:" . $id_objet] = 0;
+                } // ex : si le champ jointure_mots est versionnable sur les articles
                 elseif ($versionnes = liste_champs_versionnes($table)
-                        AND in_array($j='jointure_'.table_objet($x['args']['objet_source']),$versionnes)){
-                        $champs = array($j=>recuperer_valeur_champ_jointure($x['args']['objet'],$id_objet,$x['args']['objet_source']));
-                        ajouter_version($id_objet,$x['args']['objet'], $champs, '', $GLOBALS['visiteur_session']['id_auteur']);
+                        AND in_array($j = 'jointure_' . table_objet($x['args']['objet_source']), $versionnes)
+                ) {
+                        $champs = array(
+                                $j => recuperer_valeur_champ_jointure($x['args']['objet'], $id_objet, $x['args']['objet_source'])
+                        );
+                        ajouter_version($id_objet, $x['args']['objet'], $champs, '', $GLOBALS['visiteur_session']['id_auteur']);
                 }
 
                 $table = table_objet_sql($x['args']['objet_source']);
                 $id_objet = $x['args']['id_objet_source'];
-                if (isset($GLOBALS['premiere_revision']["$table:".$id_objet])){
-                        $GLOBALS['premiere_revision']["$table:".$id_objet] = 0;
-                }
-                // ex : si le champ jointure_articles est versionnable sur les mots
+                if (isset($GLOBALS['premiere_revision']["$table:" . $id_objet])) {
+                        $GLOBALS['premiere_revision']["$table:" . $id_objet] = 0;
+                } // ex : si le champ jointure_articles est versionnable sur les mots
                 elseif ($versionnes = liste_champs_versionnes($table)
-                        AND in_array($j='jointure_'.table_objet($x['args']['objet']),$versionnes)){
-                        $champs = array($j=>recuperer_valeur_champ_jointure($x['args']['objet_source'],$id_objet,$x['args']['objet']));
-                        ajouter_version($id_objet,$x['args']['objet_source'], $champs, '', $GLOBALS['visiteur_session']['id_auteur']);
+                        AND in_array($j = 'jointure_' . table_objet($x['args']['objet']), $versionnes)
+                ) {
+                        $champs = array(
+                                $j => recuperer_valeur_champ_jointure($x['args']['objet_source'], $id_objet, $x['args']['objet'])
+                        );
+                        ajouter_version($id_objet, $x['args']['objet_source'], $champs, '', $GLOBALS['visiteur_session']['id_auteur']);
                 }
         }
@@ -298 +311 @@
  *     Tableau des tâches et leur périodicité en seconde
  */
-function revisions_taches_generales_cron($taches_generales){
+function revisions_taches_generales_cron($taches_generales) {
         $taches_generales['optimiser_revisions'] = 86400;
+
         return $taches_generales;
 }
+
 ?>

_core_/plugins/revisions/inc/suivi_versions.php

@@ -15 +15 @@
  *
  * @package SPIP\Revisions\Versions
-**/
-if (!defined("_ECRIRE_INC_VERSION")) return;
+ **/
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 include_spip('inc/revisions');
@@ -23 +25 @@
 /**
  * Afficher un diff correspondant à une révision d'un objet
- * 
- * @param int $id_objet    Identifiant de l'objet
- * @param string $objet    Objet
- * @param int $id_version  Identifiant de la version
+ *
+ * @param int $id_objet Identifiant de l'objet
+ * @param string $objet Objet
+ * @param int $id_version Identifiant de la version
  * @param bool $court
  *     - false : affiche le diff complet
@@ -33 +35 @@
  *     Texte HTML du diff.
  */
-function revisions_diff ($id_objet, $objet, $id_version, $court = false){
-        $textes = revision_comparee($id_objet,$objet, $id_version, 'diff');
-        if (!is_array($textes)) return $textes;
+function revisions_diff($id_objet, $objet, $id_version, $court = false) {
+        $textes = revision_comparee($id_objet, $objet, $id_version, 'diff');
+        if (!is_array($textes)) {
+                return $textes;
+        }
         $rev = '';
         $nb = 0;
         foreach ($textes as $var => $t) {
-                if ($n=strlen($t)) {
-                        if ($court)
+                if ($n = strlen($t)) {
+                        if ($court) {
                                 $nb += $n;
-                        else {
+                        } else {
                                 $aff = propre_diff($t);
                                 if ($GLOBALS['les_notes']) {
-                                        $aff .= '<p>'.$GLOBALS['les_notes'].'</p>';
+                                        $aff .= '<p>' . $GLOBALS['les_notes'] . '</p>';
                                         $GLOBALS['les_notes'] = '';
                                 }
@@ -52 +56 @@
                 }
         }
+
         return $court ? _T('taille_octets', array('taille' => $nb)) : $rev;
 }
@@ -62 +67 @@
  * récupérer une version qui contient ce champ. On complète alors la liste
  * des champs avec la version du champ trouvée.
- * 
- * @param string $objet    Objet
- * @param int $id_objet    Identifiant de l'objet
- * @param int $id_version  Identifiant de la version
- * @param string $champ    Le nom du champ à retrouver
- * @param array $champs    Liste des champs déjà connus
+ *
+ * @param string $objet Objet
+ * @param int $id_objet Identifiant de l'objet
+ * @param int $id_version Identifiant de la version
+ * @param string $champ Le nom du champ à retrouver
+ * @param array $champs Liste des champs déjà connus
  * @return void
  */
-function retrouver_champ_version_objet($objet, $id_objet, $id_version, $champ, &$champs){
-        if (isset($champs[$champ]))
+function retrouver_champ_version_objet($objet, $id_objet, $id_version, $champ, &$champs) {
+        if (isset($champs[$champ])) {
                 return;
+        }
 
         // Remonter dans le temps pour trouver le champ en question
@@ -78 +84 @@
         $id_ref = $id_version-1;
         $prev = array();
-        while (!isset($prev[$champ]) AND $id_ref>0) {
-                $prev = recuperer_version($id_objet,$objet, $id_ref--);
+        while (!isset($prev[$champ]) AND $id_ref > 0) {
+                $prev = recuperer_version($id_objet, $objet, $id_ref--);
         }
         if (isset($prev[$champ])) {
@@ -96 +102 @@
  * qui ont été modifiés depuis une version précédente et la version
  * d'id_version, et les retourne.
- * 
+ *
  * La version précédente est par défaut la version juste
  * avant id_version, mais peut être définie via le paramètre id_diff.
@@ -102 +108 @@
  * Le retour est plus ou moins locace en fonction du paramètre format.
  *
- * @param int $id_objet    Identifiant de l'objet
- * @param string $objet    Objet
- * @param int $id_version  Identifiant de la version
+ * @param int $id_objet Identifiant de l'objet
+ * @param string $objet Objet
+ * @param int $id_version Identifiant de la version
  * @param string $format
  *     Type de retour
@@ -116 +122 @@
  *     Couples (champ => texte)
  */
-function revision_comparee($id_objet, $objet, $id_version, $format = 'diff', $id_diff = NULL) {
+function revision_comparee($id_objet, $objet, $id_version, $format = 'diff', $id_diff = null) {
         include_spip('inc/diff');
 
         // chercher le numero de la version precedente
         if (!$id_diff) {
-                $id_diff = sql_getfetsel("id_version", "spip_versions", "id_objet=" . intval($id_objet) . " AND id_version < " . intval($id_version)." AND objet=".sql_quote($objet), "", "id_version DESC", "1");
+                $id_diff = sql_getfetsel("id_version", "spip_versions",
+                        "id_objet=" . intval($id_objet) . " AND id_version < " . intval($id_version) . " AND objet=" . sql_quote($objet),
+                        "", "id_version DESC", "1");
         }
 
@@ -133 +141 @@
                 }
 
-                $old = recuperer_version($id_objet,$objet, $id_diff);
-                $new = recuperer_version($id_objet,$objet, $id_version);
+                $old = recuperer_version($id_objet, $objet, $id_diff);
+                $new = recuperer_version($id_objet, $objet, $id_version);
 
                 $textes = array();
@@ -152 +160 @@
 
                 // memoriser les cas les plus courant
-                $afficher_diff_champ    = charger_fonction('champ','afficher_diff');
-                $afficher_diff_jointure = charger_fonction('jointure','afficher_diff');
+                $afficher_diff_champ = charger_fonction('champ', 'afficher_diff');
+                $afficher_diff_jointure = charger_fonction('jointure', 'afficher_diff');
                 foreach ($champs as $champ) {
                         // Remonter dans le temps pour trouver le champ en question
                         // pour chaque version
-                        retrouver_champ_version_objet($objet,$id_objet,$id_version,$champ,$new);
-                        retrouver_champ_version_objet($objet,$id_objet,$id_diff,$champ,$old);
-
-                        if (!strlen($new[$champ]) && !strlen($old[$champ])) continue;
+                        retrouver_champ_version_objet($objet, $id_objet, $id_version, $champ, $new);
+                        retrouver_champ_version_objet($objet, $id_objet, $id_diff, $champ, $old);
+
+                        if (!strlen($new[$champ]) && !strlen($old[$champ])) {
+                                continue;
+                        }
 
                         // si on n'a que le vieux, ou que le nouveau, on ne
                         // l'affiche qu'en mode "complet"
-                        if ($format == 'complet')
+                        if ($format == 'complet') {
                                 $textes[$champ] = strlen($new[$champ])
                                         ? $new[$champ] : $old[$champ];
+                        }
 
                         // si on a les deux, le diff nous interesse, plus ou moins court
                         if (isset($new[$champ]) AND isset($old[$champ])) {
-                                if (!$afficher_diff = charger_fonction($objet."_".$champ,'afficher_diff',true)
-                                  AND !$afficher_diff = charger_fonction($champ,'afficher_diff',true))
-                                        $afficher_diff = (strncmp($champ,'jointure_',9)==0?$afficher_diff_jointure:$afficher_diff_champ);
-
-                                $textes[$champ] = $afficher_diff($champ,$old[$champ],$new[$champ],$format);
+                                if (!$afficher_diff = charger_fonction($objet . "_" . $champ, 'afficher_diff', true)
+                                        AND !$afficher_diff = charger_fonction($champ, 'afficher_diff', true)
+                                ) {
+                                        $afficher_diff = (strncmp($champ, 'jointure_', 9) == 0 ? $afficher_diff_jointure : $afficher_diff_champ);
+                                }
+
+                                $textes[$champ] = $afficher_diff($champ, $old[$champ], $new[$champ], $format);
                         }
                 }
@@ -181 +194 @@
         // que donner par defaut ? (par exemple si id_version=1)
         if (!$textes) {
-                $textes = recuperer_version($id_objet,$objet, $id_version);
+                $textes = recuperer_version($id_objet, $objet, $id_version);
         }
 

_core_/plugins/revisions/prive/objets/contenu/revision_fonctions.php

@@ -1 +1 @@
 <?php
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 /**
  * Trouver le label d'un champ de révision
- * 
+ *
  * Quelques champs ont un label dans dans les chaînes de langue de SPIP
  * Pour un champ particulier d'un objet particulier, le pipeline revisions_chercher_label
  * peut être utilisé
- * 
+ *
  * @param string $champ
- *              Le nom du champ révisionné
+ *    Le nom du champ révisionné
  * @param string $objet
- *              Le type d'objet révisionné
+ *    Le type d'objet révisionné
  * @return string $label
- *              Le label du champ
+ *    Le label du champ
  */
-function label_champ($champ, $objet = false){
+function label_champ($champ, $objet = false) {
         $label = "";
         // si jointure: renvoyer le nom des objets joints
-        if (strncmp($champ,'jointure_',9)==0)
-                return _T(objet_info(objet_type(substr($champ,9)),'texte_objets'));
-        
-        switch ($champ){
+        if (strncmp($champ, 'jointure_', 9) == 0) {
+                return _T(objet_info(objet_type(substr($champ, 9)), 'texte_objets'));
+        }
+
+        switch ($champ) {
                 case 'surtitre':
                         $label = "texte_sur_titre";
@@ -39 +42 @@
                         $champ = "chapeau";
                 default:
-                        $label = pipeline('revisions_chercher_label',array('args'=>array('champ'=>$champ,'objet'=>$objet),'data' => 'info_'.$champ));
+                        $label = pipeline('revisions_chercher_label',
+                                array('args' => array('champ' => $champ, 'objet' => $objet), 'data' => 'info_' . $champ));
                         break;
         }
-        return $label?_T($label):"";
+
+        return $label ? _T($label) : "";
 }
 

_core_/plugins/revisions/prive/objets/liste/versions_fonctions.php

@@ -1 +1 @@
 <?php
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 include_spip('inc/puce_statut');

_core_/plugins/revisions/prive/rss/revisions_fonctions.php

@@ -11 +11 @@
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 include_spip('inc/suivi_versions');

_core_/plugins/revisions/prive/squelettes/contenu/revision_fonctions.php

@@ -1 +1 @@
 <?php
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 include_spip('inc/revisions');

_core_/plugins/revisions/revisions_administrations.php

@@ -15 +15 @@
  *
  * @package SPIP\Revisions\Installation
-**/
-
-if (!defined('_ECRIRE_INC_VERSION')) return;
+ **/
+
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 /**
@@ -25 +27 @@
  * @param string $version_cible
  */
-function revisions_upgrade($nom_meta_base_version, $version_cible){
+function revisions_upgrade($nom_meta_base_version, $version_cible) {
         // cas particulier :
         // si plugin pas installe mais que la table existe
         // considerer que c'est un upgrade depuis v 1.0.0
         // pour gerer l'historique des installations SPIP <=2.1
-        if (!isset($GLOBALS['meta'][$nom_meta_base_version])){
-                $trouver_table = charger_fonction('trouver_table','base');
+        if (!isset($GLOBALS['meta'][$nom_meta_base_version])) {
+                $trouver_table = charger_fonction('trouver_table', 'base');
                 if ($desc = $trouver_table('spip_versions')
-                  AND isset($desc['exist']) AND $desc['exist']
-                  AND isset($desc['field']['id_article'])){
-                        ecrire_meta($nom_meta_base_version,'1.0.0');
+                        AND isset($desc['exist']) AND $desc['exist']
+                        AND isset($desc['field']['id_article'])
+                ) {
+                        ecrire_meta($nom_meta_base_version, '1.0.0');
                 }
                 // si pas de table en base, on fera une simple creation de base
@@ -42 +45 @@
         $maj = array();
         $maj['create'] = array(
-                array('maj_tables',array('spip_versions','spip_versions_fragments')),
+                array('maj_tables', array('spip_versions', 'spip_versions_fragments')),
                 array('revisions_upate_meta'),
         );
@@ -49 +52 @@
                 // Ajout du champs objet et modification du champs id_article en id_objet
                 // sur les 2 tables spip_versions et spip_versions_fragments
-                array('sql_alter',"TABLE spip_versions CHANGE id_article id_objet bigint(21) DEFAULT 0 NOT NULL"),
-                array('sql_alter',"TABLE spip_versions ADD objet VARCHAR (25) DEFAULT '' NOT NULL AFTER id_objet"),
+                array('sql_alter', "TABLE spip_versions CHANGE id_article id_objet bigint(21) DEFAULT 0 NOT NULL"),
+                array('sql_alter', "TABLE spip_versions ADD objet VARCHAR (25) DEFAULT '' NOT NULL AFTER id_objet"),
                 // Les id_objet restent les id_articles puisque les révisions n'étaient possibles que sur les articles
-                array('sql_updateq',"spip_versions",array('objet'=>'article'),"objet=''"),
+                array('sql_updateq', "spip_versions", array('objet' => 'article'), "objet=''"),
                 // Changement des clefs primaires également
-                array('sql_alter',"TABLE spip_versions DROP PRIMARY KEY"),
-                array('sql_alter',"TABLE spip_versions ADD PRIMARY KEY (id_version, id_objet, objet)"),
-
-                array('sql_alter',"TABLE spip_versions_fragments CHANGE id_article id_objet bigint(21) DEFAULT 0 NOT NULL"),
-                array('sql_alter',"TABLE spip_versions_fragments ADD objet VARCHAR (25) DEFAULT '' NOT NULL AFTER id_objet"),
+                array('sql_alter', "TABLE spip_versions DROP PRIMARY KEY"),
+                array('sql_alter', "TABLE spip_versions ADD PRIMARY KEY (id_version, id_objet, objet)"),
+
+                array('sql_alter', "TABLE spip_versions_fragments CHANGE id_article id_objet bigint(21) DEFAULT 0 NOT NULL"),
+                array('sql_alter', "TABLE spip_versions_fragments ADD objet VARCHAR (25) DEFAULT '' NOT NULL AFTER id_objet"),
                 // Les id_objet restent les id_articles puisque les révisions n'étaient possibles que sur les articles
-                array('sql_updateq',"spip_versions_fragments",array('objet'=>'article'),"objet=''"),
+                array('sql_updateq', "spip_versions_fragments", array('objet' => 'article'), "objet=''"),
                 // Changement des clefs primaires également
-                array('sql_alter',"TABLE spip_versions_fragments DROP PRIMARY KEY"),
-                array('sql_alter',"TABLE spip_versions_fragments ADD PRIMARY KEY (id_objet, objet, id_fragment, version_min)"),
+                array('sql_alter', "TABLE spip_versions_fragments DROP PRIMARY KEY"),
+                array('sql_alter', "TABLE spip_versions_fragments ADD PRIMARY KEY (id_objet, objet, id_fragment, version_min)"),
                 array('revisions_upate_meta'),
         );
         $maj['1.1.2'] = array(
                 array('revisions_upate_meta'),
-                array('sql_updateq',"spip_versions",array('objet'=>'article'),"objet=''"),
-                array('sql_updateq',"spip_versions_fragments",array('objet'=>'article'),"objet=''"),
+                array('sql_updateq', "spip_versions", array('objet' => 'article'), "objet=''"),
+                array('sql_updateq', "spip_versions_fragments", array('objet' => 'article'), "objet=''"),
         );
         $maj['1.1.3'] = array(
-                array('sql_alter',"TABLE spip_versions DROP KEY id_objet"),
-                array('sql_alter',"TABLE spip_versions ADD INDEX id_version (id_version)"),
-                array('sql_alter',"TABLE spip_versions ADD INDEX id_objet (id_objet)"),
-                array('sql_alter',"TABLE spip_versions ADD INDEX objet (objet)")
+                array('sql_alter', "TABLE spip_versions DROP KEY id_objet"),
+                array('sql_alter', "TABLE spip_versions ADD INDEX id_version (id_version)"),
+                array('sql_alter', "TABLE spip_versions ADD INDEX id_objet (id_objet)"),
+                array('sql_alter', "TABLE spip_versions ADD INDEX objet (objet)")
         );
         $maj['1.1.4'] = array(
-                array('sql_alter',"TABLE spip_versions CHANGE permanent permanent char(3) DEFAULT '' NOT NULL"),
-                array('sql_alter',"TABLE spip_versions CHANGE champs champs text DEFAULT '' NOT NULL"),
+                array('sql_alter', "TABLE spip_versions CHANGE permanent permanent char(3) DEFAULT '' NOT NULL"),
+                array('sql_alter', "TABLE spip_versions CHANGE champs champs text DEFAULT '' NOT NULL"),
         );
         $maj['1.2.0'] = array(
@@ -91 +94 @@
 }
 
-function revisions_uncompress_fragments(){
-
-        $res = sql_select("*","spip_versions_fragments","compress=".intval(1));
-        while($row = sql_fetch($res)){
+function revisions_uncompress_fragments() {
+
+        $res = sql_select("*", "spip_versions_fragments", "compress=" . intval(1));
+        while ($row = sql_fetch($res)) {
                 $fragment = @gzuncompress($row['fragment']);
 
                 // si la decompression echoue, on met en base le flag 'corrompu-gz'
                 // et au dump le framgment compresse dans un fichier
-                if (strlen($row['fragment']) AND $fragment===false){
-                        $dir_tmp = sous_repertoire(_DIR_TMP,"versions_fragments_corrompus");
-                        $f = $row['id_fragment']."-".$row['objet']."-".$row['id_objet'];
-                        spip_log("Fragment gz corrompu $f","maj"._LOG_ERREUR);
-                        $f = $f."-gz.txt";
-                        ecrire_fichier($dir_tmp . $f,$row['fragment']);
+                if (strlen($row['fragment']) AND $fragment === false) {
+                        $dir_tmp = sous_repertoire(_DIR_TMP, "versions_fragments_corrompus");
+                        $f = $row['id_fragment'] . "-" . $row['objet'] . "-" . $row['id_objet'];
+                        spip_log("Fragment gz corrompu $f", "maj" . _LOG_ERREUR);
+                        $f = $f . "-gz.txt";
+                        ecrire_fichier($dir_tmp . $f, $row['fragment']);
                         $fragment = "corrompu-gz";
                 }
@@ -113 +116 @@
                 );
 
-                sql_updateq("spip_versions_fragments",$set,"id_fragment=".intval($row['id_fragment'])." AND id_objet=".intval($row['id_objet'])." AND objet=".sql_quote($row['objet'])." AND version_min=".intval($row['version_min']));
-                if (time()>_TIME_OUT) return;
-        }
-
-        sql_updateq("spip_versions_fragments",array('compress'=>-1));
-
-}
-
-function revisions_repair_unserialized_fragments(){
-        $res = sql_select("*","spip_versions_fragments","compress=".intval(-1));
+                sql_updateq("spip_versions_fragments", $set,
+                        "id_fragment=" . intval($row['id_fragment']) . " AND id_objet=" . intval($row['id_objet']) . " AND objet=" . sql_quote($row['objet']) . " AND version_min=" . intval($row['version_min']));
+                if (time() > _TIME_OUT) {
+                        return;
+                }
+        }
+
+        sql_updateq("spip_versions_fragments", array('compress' => -1));
+
+}
+
+function revisions_repair_unserialized_fragments() {
+        $res = sql_select("*", "spip_versions_fragments", "compress=" . intval(-1));
         $n = sql_count($res);
-        spip_log("$n fragments a verifier","maj");
-        while($row = sql_fetch($res)){
+        spip_log("$n fragments a verifier", "maj");
+        while ($row = sql_fetch($res)) {
                 $fragment = $row['fragment'];
                 $set = array(
@@ -132 +138 @@
 
                 // verifier que le fragment est bien serializable
-                if(unserialize($fragment)===false AND strncmp($fragment,"corrompu",8)!==0){
-                        $dir_tmp = sous_repertoire(_DIR_TMP,"versions_fragments_corrompus");
+                if (unserialize($fragment) === false AND strncmp($fragment, "corrompu", 8) !== 0) {
+                        $dir_tmp = sous_repertoire(_DIR_TMP, "versions_fragments_corrompus");
                         $set['fragment'] = revisions_repair_serialise($fragment);
-                        if (strncmp($set['fragment'],"corrompu",8)==0){
+                        if (strncmp($set['fragment'], "corrompu", 8) == 0) {
                                 $f = $row['id_fragment'] . "-" . $row['objet'] . "-" . $row['id_objet'];
                                 spip_log("Fragment serialize corrompu $f", "maj" . _LOG_ERREUR);
@@ -142 +148 @@
                         }
                 }
-                sql_updateq("spip_versions_fragments",$set,$w = "id_fragment=".intval($row['id_fragment'])." AND id_objet=".intval($row['id_objet'])." AND objet=".sql_quote($row['objet'])." AND version_min=".intval($row['version_min']));
+                sql_updateq("spip_versions_fragments", $set,
+                        $w = "id_fragment=" . intval($row['id_fragment']) . " AND id_objet=" . intval($row['id_objet']) . " AND objet=" . sql_quote($row['objet']) . " AND version_min=" . intval($row['version_min']));
                 #spip_log($w,"maj");
 
-                if (time()>_TIME_OUT) return;
-        }
-}
-
-function revisions_repair_serialise($serialize){
-        if (unserialize($serialize))
+                if (time() > _TIME_OUT) {
+                        return;
+                }
+        }
+}
+
+function revisions_repair_serialise($serialize) {
+        if (unserialize($serialize)) {
                 return $serialize;
+        }
 
         // verifier les strings
-        preg_match_all(",s:(\d+):\"(.*)\";(?=}|\w:\d+),Uims",$serialize,$matches,PREG_SET_ORDER);
+        preg_match_all(",s:(\d+):\"(.*)\";(?=}|\w:\d+),Uims", $serialize, $matches, PREG_SET_ORDER);
         $serialize_repair = $serialize;
-        foreach($matches as $match){
+        foreach ($matches as $match) {
                 $s = $match[2];
                 $l = $match[1];
-                if (strlen($s)!==$l){
-                        if (strlen($s)<$l){
-                                $s = str_replace("\r\n","\n",$s);
-                                $s = str_replace("\r","\n",$s);
-                                $s = str_replace("\n","\r\n",$s);
-                        }
-                        if (strlen($s)>$l){
-                                $s = str_replace("\r\n","\n",$s);
-                                $s = str_replace("\r","\n",$s);
-                        }
-                        if (strlen($s)<$l){
-                                $s .= str_pad("",$l-strlen($s)," ");
-                        }
-                        if (strlen($s)==$l){
-                                $s = str_replace($match[2],$s,$match[0]);
-                                $serialize_repair = str_replace($match[0],$s,$serialize_repair);
-                        }
-                }
-        }
-        if (unserialize($serialize_repair))
+                if (strlen($s) !== $l) {
+                        if (strlen($s) < $l) {
+                                $s = str_replace("\r\n", "\n", $s);
+                                $s = str_replace("\r", "\n", $s);
+                                $s = str_replace("\n", "\r\n", $s);
+                        }
+                        if (strlen($s) > $l) {
+                                $s = str_replace("\r\n", "\n", $s);
+                                $s = str_replace("\r", "\n", $s);
+                        }
+                        if (strlen($s) < $l) {
+                                $s .= str_pad("", $l-strlen($s), " ");
+                        }
+                        if (strlen($s) == $l) {
+                                $s = str_replace($match[2], $s, $match[0]);
+                                $serialize_repair = str_replace($match[0], $s, $serialize_repair);
+                        }
+                }
+        }
+        if (unserialize($serialize_repair)) {
                 return $serialize_repair;
+        }
 
         // on essaye brutalement
         $serialize_repair = $serialize;
-        $serialize_repair = str_replace("\r\n","\n",$serialize_repair);
-        $serialize_repair = str_replace("\r","\n",$serialize_repair);
-        if (unserialize($serialize_repair))
+        $serialize_repair = str_replace("\r\n", "\n", $serialize_repair);
+        $serialize_repair = str_replace("\r", "\n", $serialize_repair);
+        if (unserialize($serialize_repair)) {
                 return $serialize_repair;
-        $serialize_repair = str_replace("\n","\r\n",$serialize_repair);
-        if (unserialize($serialize_repair))
+        }
+        $serialize_repair = str_replace("\n", "\r\n", $serialize_repair);
+        if (unserialize($serialize_repair)) {
                 return $serialize_repair;
+        }
 
         #echo "Impossible de reparer la chaine :";
@@ -195 +208 @@
         #var_dump($matches);
         #die("corrompu-serialize");
-        return  "corrompu-serialize";
+        return "corrompu-serialize";
 }
 
@@ -213 +226 @@
 /**
  * Mettre a jour la meta des versions
+ *
  * @return void
  */
-function revisions_upate_meta(){
+function revisions_upate_meta() {
         // Si dans une installation antérieure ou un upgrade, les articles étaient versionnés
         // On crée la meta correspondante
         // mettre les metas par defaut
-        $config = charger_fonction('config','inc');
+        $config = charger_fonction('config', 'inc');
         $config();
-        if (isset($GLOBALS['meta']['articles_versions']) and $GLOBALS['meta']['articles_versions'] == 'oui'){
-                ecrire_meta('objets_versions',serialize(array('articles')));
+        if (isset($GLOBALS['meta']['articles_versions']) and $GLOBALS['meta']['articles_versions'] == 'oui') {
+                ecrire_meta('objets_versions', serialize(array('articles')));
         }
         effacer_meta('articles_versions');
-        if (!$versions = unserialize($GLOBALS['meta']['objets_versions']))
+        if (!$versions = unserialize($GLOBALS['meta']['objets_versions'])) {
                 $versions = array();
-        $versions = array_map('table_objet_sql',$versions);
-        ecrire_meta('objets_versions',serialize($versions));
+        }
+        $versions = array_map('table_objet_sql', $versions);
+        ecrire_meta('objets_versions', serialize($versions));
 }
 

_core_/plugins/revisions/revisions_ieconfig.php

@@ -2 +2 @@
 
 /**
- * Enregistrer les config avec le plugin IEConfig 
+ * Enregistrer les config avec le plugin IEConfig
  *
  * @package SPIP\Revisions\Pipelines
-**/
-if (!defined("_ECRIRE_INC_VERSION")) return;
+ **/
+if (!defined("_ECRIRE_INC_VERSION")) {
+        return;
+}
 
 /**
@@ -12 +14 @@
  *
  * @pipeline ieconfig_metas
- * @param array $table  Description des configurations
+ * @param array $table Description des configurations
  * @return array        Description des configurations
-**/
-function revisions_ieconfig_metas($table){
+ **/
+function revisions_ieconfig_metas($table) {
         $table['revisions']['titre'] = _T('revisions:titre_revisions');
         $table['revisions']['icone'] = 'revision-16.png';
         $table['revisions']['metas_serialize'] = 'objets_versions';
-        
+
         return $table;
 }