Changeset 788

Timestamp:

02/18/09 08:19:53 (14 years ago)

Author:

kwon

Message:

Newsletter 3.0.5rc6 : Ticket #65, #67

Location:

plugins/newsletter/trunk

Files:

• _admin.php (modified) (2 diffs)
• _prepend.php (modified) (1 diff)
• _public.php (modified) (2 diffs)
• _widgets.php (deleted)
• adapt_themes (added)
• adapt_themes/Notepad-Chaos.subscribe.newsletter.html (added)
• adapt_themes/Noviny.subscribe.newsletter.html (added)
• adapt_themes/freshy2.subscribe.newsletter.html (added)
• changelog.txt (modified) (1 diff)
• class.admin.php (modified) (1 diff)
• class.dc.newsletter.php (modified) (1 diff)
• class.modele.php (modified) (1 diff)
• class.plugin.php (modified) (7 diffs)
• default-templates/template.newsletter.html (added)
• locales/fr/help/help.html (modified) (1 diff)
• locales/fr/main.lang.php (modified) (1 diff)

plugins/newsletter/trunk/_admin.php

@@ -31 +31 @@
 $core->addBehavior('adminAfterPostUpdate', array('dcBehaviorsNewsletter', 'adminAutosend'));
 
+// initialisation du widget
+$core->addBehavior('initWidgets', array('dcBehaviorsNewsletter', 'widget'));
+
 // ajout de la gestion des url
 $core->url->register('newsletter', 'newsletter', '^newsletter/(.+)$', array('urlNewsletter', 'newsletter'));
