Changeset 117936 in spip-zone

Timestamp:

Sep 24, 2019, 10:15:02 AM (5 months ago)

Author:

root

Message:

Refactoring de _T_ou_typo : plus clair, evider les preg_match et include qui ne sont pas necessaires

Location:

_plugins_/saisies/trunk

Files:

• paquet.xml (modified) (1 diff)
• saisies_options.php (modified) (2 diffs)

_plugins_/saisies/trunk/paquet.xml

@@ -2 +2 @@
         prefix="saisies"
         categorie="outil"
-        version="3.28.5"
+        version="3.28.6"
         etat="stable"
         compatibilite="[3.0.0;3.2.*]"

_plugins_/saisies/trunk/saisies_options.php

@@ -11 +11 @@
         return;
 }
-include_spip('plugins/installer');
+
+
 if (!function_exists('_T_ou_typo')) {
         /**
@@ -32 +33 @@
 
                 // Si la valeur est bien une chaine (et pas non plus un entier déguisé)
-                if (is_string($valeur) and !intval($valeur)) {
-                        // Si on a dépassé 3.2, on peut uniquement utilser typo() car ça extrait les <:chaine:>
+                if (is_string($valeur) and !is_numeric($valeur)) {
+                        // Si on est en >=3.2, on peut extraire les <:chaine:>
                         $version = explode('.',$GLOBALS['spip_version_branche']);
-                        if ($version[0] > 3 or ($version[1] >= 2 and $version[0] == 3)) {
+                        $extraction_chaines = (($version[0] > 3 or $version[1] >= 2) ? true : false);
+                        // Si la chaine est du type <:truc:> on passe à _T()
+                        if (strpos($valeur, '<:') !== false
+                          and preg_match('/^\<:([^>]*?):\>$/', $valeur, $match)) {
+                                $valeur = _T($match[1]);
+                        } else {
+                                // Sinon on la passe a typo() si c'est pertinent
                                 if (
-                                        $mode_typo == 'toujours'
-                                        or (
-                                                $mode_typo == 'multi'
-                                                and include_spip('inc/filtres')
-                                                and (preg_match(_EXTRAIRE_IDIOME, $valeur) or strpos($valeur, '<multi>') !== false)
-                                        )
+                                        $mode_typo === 'toujours'
+                                        or ($mode_typo === 'multi' and strpos($valeur, '<multi>') !== false)
+                                        or ($extraction_chaines
+                                          and $mode_typo === 'multi'
+                                          and strpos($valeur, '<:') !== false
+                                          and include_spip('inc/filtres')
+                                          and preg_match(_EXTRAIRE_IDIOME, $valeur))
                                 ) {
                                         include_spip('inc/texte');
                                         $valeur = typo($valeur);
-                                } elseif ($mode_typo == 'jamais' and preg_match('/^\<:(.*?):\>$/', $valeur, $match)) {
-                                        $valeur = _T($match[1]);
-                                }
-                        }
-                        // Si on est avant 3.2, on fait comme avant
-                        else {
-                                // Si la chaine est du type <:truc:> on passe à _T()
-                                if (preg_match('/^\<:(.*?):\>$/', $valeur, $match)) {
-                                        $valeur = _T($match[1]);
-                                } else {
-                                        // Sinon on la passe a typo()
-                                        if ($mode_typo == 'toujours' or ($mode_typo == 'multi' and strpos($valeur, '<multi>') !== false)) {
-                                                include_spip('inc/texte');
-                                                $valeur = typo($valeur);
-                                        }
                                 }
                         }