Changeset 90365 in spip-zone

Timestamp:

Jun 19, 2015, 3:45:18 PM (4 years ago)

Author:

cedric@…

Message:

on utilise le cache meta pour le compte des listes uniquement si il y a beaucoup d'abonnes (>10000) ce qui evite le lags sur les petits sites

Location:

_plugins_/mailsubscribers/trunk

Files:

• newsletter/subscribers.php (modified) (2 diffs)
• paquet.xml (modified) (1 diff)

_plugins_/mailsubscribers/trunk/newsletter/subscribers.php

@@ -42 +42 @@
                         AND !_request('var_mode')
                   AND isset($GLOBALS['meta']['newsletter_subscribers_count'])
-                  AND $c = unserialize($GLOBALS['meta']['newsletter_subscribers_count']))
-                        $count = $c;
+                  AND $c = unserialize($GLOBALS['meta']['newsletter_subscribers_count'])){
+                        // si beaucoup d'inscrits on utilise le cache,
+                        // sinon on fait un calcul peu couteux pour eviter les bugs, notamment au demarrage
+                        if (array_sum($c)>10000){
+                                $count = $c;
+                        }
+                }
                 if (is_null($count)){
                         $rows = sql_allfetsel("listes,count(id_mailsubscriber) as n","spip_mailsubscribers",$where,"listes");
@@ -55 +60 @@
                                 }
                         }
-                        ecrire_meta("newsletter_subscribers_count",serialize($count));
+                        // si beaucoup d'inscrits on met en cache
+                        if (array_sum($count)>10000){
+                                ecrire_meta("newsletter_subscribers_count",serialize($count));
+                        }
                 }
                 $liste = mailsubscribers_normaliser_nom_liste(reset($listes));

_plugins_/mailsubscribers/trunk/paquet.xml

@@ -2 +2 @@
         prefix="mailsubscribers"
         categorie="communication"
-        version="1.7.0"
+        version="1.7.1"
         etat="stable"
         compatibilite="[3.0.0;3.0.*]"