Changeset 60822 in spip-zone

Timestamp:

Apr 30, 2012, 1:57:09 PM (8 years ago)

Author:

marcimat@…

Message:

Aérer un peu les diffs Unix affichés dans l'interface en ne gardant que le chemin relatif et en espaçant.

Location:

_plugins_/fabrique

Files:

• changelog.txt (modified) (1 diff)
• formulaires/fabriquer_plugin_actions.php (modified) (1 diff)
• inc/fdiff.php (modified) (6 diffs)

_plugins_/fabrique/changelog.txt

@@ -10 +10 @@
 Interface :
 * Gestion des diff sous Windows (Julien Lanfray)
-
+* Affichage des diff unix plus aérés et simples
 
 

_plugins_/fabrique/formulaires/fabriquer_plugin_actions.php

@@ -305 +305 @@
                 $fdiff->add_ignorer(array("fabrique_diff.diff", "fabrique_".$prefixe.".php"));
                 $tab = $fdiff->get_diff();
-                $diff = $tab["diff"];
+                $diff         = $tab["diff"];
+                $diff_humain  = $tab["affiche"];
                 $suppressions = $tab["suppressions"];
 
                 ecrire_fichier($nouveau . 'fabrique_diff.diff', $diff);
                 // coloration si le plugin 'coloration_code' est la
-                $diff = propre("<cadre class='diff'>\n$diff\n</cadre>");
+                $diff = propre("<cadre class='diff'>\n$diff_humain\n</cadre>");
                 set_request('message_diff', $diff);
                 if ($suppressions) {

_plugins_/fabrique/inc/fdiff.php

@@ -219 +219 @@
          * base sur la commande "fc" de Windows
          * 
-         * @return array("diff"=>Texte, "suppressions"=>array(noms de fichier))
+         * @return array("diff"=>Texte, "affiche"=>Texte, "suppressions"=>array(noms de fichier))
          * 
         **/
@@ -293 +293 @@
                 $tab = array(
                         "diff" => $diff,
+                        "affiche" => $diff, // diff plus humainement lisible
                         "suppressions" => $que_dossier1
                 );
@@ -304 +305 @@
          * base sur la commande "diff" des systemes Unix
          * 
-         * @return array("diff"=>Texte, "suppressions"=>array(noms de fichier))
+         * @return array("diff"=>Texte, "affiche"=>Texte, "suppressions"=>array(noms de fichier))
          * 
         **/
@@ -321 +322 @@
                 // supprimes de la nouvelle pour avertir
                 $suppressions = array();
-                foreach($diff as $l) {
+                // on en profite pour raccourcir la ligne diff
+                // pour un retour plus humainement lisible
+                $affiche = $diff;
+                foreach($diff as $k => $l) {
+                        // trouver les suppressions
                         // Only in ../plugins/fabrique_auto/.backup/prefixe/dir: fichier.php
                         if ($l[0] == 'O' AND substr($l, 0, 7) == 'Only in') {
@@ -327 +332 @@
                                         $suppressions[] = str_replace(': ', '/', trim(substr($l, 8 + strlen($this->dossier1))));
                                 }
+                                $affiche[$k] = "\n\n$l";
+                        }
+                        // rendre le diff plus lisible
+                        if ($l[0] == 'd' AND substr($l, 0, 4) == 'diff') {
+                                // ne garder que le chemin relatif du fichier
+                                $fichier = explode(' ', $l);
+                                $fichier = array_pop($fichier);
+                                $fichier = substr($fichier, strlen($this->dossier2));
+                                $affiche[$k] = "\n\n$fichier";
                         }
                 }
                 $diff = implode("\n", $diff);
+                $affiche = implode("\n", $affiche);
 
                 $tab = array(
                         "diff" => $diff,
+                        "affiche" => $affiche, // diff plus humainement lisible
                         "suppressions" => $suppressions
                 );
@@ -346 +362 @@
          * $fdiff->get_diff();
          * 
-         * @return array("diff"=>Texte, "suppressions"=>array(noms de fichier))
+         * @return array(
+         *              "diff"=>Texte,
+         *              "affiche"=>Texte, // diff plus lisible pour affichage
+         *              "suppressions"=>array(noms de fichier))
          * 
         **/