Changeset 91599 in spip-zone for _plugins_/plan

Timestamp:

Sep 3, 2015, 1:08:07 AM (6 years ago)

Author:

marcimat@…

Message:

Merge de r91595, r91596 et r91598

Location:

_plugins_/plan/branches/v1

Files:

• . (modified) (1 prop)
• action/deplacer_objets.php (modified) (6 diffs)
• css/plan_prive.css.html (modified) (2 diffs)
• lang/plan_fr.php (modified) (1 diff)
• paquet.xml (modified) (1 diff)
• prive/javascript/spiptree.js (modified) (5 diffs)
• prive/squelettes/contenu/plan.html (modified) (2 diffs)
• prive/squelettes/inclure/plan.html (copied) (copied from _plugins_/plan/trunk/prive/squelettes/inclure/plan.html) (2 diffs)

_plugins_/plan/branches/v1/action/deplacer_objets.php

@@ -18 +18 @@
         include_spip('inc/autoriser');
         if (!autoriser('ecrire')) {
-                return plan_json_erreur("Authorization failed");
+                return plan_json_erreur(_T("plan:erreur_autorisation_insuffisante") . " " . _T("plan:erreur_deplacement_impossible"));
         }
 
@@ -32 +32 @@
 
         if (!is_array($ids) or !$objet) {
-                return plan_json_erreur("Aucun identifiant");
+                return plan_json_erreur(_T("plan:erreur_aucun_identifiant") . " " . _T("plan:erreur_deplacement_impossible"));
         }
         if ($id_rubrique_old == $id_rubrique_new) {
-                return plan_json_erreur("Rubriques parentes incorrectes");
+                return plan_json_erreur(_T("plan:erreur_rubriques_parentes_incorrectes") . " "  . _T("plan:erreur_deplacement_impossible"));
         }
         if ($objet != 'rubrique' and !$id_rubrique_new) {
-                return plan_json_erreur("Rubriques parentes incorrectes");
+                return plan_json_erreur(_T("plan:erreur_rubriques_parentes_incorrectes") . " " . _T("plan:erreur_deplacement_impossible"));
         }
 
@@ -55 +55 @@
         include_spip('action/editer_objet');
 
-        $erreurs = array();
+        $errors = $success = array();
         $modifs = array('id_parent' => $id_rubrique_new);
 
@@ -61 +61 @@
                 if (autoriser('modifier', $objet, $id)) {
                         if ($err = objet_modifier($objet, $id, $modifs)) {
-                                $erreurs[] = $err;
+                                $errors["$objet-$id"] = $err;
+                        } else {
+                                $success["$objet-$id"] = true;
+                        }
+                } else {
+                        $errors["$objet-$id"] = _T("plan:erreur_autorisation_insuffisante") . " " . _T("plan:erreur_deplacement_impossible");
+                }
+        }
+
+        // dans certains cas… on ne reçoit pas d'erreur… et pourtant !
+        if (!$errors) {
+                // on verifie qu'il n'y a plus d'objets à l'ancien emplacement
+                $ids = sql_allfetsel($_id_table, $table, array(sql_in($_id_table, $ids), $champ . '=' . sql_quote($id_rubrique_old)));
+                $ids = array_map('array_shift', $ids);
+                if ($ids) {
+                        foreach ($ids as $id) {
+                                $errors["$objet-$id"] = _T("plan:erreur_deplacement");
+                                unset($success["$objet-$id"]);
                         }
                 }
@@ -67 +84 @@
 
         return plan_json_envoi(array(
-                'success' => true,
-                'messages' => $erreurs
+                'done' => true,
+                'success' => $success,
+                'errors'  => $errors,
         ));
 }
@@ -78 +96 @@
 
 function plan_json_erreur($msg) {
-        return plan_json_envoi(array('erreur' => $msg));
+        return plan_json_envoi(array(
+                'done' => false,
+                'success' => array(),
+                'errors'  => array($msg)
+        ));
 }

_plugins_/plan/branches/v1/css/plan_prive.css.html

@@ -3 +3 @@
 #HTTP_HEADER{Vary: Accept-Encoding}
 
