Refuse empty articles

git-svn-id: svn+ssh://murphy/home/svn/platal/trunk@1350 839d8a87-29fc-0310-9880-83ba4fa771e5

diff --git a/modules/events.php b/modules/events.php
index 454a55c..38244aa 100644 (file)
--- a/modules/events.php
+++ b/modules/events.php
@@ -200,7 +200,9 @@ class EventsModule extends PLModule
         $page->assign('valid_mesg', $valid_mesg);
         $page->assign('action', strtolower($action));
 
-        if ($action == 'Confirmer') {
+        if ($action && (!trim($texte) || !trim($titre))) {
+            $page->trig("L'article doit avoir un titre et un contenu");
+        } elseif ($action == 'Confirmer') {
                $texte = $texte_catch_url;
             require_once 'validations.inc.php';
             $evtreq = new EvtReq($titre, $texte, $promo_min, $promo_max,

diff --git a/modules/newsletter.php b/modules/newsletter.php
index a24757a..8ede54a 100644 (file)
--- a/modules/newsletter.php
+++ b/modules/newsletter.php
@@ -30,12 +30,6 @@ class NewsletterModule extends PLModule
             'admin/newsletter'             => $this->make_hook('admin_nl', AUTH_MDP, 'admin'),
             'admin/newsletter/categories'  => $this->make_hook('admin_nl_cat', AUTH_MDP, 'admin'),
             'admin/newsletter/edit'        => $this->make_hook('admin_nl_edit', AUTH_MDP, 'admin'),
-
-            'ax'             => $this->make_hook('ax',        AUTH_COOKIE),
-            'ax/show'        => $this->make_hook('ax_show',   AUTH_COOKIE),
-            'ax/submit'      => $this->make_hook('ax_submit', AUTH_MDP),
-            'ax/edit'        => $this->make_hook('ax_submit', AUTH_MDP),
-            'admin/axletter/rights'        => $this->make_hook('admin_ax_rights', AUTH_MDP, 'admin'),
         );
     }
 
@@ -81,7 +75,10 @@ class NewsletterModule extends PLModule
 
         require_once 'newsletter.inc.php';
 
-        if (Post::has('see')) {
+        if (Post::has('see') || (Post::has('valid') && (!trim(Post::v('title')) || !trim(Post::v('body'))))) {
+            if (!Post::has('see')) {
+                $page->trig("L'article doit avoir un titre et un contenu");
+            }
             $art = new NLArticle(Post::v('title'), Post::v('body'), Post::v('append'));
             $page->assign('art', $art);
         } elseif (Post::has('valid')) {
@@ -91,6 +88,7 @@ class NewsletterModule extends PLModule
             $art->submit();
             $page->assign('submited', true);
         }
+        $page->addCssLink('nl.css');
     }
 
     function handler_admin_nl(&$page, $new = false) {
@@ -161,40 +159,6 @@ class NewsletterModule extends PLModule
         $table_editor->describe('pos','position',true);
         $table_editor->apply($page, $action, $id);
     }
-
-    function handler_ax(&$page, $action = null)
-    {
-        require_once 'newsletter.inc.php';
-
-        $page->changeTpl('newsletter/ax.tpl');
-        $page->assign('xorg_title','Polytechnique.org - Envois de l\'AX');
-
-        switch ($action) {
-          case 'out': AXLetter::unsubscribe(); break;
-          case 'in':  AXLetter::subscribe(); break;
-          default: ;
-        }
-
-        $perm = AXLetter::hasPerms();
-        if ($perm) {
-            $waiting = AXLetter::awaiting();
-            if ($waiting) {
-                $new = new AXLetter($waiting);
-                $page->assign('new', $new);
-            }
-        }
-        $page->assign('axs', AXLetter::subscriptionState());
-        $page->assign('ax_list', AXLetter::listSent());
-        $page->assign('ax_rights', AXLetter::hasPerms());
-    }
-
-    function handler_ax_submit(&$page)
-    {
-        require_once('newsletter.inc.php');
-        if (!AXLetter::hasPerms()) {
-            return PL_FORBIDDEN;
-        }
-    }
 }
 
 ?>

diff --git a/modules/register.php b/modules/register.php
index 48aa521..4e57aff 100644 (file)
--- a/modules/register.php
+++ b/modules/register.php
@@ -131,7 +131,7 @@ class RegisterModule extends PLModule
                         $err[] = $sub_state['forlife']." doit renvoyer vers un email existant ".
                             "valide, en particulier, il ne peut pas être renvoyé vers lui-même.";
                     }
