From 1970c12b2252d9357f59598e69b6c6044ef4e4c2 Mon Sep 17 00:00:00 2001 From: x2003bruneau Date: Wed, 17 Jan 2007 13:31:00 +0000 Subject: [PATCH] Refuse empty articles git-svn-id: svn+ssh://murphy/home/svn/platal/trunk@1350 839d8a87-29fc-0310-9880-83ba4fa771e5 --- modules/events.php | 4 +++- modules/newsletter.php | 46 +++++------------------------------------- modules/register.php | 2 +- modules/xnetgrp.php | 5 +++++ templates/events/index.tpl | 4 +++- templates/events/submit.tpl | 2 +- templates/newsletter/index.tpl | 10 ++++----- templates/newsletter/nl.tpl | 2 +- templates/xnetgrp/asso.tpl | 2 +- 9 files changed, 25 insertions(+), 52 deletions(-) diff --git a/modules/events.php b/modules/events.php index 454a55c..38244aa 100644 --- 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 --- 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 --- 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 --- 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 --- a/templates/events/index.tpl +++ b/templates/events/index.tpl @@ -148,7 +148,9 @@ Bienvenue {$smarty.session.prenom}

- {icon name=add}Proposer une information événementielle + {icon name=page_edit}  + Proposer une information événementielle • + Proposer un article pour la Lettre mensuelle

{/if} diff --git a/templates/events/submit.tpl b/templates/events/submit.tpl index 27ad082..756c3e4 100644 --- a/templates/events/submit.tpl +++ b/templates/events/submit.tpl @@ -22,7 +22,7 @@

Proposition d'information événementielle

-{if $action eq "proposer"} +{if $action eq "proposer" && trim($titre) && trim($texte)}

Voici ton annonce : diff --git a/templates/newsletter/index.tpl b/templates/newsletter/index.tpl index 8b30458..0629a8a 100644 --- a/templates/newsletter/index.tpl +++ b/templates/newsletter/index.tpl @@ -25,9 +25,9 @@ Lettre de Polytechnique.org -

-Pour demander l'ajout d'une annonce dans la prochaine lettre mensuelle, -utilise le formulaire dédié ! + +

+ {icon name=page_edit value="Proposer un article"} Proposer un article pour la lettre mensuelle

Ton statut

@@ -37,14 +37,14 @@ utilise 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 des préférences)

- [me désinscrire de la lettre mensuelle] + [{icon name=delete} me désinscrire de la lettre mensuelle]
{else}

Tu n'es actuellement pas inscrit à la lettre mensuelle de Polytechnique.org.

- [m'inscrire à la lettre mensuelle] + [{icon name=add} m'inscrire à la lettre mensuelle]
{/if} diff --git a/templates/newsletter/nl.tpl b/templates/newsletter/nl.tpl index 3e758d6..5225639 100644 --- 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}
- 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.
[archives | diff --git a/templates/xnetgrp/asso.tpl b/templates/xnetgrp/asso.tpl index 45136aa..659f346 100644 --- a/templates/xnetgrp/asso.tpl +++ b/templates/xnetgrp/asso.tpl @@ -157,7 +157,7 @@ interpr {/if} {elseif $is_admin}
- [{icon name=add} Ajouter une annonce] + [{icon name=page_edit} Ajouter une annonce]
{/if} -- 2.1.4