+.plan #contenu p.removable { cursor:pointer; }
+.plan #contenu p.removable:hover { opacity:.6; }
 #mytree_actions { margin-bottom:1em; margin-top:1em; }
 #mytree_actions input,
@@ -43 +45 @@
         background:url([(#VAL{rubrique}|objet_icone{16}|extraire_attribut{src})]) center left no-repeat;
 }
+.plan .jstree .puce_objet { margin-right:2px; }
+.plan .jstree.drag .item .jstree-icon {
+        cursor: move;
+}
 <BOUCLE_icone_objets(DATA){source table, #REM|plan_lister_objets_rubrique}>
 .plan .jstree-icon.[(#VALEUR|objet_type)] {

_plugins_/plan/branches/v1/lang/plan_fr.php

@@ -6 +6 @@
 $GLOBALS[$GLOBALS['idx_lang']] = array(
 
-        // T
-        'tous_les_statuts' => 'Tous les statuts',
+        // C
+        'confirmer_deplacement' => 'Êtes-vous sûr de vouloir déplacer ces éléments ?',
+
+        // D
+        'deplacement_reussi' => 'Déplacement réussi.',
+        'deplacements_reussis' => '@nb@ déplacements réussis.',
+        'deplacement_en_echec' => "Échec d'un déplacement.",
+        'deplacements_en_echec' => '@nb@ déplacements en échec.',
+        
+
+        // E
+        'erreur_deplacement_impossible' => "Déplacement impossible !",
+        'erreur_autorisation_insuffisante' => "Autorisation insuffisante.",
+        'erreur_aucun_identifiant' => "Aucun identifiant d'objet.",
+        'erreur_rubriques_parentes_incorrectes' => "Rubriques parentes incorrectes.",
+        'erreur_deplacement' => "Déplacement non effectué.",
 
         // P
         'plan_titre' => 'Plan du site dans l’espace privé',
 
+        // T
+        'tous_les_statuts' => 'Tous les statuts',
+
+
 );

_plugins_/plan/branches/v1/paquet.xml

@@ -2 +2 @@
         prefix="plan"
         categorie="navigation"
-        version="1.2.4"
+        version="1.2.5"
         etat="test"
         compatibilite="[3.0.0;3.0.*]"

_plugins_/plan/branches/v1/prive/javascript/spiptree.js

@@ -21 +21 @@
                 }
         }
+
+        options.confirm = {
+                move: null
+        };
 
         $.each(options.objets, function(nom, desc) {
@@ -41 +45 @@
                 "core" : {
                         "animation" : 0,
-                        "check_callback" : true,
+                        "check_callback" : function (op, node, par, pos, more) {
+                                if (op === "move_node") {
+                                        // à la fin d'un déplacement, demander 1 fois (et 1 seule) 
+                                        // une confirmation, même si on déplace 5 items d'un coup
+                                        if (more && more.core) {
+                                                if (options.confirm.move === null) {
+                                                        options.confirm.move = confirm( options.textes.deplacement.confirmation );
+                                                }
+                                                return options.confirm.move;
+                                        } else {
+                                                // redemander la confirmation au prochain tour
+                                                options.confirm.move = null;
+                                        }
+                                }
+                                return true;
+                        },
                         "data" : function (node, cb) {
                                 // on est obligé de tout charger en ajax (même la racine)
@@ -77 +96 @@
         });
 
+        if (options.drag) {
+                $mytree.addClass('drag');
+        }
+
         // un clic d'une feuille amène sur son lien
         // mais… éviter que le plugin 'state' clique automatiquement lorsqu'il restaure
@@ -85 +108 @@
                         var node = data.instance.get_node(data.node, true);
                         if (node) {
-                                location.href = node.children('a').attr('href');
+                                var lien = node.children('a').attr('href');
+                                if (lien && !(options.drag && $(data.event.target).hasClass('jstree-icon'))) {
+                                        location.href = lien;
+                                }
                         }
                 });
         });
 
+        var recharge_plan = false;
         // lorsqu'on déplace un nœud
         $mytree.on("move_node.jstree", function(event, data) {
                 // si les parents sont identiques : pas de changement,
                 // on ne peut/veut pas gérer ici les positionnements
-
-                // console.log(data);
 
                 if (data.old_parent == data.parent) {
@@ -145 +170 @@
                         cache: false,
                 }).done(function(response) {
-                        // console.log('done', response);
-                        ajaxReload('contenu');
+
+                        if (response) {
+                                var nb_success = Object.keys(response.success).length;
+                                var nb_errors = Object.keys(response.errors).length;
+                                if (nb_success) {
+                                        $("#contenu #mytree_actions").after(
+                                                "<p class='success removable' onClick='$(this).remove();'>" +
+                                                        ((nb_success == 1)
+                                                                ? options.textes.deplacement.reussi
+                                                                : options.textes.deplacement.reussis.replace('@nb@', nb_success)) +
+                                                "</p>"
+                                        );
+                                }
+                                if (nb_errors) {
+                                        var texte = ((nb_errors == 1)
+                                                ? options.textes.deplacement.echec
+                                                : options.textes.deplacement.echecs.replace('@nb@', nb_errors));
+                                        $.each(response.errors, function(i, error) {
+                                                texte += "<br />[ " + i + "] " + error;
+                                        });
+                                        $("#contenu #mytree_actions").after(
+                                                "<p class='error removable' onClick='$(this).remove();'>" + texte + "</p>"
+                                        );
+                                }
+                        }
+
+                        if (recharge_plan) {
+                                clearTimeout(recharge_plan);
+                        }
+                        recharge_plan = setTimeout(function () {
+                                ajaxReload('plan');
+                        }, 500);
                 });
 

_plugins_/plan/branches/v1/prive/squelettes/contenu/plan.html

@@ -7 +7 @@
 (function($){
 $(document).ready(function(){
-        $('body.plan #contenu #racine').spiptree({
-                drag: [(#AUTORISER{configurer}|?{true,false})],
-                statut: "#ENV{statut}",
-                urls: {
-                        plan: "[(#VAL{plan}|generer_url_action{"", 1})]",
-                        deplacer: "[(#VAL{deplacer_objets}|generer_url_action{"", 1})]"
-                },
-                default: {
-                        icon: "[(#VAL{rubrique}|objet_icone{16}|extraire_attribut{src})]"
-                },
-                objets: [
-                        <BOUCLE_objets(DATA){source table,#REM|plan_lister_objets_rubrique}{", "}>
-                        {
-                                icon: "[(#VALEUR|objet_icone{16}|extraire_attribut{src})]",
-                                type: "[(#VALEUR|objet_type)]"
-                        }
-                        </BOUCLE_objets>
-                ]
-        });
         $('body.plan #contenu #mytree_actions .pliage').html(
                 "<a href=\"#\" onClick=\"My_jQuery('#racine').jstree('open_all'); return false;\"><:lien_tout_deplier:></a> | " +
@@ -68 +49 @@
 
 #BOITE_OUVRIR{'', simple}
-<div id="racine">
-<INCLURE{fond=prive/squelettes/inclure/plan-rubriques, id_parent=0, statut=#GET{statut}, objet=#GET{objet}, lister}>
-</div>
+<INCLURE{fond=prive/squelettes/inclure/plan, statut=#GET{statut}, objet=#GET{objet}, lister, ajax=plan}>
 #BOITE_FERMER

_plugins_/plan/branches/v1/prive/squelettes/inclure/plan.html

@@ -5 +5 @@
                 drag: [(#AUTORISER{configurer}|?{true,false})],
                 statut: "#ENV{statut}",
+                textes: {
+                        deplacement: {
+                                confirmation: <:plan:confirmer_deplacement|json_encode:>,
+                                reussi: <:plan:deplacement_reussi|json_encode:>,
+                                reussis: <:plan:deplacements_reussis|json_encode:>,
+                                echec: <:plan:deplacement_en_echec|json_encode:>,
+                                echecs: <:plan:deplacements_en_echec|json_encode:>,
+                        }
+                },
                 urls: {
                         plan: "[(#VAL{plan}|generer_url_action{"", 1})]",
@@ -22 +31 @@
         });
 });
-})(jQuery);
+})(My_jQuery);
 </script>