-                    $birth = Env::v('naissance');
+                    $birth = trim(Env::v('naissance'));
                     if (!preg_match('/^[0-3][0-9][01][0-9][12][90][0-9][0-9]$/', $birth)) {
                         $err[] = "La 'Date de naissance' n'est pas correcte.";
                     } else {

diff --git a/modules/xnetgrp.php b/modules/xnetgrp.php
index 94c0809..bc8bed5 100644 (file)
--- a/modules/xnetgrp.php
+++ b/modules/xnetgrp.php
@@ -990,6 +990,11 @@ class XnetGrpModule extends PLModule
                 $page->trig("L'intervalle de promotions est invalide");
                 Post::kill('valid');
             }
+
+            if (!trim($art['titre']) || !trim($art['texte'])) {
+                $page->trig("L'article doit avoir un titre et un contenu");
+                Post::kill('valid');
+            }
         }
 
         if (Post::v('valid') == 'Enregistrer') {

diff --git a/templates/events/index.tpl b/templates/events/index.tpl
index ad06e8b..58628f1 100644 (file)
--- a/templates/events/index.tpl
+++ b/templates/events/index.tpl
@@ -148,7 +148,9 @@ Bienvenue {$smarty.session.prenom}
   </p>
 
   <p class="center">
-  <a href="events/submit">{icon name=add}Proposer une information événementielle</a>
+    {icon name=page_edit}&nbsp;
+    <a href="events/submit">Proposer une information événementielle</a>&nbsp;&bull;
+    <a href="nl/submit">Proposer un article pour la Lettre mensuelle</a>
   </p>
 {/if}
 

diff --git a/templates/events/submit.tpl b/templates/events/submit.tpl
index 27ad082..756c3e4 100644 (file)
--- a/templates/events/submit.tpl
+++ b/templates/events/submit.tpl
@@ -22,7 +22,7 @@
 
 <h1>Proposition d'information événementielle</h1>
 
-{if $action eq "proposer"}
+{if $action eq "proposer" && trim($titre) && trim($texte)}
 
 <p>
 Voici ton annonce :

diff --git a/templates/newsletter/index.tpl b/templates/newsletter/index.tpl
index 8b30458..0629a8a 100644 (file)
--- a/templates/newsletter/index.tpl
+++ b/templates/newsletter/index.tpl
@@ -25,9 +25,9 @@
   Lettre de Polytechnique.org
 </h1>
 
-<p>
-<strong>Pour demander l'ajout d'une annonce dans la prochaine lettre mensuelle</strong>,
-utilise <a href='nl/submit'>le formulaire dédié !</a>
+
+<p class="center">
+  <a href="nl/submit">{icon name=page_edit value="Proposer un article"} Proposer un article pour la lettre mensuelle</a>
 </p>
 
 <h2>Ton statut</h2>
@@ -37,14 +37,14 @@ utilise <a href='nl/submit'>le formulaire d
 Tu es actuellement inscrit à la lettre mensuelle de Polytechnique.org (pour choisir le format HTML ou texte, rends toi sur la page <a href='prefs'>des préférences</a>)
 </p>
 <div class='center'>
-  [<a href='nl/out'>me désinscrire de la lettre mensuelle</a>]
+  [<a href='nl/out'>{icon name=delete} me désinscrire de la lettre mensuelle</a>]
 </div>
 {else}
 <p>
 Tu n'es actuellement pas inscrit à la lettre mensuelle de Polytechnique.org.
 </p>
 <div class='center'>
-  [<a href='nl/in'>m'inscrire à la lettre mensuelle</a>]
+  [<a href='nl/in'>{icon name=add} m'inscrire à la lettre mensuelle</a>]
 </div>
 {/if}
 

diff --git a/templates/newsletter/nl.tpl b/templates/newsletter/nl.tpl
index 3e758d6..5225639 100644 (file)
--- a/templates/newsletter/nl.tpl
+++ b/templates/newsletter/nl.tpl
@@ -108,7 +108,7 @@ ne plus recevoir : [https://www.polytechnique.org/nl/out]
       {/foreach}
       {/foreach}
       <div class="foot1">
-        Cette lettre est envoyée à tous les Polytechniciens sur Internet par l'intermédiaire de lytechnique.org.
+        Cette lettre est envoyée à tous les Polytechniciens sur Internet par l'intermédiaire de Polytechnique.org.
       </div>
       <div class="foot2">
         [<a href="https://www.polytechnique.org/nl">archives</a>&nbsp;|

diff --git a/templates/xnetgrp/asso.tpl b/templates/xnetgrp/asso.tpl
index 45136aa..659f346 100644 (file)
--- a/templates/xnetgrp/asso.tpl
+++ b/templates/xnetgrp/asso.tpl
@@ -157,7 +157,7 @@ interpr
 {/if}
 {elseif $is_admin}
 <div class="center">
-  [<a href="{$platal->ns}announce/new">{icon name=add} Ajouter une annonce</a>]
+  [<a href="{$platal->ns}announce/new">{icon name=page_edit} Ajouter une annonce</a>]
 </div>
 {/if}