@@ -45 +48 @@
 class dcBehaviorsNewsletter
 {
+     /**
+     * initialisation du widget
+     */
+     public static function widget(&$w)
+     {
+          global $core, $plugin_name;
+          try {
+               $w->create(pluginNewsletter::pname(), __('Newsletter'), array('WidgetsNewsletter', 'widget'));
+
+         $w->newsletter->setting('title', __('Title:'), __('Newsletter'));
+               $w->newsletter->setting('showtitle', __('Show title'), 1, 'check');
+               $w->newsletter->setting('homeonly', __('Home page only'), 0, 'check');
+               $w->newsletter->setting('inwidget', __('In widget'), 0, 'check');
+               $w->newsletter->setting('insublink', __('In sublink'), 1, 'check');
+               $w->newsletter->setting('subscription_link',__('Title subscription link:'),__('Subscription link'));
+           
+          } catch (Exception $e) { 
+               $core->error->add($e->getMessage()); 
+          }
+     }
+    
      /**
      * avant suppression du plugin par le gestionnaire, on le déinstalle proprement

plugins/newsletter/trunk/_prepend.php

@@ -24 +24 @@
 $GLOBALS['__autoload']['dcNewsletter'] = dirname(__FILE__).'/class.modele.php';
 
-require dirname(__FILE__).'/_widgets.php';
-
 ?>

plugins/newsletter/trunk/_public.php

@@ -51 +51 @@
      * initialisation du widget
      */
-    public static function widget($w)
-    {
+     public static function widget($w)
+     {
           global $core;
-          try
-          {
+          try {
                // prise en compte de l'état d'activation du plugin
-               if (!pluginNewsletter::isActive()) return;
+               if (!pluginNewsletter::isActive()) 
+                    return;
 
                // prise en compte paramètre: uniquement sur la page d'accueil
                $url = &$core->url;
-             if ($w->homeonly && $url->type != 'default') return;
+               if ($w->homeonly && $url->type != 'default')  {
+                    return;
+               }
 
                // paramétrage des variables
@@ -67 +69 @@
                $title = ($w->title) ? html::escapeHTML($w->title) : $plugin_name;
                $showTitle = ($w->showtitle) ? true : false;
-               $subtitle = ($w->subtitle) ? html::escapeHTML($w->subtitle) : __('Subscribe');
+               $subscription_link = ($w->subscription_link) ? html::escapeHTML($w->subscription_link) : __('Subscription link');
+               
                $text = '';
 
                // mise en place du contenu du widget dans $text
-             if ($w->inwidget)
-             {
-                $link = dcNewsletter::url('submit');
-                 $text .=
-                         '<form action="'.$link.'" method="post" id="nl_form">'.
-                         '<fieldset>'.
-                              $core->formNonce().
-                              form::hidden(array('nl_random'),dcNewsletter::getRandom()).
+               if ($w->inwidget) {
+                    $link = dcNewsletter::url('submit');
+                    $text .=
+                    '<form action="'.$link.'" method="post" id="nl_form">'.
+                    '<fieldset>'.
+                    $core->formNonce().
+                    form::hidden(array('nl_random'),dcNewsletter::getRandom()).
                               
-                              '<label for="nl_email">'.__('Email:').'</label>'.
-                              form::field(array('nl_email','nl_email'),30,255);
+                    '<label for="nl_email">'.__('Email:').'</label>'.
+                    form::field(array('nl_email','nl_email'),30,255);
 
-                              if (pluginNewsletter::getCaptcha())
-                              {
-                                   require_once dirname(__FILE__).'/class.captcha.php';                                 
+                    if (pluginNewsletter::getCaptcha()) {
+                         require_once dirname(__FILE__).'/class.captcha.php';                                 
                                    
-                                   $as = new Captcha(80, 30, 5);
-                                   $as->generate();
-                                   $as->file();
-                                   $as->write();
+                         $as = new Captcha(80, 30, 5);
+                         $as->generate();
+                         $as->file();
+                         $as->write();
                               
-                                   $text .=
-                                   '<label for="nl_captcha">'.__('Captcha:').'</label>'.
-                                   form::field(array('nl_captcha','nl_captcha'),30,255, '', 'style="width:90px; vertical-align:top;').
-                                   '<img src="'.Captcha::www().'/captcha.img.png" style="vertical-align: middle;" alt="'.__('Captcha').'" />';
-                              }
+                         $text .=
+                         '<label for="nl_captcha">'.__('Captcha:').'</label>'.
+                         form::field(array('nl_captcha','nl_captcha'),30,255, '', 'style="width:90px; vertical-align:top;').
+                         '<img src="'.Captcha::www().'/captcha.img.png" style="vertical-align: middle;" alt="'.__('Captcha').'" />';
+                    }
 
-                 $text .=
-                              '<label>'.form::radio(array('nl_option'),'subscribe', true).__('Subscribe').'</label><br />'.
-                              '<label>'.form::radio(array('nl_option'),'unsubscribe').__('Unsubscribe').'</label><br />'.
-                              '<input type="submit" name="nl_submit" id="nl_submit" value="'.__('Ok').'" />'.
-                         '</fieldset>'.
-                         '</form>';
-             }
-             else
-             {
-                $link = dcNewsletter::url('form');
-                 if ($w->insublink)
-                 {
-                     $title = str_replace('%I', '<img src="?pf=newsletter/icon.png" alt="" width="16" height="16" style="margin: 0 3px; vertical-align: middle;" />', $title);
-                     $text = $w->text ? $w->text : '';
+                    $text .=
+                    '<label>'.form::radio(array('nl_option'),'subscribe', true).__('Subscribe').'</label><br />'.
+                    '<label>'.form::radio(array('nl_option'),'unsubscribe').__('Unsubscribe').'</label><br />'.
+                    '<input type="submit" name="nl_submit" id="nl_submit" value="'.__('Ok').'" />'.
+                    '</fieldset>'.
+                    '</form>';
+               } else {
+                    $link = dcNewsletter::url('form');
+                    if ($w->insublink) {
+                         $title = str_replace('%I', '<img src="?pf=newsletter/icon.png" alt="" width="16" height="16" style="margin: 0 3px; vertical-align: middle;" />', $title);
+                         $text = $w->text ? $w->text : '';
 
-                     $text .= '<ul><li><a href="'.$link.'">'.$subtitle.'</a></li></ul>';
-                 }
-                 else
-                 {
-                     $title = '<a href="'.$link.'">'.$title.'</a>';
-                     $title = str_replace('%I', '</a><img src="?pf=newsletter/icon.png" alt="newsletter" width="16" height="16" style="margin:0 3px; vertical-align:middle;" /> <a href="'.$link.'">', $title);
-                 }
-             }
+                         $text .= '<ul><li><a href="'.$link.'">'.$subscription_link.'</a></li></ul>';
+                    } else {
+                         $title = '<a href="'.$link.'">'.$title.'</a>';
+                         $title = str_replace('%I', '</a><img src="?pf=newsletter/icon.png" alt="newsletter" width="16" height="16" style="margin:0 3px; vertical-align:middle;" /> <a href="'.$link.'">', $title);
+                    }
+               }
 
+               // renvoi du code à afficher pour le widget
+               if ($showTitle === true) 
+                    $title = '<h2>'.$title.'</h2>';
+               else 
+                    $title = '';
                
-               // renvoi du code à afficher pour le widget
-             if ($showTitle === true) $title = '<h2>'.$title.'</h2>';
-             else $title = '';
-             return "\n".'<div id="'.pluginNewsletter::pname().'">'.$title.$text.'</div>'."\n";
+               return "\n".'<div id="'.pluginNewsletter::pname().'">'.$title.$text.'</div>'."\n";
+          } catch (Exception $e) { 
+               $core->error->add($e->getMessage()); 
           }
-         catch (Exception $e) { $core->error->add($e->getMessage()); }
      }
 }

plugins/newsletter/trunk/changelog.txt

@@ -3 +3 @@
 * changement de la gestion de l'aide de la partie admin
 * remaniement de l'interface d'administration
-* désactivation pour le moment de la possibilité de s'inscrire par le widget
 * désactivation de la fonction de mise à jour pour éviter la redondance avec dotaddict
-* correction du Ticket #47 sur l'affichage d'une page vide lors du submit
-* correction du Ticket #61 sur le problème du caratère '=' dans l'URL
-* correction du Ticket #62 sur un bug dans l'envoi automatique
 * ajout d'une période de temps pour les billets de la newsletter (fixée à 1 mois dans cette version) 
 * remaniement du code
+* resolution du Ticket #47 : bug d'affichage d'une page vide lors du submit
+* resolution du Ticket #61 : bug avec le caratère '=' dans l'URL
+* resolution du Ticket #62 : bug dans l'envoi automatique
+* résolution du Ticket #65 : création de templates spécifiques
+* résolution du Ticket #67 : définir l'intitulé du bouton "S'abonner" 
 
 3.4.2

plugins/newsletter/trunk/class.admin.php

@@ -315 +315 @@
                $fautosend = pluginNewsletter::getAutosend();
                $fcaptcha = pluginNewsletter::getCaptcha();
+               /*
+               $f_subscribe_link = pluginNewsletter::getSubscribeLink();
+               //*/
                /* for 3.5.1 : add period
                $fperiod = pluginNewsletter::getPeriod();

plugins/newsletter/trunk/class.dc.newsletter.php

@@ -39 +39 @@
                $GLOBALS['newsletter']['cmd'] = null;
                $GLOBALS['newsletter']['msg'] = false;
-          $GLOBALS['newsletter']['form'] = false;
-          $GLOBALS['newsletter']['email'] = null;
-          $GLOBALS['newsletter']['code'] = null;
+               $GLOBALS['newsletter']['form'] = false;
+               $GLOBALS['newsletter']['email'] = null;
+               $GLOBALS['newsletter']['code'] = null;
 
                // décomposition des arguments et aiguillage
-          $params = explode('/', $args);
+               $params = explode('/', $args);
                if (isset($params[0]) && !empty($params[0])) 
                     $cmd = (string)html::clean($params[0]);

plugins/newsletter/trunk/class.modele.php

@@ -567 +567 @@
                // intervalle de récupération des messages (fixé à 1 mois pour le moment)
                //$params['sql'] .= ' AND P.post_dt BETWEEN SUBDATE(CURDATE(), INTERVAL 1 MONTH) AND NOW() ';
-         $params['sql'] .= ' AND P.post_dt BETWEEN SUBDATE(NOW(), INTERVAL 1 MONTH) AND NOW() ';
-         
+         // $params['sql'] .= ' AND P.post_dt BETWEEN SUBDATE(NOW(), INTERVAL 1 MONTH) AND NOW() ';
 
                // récupération des billets

plugins/newsletter/trunk/class.plugin.php

@@ -158 +158 @@
      * utilisation d'un captcha
      */
-   public static function getCaptcha() 
-   { 
-     return (boolean)self::get('captcha'); 
-   }
+     public static function getCaptcha() 
+     { 
+          return (boolean)self::get('captcha'); 
+     }
      
     /**
@@ -282 +282 @@
      * namespace pour le plugin
      */
-   static protected function namespace() 
-   { 
-     return (string)self::pname(); 
-   }
+     protected static function namespace() 
+     { 
+          return (string)self::pname(); 
+     }
 
      /**
      * préfix pour ce plugin
      */
-   static protected function prefix() 
-   { 
-     return (string)self::namespace().'_'; 
-   }
+     protected static function prefix() 
+     { 
+          return (string)self::namespace().'_'; 
+     }
 
      /**
@@ -301 +301 @@
      {
           global $core;
-      try {
-          $blog = &$core->blog;
+          try {
+               $blog = &$core->blog;
                $blog->triggerBlog();
-      }
-        catch (Exception $e) { 
-          $core->error->add($e->getMessage()); 
-        }
+          } catch (Exception $e) { 
+               $core->error->add($e->getMessage()); 
+          }
      }
 
@@ -316 +315 @@
      {
           global $core;
-      try {
+          try {
                http::redirect($url);
-      }
-        catch (Exception $e) { 
-          $core->error->add($e->getMessage()); 
-        }
+          } catch (Exception $e) { 
+               $core->error->add($e->getMessage()); 
+          }
      }
 
@@ -327 +325 @@
      * lit le paramètre
      */
-   public static function get($param, $global=false)
-   {
-          global $core;
-      try {
+     public static function get($param, $global=false)
+     {
+          global $core;
+          try {
                $blog = &$core->blog;
-           $settings = &$blog->settings;
-         if (!$global) 
-          $settings->setNamespace(self::namespace());
-         return (string)$settings->get(self::prefix().$param);
-     } catch (Exception $e) { 
-          $core->error->add($e->getMessage()); 
-     }
-   }
+               $settings = &$blog->settings;
+               if (!$global) {
+                    $settings->setNamespace(self::namespace());
+               }
+               return (string)$settings->get(self::prefix().$param);
+          } catch (Exception $e) { 
+               $core->error->add($e->getMessage()); 
+          }
+     }
 
      /**
@@ -347 +346 @@
      {
           global $core;
-      try {
-           $blog = &$core->blog;
-           $settings = &$blog->settings;
-         if (isset($settings->$param)) 
-          return true;
+          try {
+               $blog = &$core->blog;
+               $settings = &$blog->settings;
+               if (isset($settings->$param)) 
+                    return true;
                else 
                     return false;
-     } catch (Exception $e) { 
-          $core->error->add($e->getMessage()); 
-     }
+          } catch (Exception $e) { 
+               $core->error->add($e->getMessage()); 
+          }
      }
 
@@ -362 +361 @@
      * enregistre une chaine dans le paramètre
      */
-   public static function setS($param, $val, $global=false)
-   {
-          global $core;
-      try {
+     public static function setS($param, $val, $global=false)
+     {
+          global $core;
+          try {
                $blog = &$core->blog;
-           $settings = &$blog->settings;
-         $settings->setNamespace(self::namespace());
-         $settings->put((string)self::prefix().$param, (string)$val, 'string', null, true, $global);
-     } catch (Exception $e) { 
-          $core->error->add($e->getMessage()); 
-     }
+               $settings = &$blog->settings;
+               $settings->setNamespace(self::namespace());
+               $settings->put((string)self::prefix().$param, (string)$val, 'string', null, true, $global);
+          } catch (Exception $e) { 
+               $core->error->add($e->getMessage()); 
+          }
    }
 

plugins/newsletter/trunk/locales/fr/help/help.html

@@ -74 +74 @@
 <dl>
      <dt>Intégration au thème :</dt>
-     <dd>Vous pouvez ajouter le code à suivant votre thème pour afficher le formulaire d'abonnement à la newsletter :
-     <pre>{{tpl:NewsletterSubscription}}</pre>
-     </dd>
+     <dd><ul>
+     <li><strong>Installer le template spécifique pour votre thème : </strong>
+     Copier le fichier depuis le répertoire <pre>plugins/newsletter/adapt_themes/nom_theme.subscribe.newsletter.html</pre>
+          qui vous concerne et déposer dans le répertoire tpl de votre theme. <br />
+          Renomer le fichier en subscribe.newsletter.html 
+          </li>
+     <li>Vous pouvez ajouter le code suivant à votre thème pour afficher le formulaire d'abonnement à la newsletter :
+          <pre>{{tpl:NewsletterSubscription}}</pre>
+     </li>
+     </ul></dd>
 </dl>
 <h3>A propos</h3>

plugins/newsletter/trunk/locales/fr/main.lang.php

@@ -236 +236 @@
 $GLOBALS['__l10n']['Subscribe'] = "S'abonner";
 $GLOBALS['__l10n']['Unsubscribe'] = "Se désabonner";
+$GLOBALS['__l10n']['Title subscription link:'] = "Titre du lien de souscription:";
+$GLOBALS['__l10n']['Subscription link'] = "S'abonner";
 
 // _install