Changeset 77175 in spip-zone

Timestamp:

Oct 12, 2013, 8:52:17 AM (8 years ago)

Author:

tcharlss@…

Message:

Vérifications supplémentaires sur la saisie du champ "page" + messages d'erreur adéquats :

• nombre de caractères
• caractères alphanumériques ou "_"
• doublons

Location:

_plugins_/pages/trunk

Files:

• lang/pages_fr.php (modified) (2 diffs)
• pages_pipelines.php (modified) (1 diff)

_plugins_/pages/trunk/lang/pages_fr.php

@@ -7 +7 @@
 
         // A
-        'aucune_page' => 'Il n’y a aucune page pour l’instant.',
+        'aucune_page' => 'Il n\'y a aucune page pour l\'instant.',
 
         // C
@@ -14 +14 @@
         'creer_page' => 'Créer une nouvelle page',
 
+        // E
+        'erreur_champ_page_doublon' => 'Cet identifiant existe déjà',
+        'erreur_champ_page_format' => 'Charactères alphanumériques en minuscules ou "_" uniquement',
+        'erreur_champ_page_taille' => '255 charactères maximum',
+
+        // L
+        'label_champ_page' => 'Page :',
+
         // M
-        'modifier_page' => 'Modifier la page :',
+        'modifier_page' => 'Modifier la page :',
 
         // P

_plugins_/pages/trunk/pages_pipelines.php

@@ -71 +71 @@
 
 
-// Vérifier que la page n'est pas vide
+/**
+ * Vérifications de l'identifiant d'une page
+ * 
+ * @param array $flux
+ *              Le contexte du pipeline
+ * @return array $flux
+ *              Le contexte du pipeline modifié
+ */
 function pages_formulaire_verifier($flux){
-        // Si on est dans l'édition d'un article
-        if (is_array($flux) and $flux['args']['form'] == 'editer_article'){
-                // Si on est dans un article de type page mais que le champ "page" est vide
-                if (_request('modele') == 'page' and !_request('champ_page'))
+        // Si on est dans l'édition d'un article de type page
+        if (
+                is_array($flux)
+                and $flux['args']['form'] == 'editer_article'
+                and _request('modele') == 'page'
+        ){
+                $erreur = '';
+                $page = _request('champ_page');
+                // champ "page" vide
+                if ( !$page )
                         $flux['data']['champ_page'] .= _T('info_obligatoire');
-        }
-        return $flux;
-
-}
+                // nombre de charactères : 255 max
+                elseif (strlen($page) > 255)
+                         $erreur = _T('pages:erreur_champ_page_taille');
+                // format : charactères alphanumériques en minuscules ou "_"
+                elseif (!preg_match('/^[a-z0-9_]+$/', $page))
+                         $erreur = _T('pages:erreur_champ_page_format');
+                // doublon
+                elseif (sql_countsel(table_objet_sql('article'), "page=".sql_quote($page) . " AND id_article!=".intval($id_page)))
+                        $erreur = _T('pages:erreur_champ_page_doublon');
+                if ($erreur) $flux['data']['champ_page'] .= $erreur;
+        }
+        return $flux;
+
+}
+
 
 /**