Changeset 48831 in spip-zone

Timestamp:

Jun 16, 2011, 10:21:03 AM (10 years ago)

Author:

cedric@…

Message:

(oups, il manquait un morceau de commit dans r48830)
Ne pas compiler une meme less deux fois dans le hit (cas si echec au premier coup)
Generer une erreur de compilation SPIP en cas d'erreur de compilation LESS
Definir le importDir de la LESS compilee pour résoudre les @import less
Corriger un bug sur l'import qui provoque une erreur fatale si le fichier importé commence par des commentaires (on fix ici sans reporter au dev de lessPHP car le modele interne de compilation LESS a completement changé depuis cette release)

Location:

_plugins_/less-css/trunk

Files:

• lesscss_fonctions.php (modified) (5 diffs)
• plugin.xml (modified) (1 diff)

_plugins_/less-css/trunk/lesscss_fonctions.php

@@ -6 +6 @@
  * @return string
  */
-function less_compile($style){
+function less_compile($style, $import_dir="", $contexte = array()){
         require_once 'lessphp/lessc.inc.php';
 
         // le compilateur lessc compile le contenu
         $less = new lessc();
+        if ($import_dir){
+                $less->importDir = rtrim($import_dir,"/")."/";
+        }
         try {
                 $out = $less->parse($style);
@@ -18 +21 @@
         catch (exception $ex) {
                 spip_log('lessc fatal error:'.$ex->getMessage(),'less'._LOG_ERREUR);
+                erreur_squelette(
+                        "LESS : Echec compilation"
+                        . (isset($contexte['file'])?" fichier ".$contexte['file']:"")
+                  . "<br />".$ex->getMessage()
+                );
                 return '';
         }
@@ -41 +49 @@
           AND preg_match(',\.(less|css)$,i', $source, $r)
           AND file_exists($source)) {
+                static $done = array();
+                // ne pas essayer de compiler deux fois le meme fichier dans le meme hit
+                // si on a echoue une fois, on echouera pareil
+                if (isset($done[$source])) return $done[$source];
 
                 $f = basename($source,$r[0]);
@@ -58 +70 @@
 
                 # compiler le LESS
-                $contenu = less_compile($contenu);
+                $contenu = less_compile($contenu, dirname($source), array('file'=>$source));
                 // si erreur de compilation on renvoit la source, et il y a deja eu un log
                 if (!$contenu)
-                        return $source;
+                        return $done[$source] = $source;
                 # passer la css en url absolue (on ne peut pas le faire avant, car c'est du LESS, pas des CSS)
                 $contenu = urls_absolues_css($contenu, $source);
@@ -67 +79 @@
                 // ecrire le fichier destination, en cas d'echec renvoyer la source
                 if (ecrire_fichier($f, $contenu, true))
-                        return $f;
+                        return $done[$source] = $f;
                 else
-                        return $source;
+                        return $done[$source] = $source;
         }
         $source = less_compile($source);

_plugins_/less-css/trunk/plugin.xml

@@ -3 +3 @@
         <auteur>Jean-Baptiste Bourgoin</auteur>
         <licence>MIT</licence>
-        <version>0.3.0</version>
+        <version>0.3.1</version>
         <etat>test</etat>
         <icon>img/lesscss-32.png</icon>