Changeset 1782839

Timestamp:

12/07/2017 05:02:16 PM (8 years ago)

Author:

bgin

Message:

Suppression d'un dysfonctionnement : perte de la sélection d'un post lors de la mise à jour de celui-ci.

Location:

org-frontpage/trunk

Files:

• org-frontpage-plugin-admin.php (modified) (1 diff)
• org-frontpage.php (modified) (11 diffs)
• readme.txt (modified) (3 diffs)

org-frontpage/trunk/org-frontpage-plugin-admin.php

@@ -19 +19 @@
         }
         }
-        unset($_REQUEST['action'] ); // ne faudrait-iil pas 'save' au lieu de 'action'
+        unset($_REQUEST['save'] ); // remplacé 'action' par 'save' le 7/12/2017 ... à surveiller
         
 }

org-frontpage/trunk/org-frontpage.php

@@ -3 +3 @@
 Plugin Name: Organisation FrontPage
 Description: Adding a meta box in the post editor to set the rank of post in the home page
-Version: 1.4.7
+Version: 1.5
 Author:      B Gineste
 Text Domain: org-frontpage
@@ -11 +11 @@
 */
 ?><?php
-$en_test = true;
+$en_test = false;
 
 class org_frontpage {
@@ -28 +28 @@
     // tableau des libellés (pour la saisie dans la méta box)
     private $_libelles = array();
+    //fichier mouchard-org-fpg
+    private $_f;
+    private $_trace = false;
     
     function __construct() {
@@ -33 +36 @@
         add_action( 'plugins_loaded', array($this,'org_frontpage_load_plugin_textdomain') );
 
+        //ouverture du mouchard
+        if ($this->_trace) { $this->_f=fopen($_SERVER['DOCUMENT_ROOT'].'\mouchard-org-fpg.txt','a+'); }
         //définition des variables globales
             // Nombre de posts en frontpage (méta donnée dans wp_options)
@@ -51 +56 @@
             for ($i=1;$i<=$this->_nb_posts_frontpage;$i++) {$this->_option_names[$i] = "ofp_post_frontpage_".$i; } 
             // tableau des valeurs dans wp_options (option_value)
-            foreach ($this->_option_names as $option_name) {$this->_option_values[$option_name]=get_option($option_name); }
+            foreach ($this->_option_names as $option_name) {
+                //après une mise à jour du post, l'option_value prend la valeur de l'ID du post révision, il faut rétablir l'ID du post parent
+                //cet inconvénient  ne semble pouvoir être évité, même si les options ont 'no' comme valeur du champ autoload
+                
+                $thepost_id = get_option($option_name);
+                $thepost = get_post($thepost_id);
+                if ($thepost->post_parent > 0) {
+                    $thepost_revision_id = $thepost_id;
+                    $thepost_id = $thepost->post_parent; 
+                    update_option($option_name,$thepost_id); 
+                    if ($this->_trace) { 
+                        fwrite($this->_f,"Rétabli post parent"."\r\n");
+                        fwrite($this->_f,$thepost_revision_id." ".$thepost->post_parent."\r\n");
+                    }
+                }
+                $this->_option_values[$option_name]=$thepost_id;
+                }
             // tableau des libellés (pour la saisie dans la méta box) : initialisé au moment de l'utiliser  (fonction les_libelles) : ici, il n'est pas traduit
-            /*
-            $libelles["pas_sur_frontpage"]="Pas sur la page d'accueil";
-            $i=1;
-            foreach ($this->_option_names as $option_name) {$libelles[$option_name]="N° ".$i." / page d'accueil"; $i++;}
-            */
+
             add_action('add_meta_boxes', array($this,'rang_post_meta_box_add' ));
             add_action('save_post', array($this,'rang_post_meta_box_save' ));
@@ -80 +97 @@
     }
 
-
+    function __destruct() {
+        if ($this->_trace) { fclose($this->_f); }
+    }
 /*
 ============================================================================================================================
@@ -102 +121 @@
     $this->generer_les_libelles();
     // déterminer le rang de l'article sur la frontpage (par son option_name)
+    // ici on pourrait vérifier que le post n'a pas de parent. Et le cas échéant remplacer l'ID du post par l'ID du parent (semble inutile)
+    //$post_id = $post->ID;
+    //if ($post->post_parent > 0) {$post_id = $post->post_parent; }
+    //$rang_post = array_search($post_id,$this->_option_values);
     $rang_post = array_search($post->ID,$this->_option_values);
+    
     if ($rang_post == false) {$rang_post="pas_sur_frontpage";}
+    
     ?>
     <div id="box-org-frontpage" style="display: flex; flex-wrap: wrap;">
@@ -111 +136 @@
                 // We'll use this nonce field later on when saving.
                  wp_nonce_field( basename( __FILE__ ), 'organisation_frontpage' );
+                 //echo $post->ID; echo " - "; echo $post_id; echo " - "; echo $rang_post;
             ?>
             <label for="rang_frontpage"><?php _e( 'Post location', 'org-frontpage' ); ?> </label>
@@ -127 +153 @@
                 $nom_option = "ofp_post_frontpage_".$i; 
                 $post_id = get_option($nom_option);
-                $titre='';
-                $article=null;
-                if (!empty($post_id)) {
-                    $article = get_post( $post_id );
-                }
-                if (!empty($article)) {
-                    $titre = $article->post_title;
+                if ($post_id != false) {
+                $article = get_post( $post_id ); 
+                $titre = $article->post_title;
         ?> 
         <!--Articles -->
@@ -158 +180 @@
 function rang_post_meta_box_save( $post_id )
 {
+    if ($this->_trace) { fwrite($this->_f,"\r\n"."Début séquence save ".date('d/m/Y H:i')."\r\n"); }
+
     $this->generer_les_libelles();
     // Bail if we're doing an auto save
@@ -175 +199 @@
         // déterminer le rang précédent de l'article sur la frontpage
         $name_option_post_prec = array_search($post_id,$this->_option_values);
-        if ($name_option_post_prec == false) {$name_option_post_rec="pas_sur_frontpage";}
+            fwrite($this->_f, "name_option_post_prec (devrait être false si pas sur la frontpage) :");
+            fwrite($this->_f,$name_option_post_prec."\r\n");
+        if (empty($name_option_post_prec)) {$name_option_post_rec="pas_sur_frontpage";}
+        if ($this->_trace) {
+            
+            fwrite($this->_f, "Post ID :");
+            fwrite($this->_f,$post_id."\r\n");
+            
+            fwrite($this->_f, "libelle_frontpage :");
+            fwrite($this->_f,$libelle_frontpage."\r\n");
+            fwrite($this->_f,"name_option_post :");
+            fwrite($this->_f,$name_option_post."\r\n");
+            fwrite($this->_f,"name_option_post_prec :");
+            fwrite($this->_f,$name_option_post_prec."\r\n");
+            fwrite($this->_f,"Id des stickés (courant ".$post_id.") : "."\r\n");
+        
+            //var_dump(get_post($post_id));
+            foreach($this->_option_values as $opt) {
+                $article = get_post( $opt ); 
+                fwrite($this->_f,$article->post_title." - ");
+                fwrite($this->_f,$opt."\r\n");
+                }
+            
+            foreach($this->_libelles as $lib) {
+                fwrite($this->_f,$lib."\r\n");          
+            }
+        }
         
         if ($name_option_post != $name_option_post_prec) {
-            if ($name_option_post_prec != "pas_sur_frontpage") { update_option($name_option_post_prec,FALSE); }
-            if ($name_option_post != "pas_sur_frontpage") { update_option($name_option_post,$post_id); }
+            if ($name_option_post_prec == "pas_sur_frontpage") {
+                update_option($name_option_post,$post_id);
+                    if ($this->_trace) {
+                        fwrite($this->_f,"UPDATE name_option_post :"."\r\n");
+                        fwrite($this->_f,$name_option_post.' : '.$post_id."\r\n");
+                    }
+}
+            if ($name_option_post == "pas_sur_frontpage") {
+                update_option($name_option_post_prec,FALSE);
+                    if ($this->_trace) {
+                        fwrite($this->_f,"UPDATE name_option_post_prec :"."\r\n");
+                        fwrite($this->_f,$name_option_post_prec.' : false'."\r\n");
+                    }
+                }
+            if ($name_option_post_prec != "pas_sur_frontpage" && $name_option_post != "pas_sur_frontpage") {
+                update_option($name_option_post_prec,FALSE);
+                update_option($name_option_post,$post_id);
+                    if ($this->_trace) {
+                        fwrite($this->_f,"UPDATE name_option_post_prec :"."\r\n");
+                        fwrite($this->_f,$name_option_post_prec.' : false'."\r\n");
+                        fwrite($this->_f,"UPDATE name_option_post :"."\r\n");
+                        fwrite($this->_f,$name_option_post.' : '.$post_id."\r\n");
+                    }
+            }
+            
         }           
     }
-        
+        if ($this->_trace) {
+            fwrite($this->_f, "Post ID :");
+            fwrite($this->_f,$post_id."\r\n");
+            foreach($this->_option_values as $opt) {
+                $article = get_post( $opt ); 
+                fwrite($this->_f,$article->post_title." - ");
+                fwrite($this->_f,$opt."\r\n");
+            }
+        }
+//  fclose($f);         
 }
 

org-frontpage/trunk/readme.txt

@@ -4 +4 @@
 Requires at least: 4.4.0
 Tested up to: 4.9
-Stable tag: 1.4
+Stable tag: 1.5
 License: GPLv2 or later
 License URI: https://www.gnu.org/licenses/gpl-2.0.html
 
-Permet de choisir les posts qui doivent figurer sur la frontpage.
+Permet de choisir et agencer les posts de la frontpage.
 
 == Description ==
@@ -61 +61 @@
 = Ce plugin est-il nécessaire ? =
 
-J'ai éprouvé le besoin de marquer les posts à mettre en évidence sur la page d'accueil des blogs que je gère
+Je souhaitais choisir les posts à mettre sur la page d'accueil et les disposer selon leur niveau d'importance
 et je n'ai pas trouvé le moyen de le faire simplement.
+
+= Il existe un moyen simple de désigner les articles à mettre en avant (attribut sticky disponible dans le tableau de bord). Ne permet-il pas de faire la même chose sans plugin ? =
+
+Hélas non !
+On pourrait en effet donner l'attribut sticky aux posts qu'on souhaite mettre sur la page d'accueil, mais cela ne permet pas de les agencer à sa convenance sur cette page : on peut en effet les récupérer classés par date et les afficher dans cet ordre, mais il est impossible de les classer à son gré.
+Le plugin org-frontpage permet une mise en page fine car il introduit un attribut de classement (Numéro 1, 2, 3,...) des posts sélectionnés. Ce clasement pourra être exploité pour positionner les articles dans des emplacements dédiés au posts 1, 2, 3 ...
+
+De plus, contrairement au système natif de mise en avant, les articles retenus pour la frontpage ne viennent pas automatiquement en tête des listes d'articles de type blog.
+ 
 
 = Comment savoir quels sont les posts épinglés à la page d'accueil ? =
 
-On peut évidemment les voir sur le blog, mais aussi à partir du tableau de bord, dans le sous-menu *Org frontpage* du menu *Réglages* .
+On peut évidemment les voir sur le blog, mais aussi à partir du tableau de bord, dans le sous-menu *Org frontpage* du menu *Réglages* . Le rang des posts choisis apparaît également dans une colonne supplémentaire le la liste de tous les articles.
 
 = Pourquoi ce readme est-il en français ? =
@@ -115 +124 @@
 = 1.4.7 =
 * Correction d'un bug
+
+= 1.5 =
+* Correction d'un dysfonctionnement : lorsque on procède à la mise à jour d'un post marqué pour la page d'accueil, la sélection était perdue et il fallait la rétablir.