Vérifications plus strictes lors d'une souscription.

[spip_souscriptions.git] / formulaires / souscription.php

diff --git a/formulaires/souscription.php b/formulaires/souscription.php
index 82a523d..0ee265d 100644 (file)
--- a/formulaires/souscription.php
+++ b/formulaires/souscription.php
@@ -13,6 +13,7 @@ if (!defined('_ECRIRE_INC_VERSION')) return;
 
 include_spip('inc/actions');
 include_spip('inc/editer');
+include_spip('inc/config');
 
 /**
  * Identifier le formulaire en faisant abstraction des paramètres qui ne représentent pas l'objet edité
@@ -74,10 +75,15 @@ function formulaires_souscription_charger_dist($id_souscription_campagne) {
                         "spip_souscription_campagnes",
                         "id_souscription_campagne=$id_souscription_campagne");
 
+  $recu_fiscal = "";
+  if($type == "adhesion")
+    $recu_fiscal = "on";
+
   return array('montant' => '',
                'courriel' => '',
-               'recu_fiscal' => '',
+               'recu_fiscal' => $recu_fiscal,
                'envoyer_info' => 'on',
+               'informer_comite_local' => 'on',
                'prenom' => '',
                'nom' => '',
                'adresse' => '',
@@ -123,14 +129,6 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne)
     $erreurs['message_erreur'] = "La campagne à laquelle est associée cette souscription est invalide";
   }
 
-  if(_request('recu_fiscal')) {
-    foreach(array('prenom', 'nom', 'adresse', 'code_postal', 'ville') as $obligatoire) {
-      if(!_request($obligatoire)) {
-        $erreurs[$obligatoire] = 'Ce champ est obligatoire (reçu fiscal demandé)';
-      }
-    }
-  }
-
   if(!$id_souscription_campagne || intval($id_souscription_campagne) != intval($campagne)) {
       $erreurs['message_erreur'] = "Campagne invalide";
   }
@@ -141,7 +139,7 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne)
                         "spip_souscription_campagnes",
                         "id_souscription_campagne=$id_souscription_campagne");
 
-  if(!$type || !in_array($type, array("don", "adhesion", "abonnement")))
+  if(!$type || !in_array($type, array("don", "adhesion")))
     $erreurs['message_erreur'] = "Type de souscription invalide";
 
   /* Le champ 'type' (hidden) doit être le même que celui défini dans
@@ -149,12 +147,39 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne)
   if(_request('type_souscription') != $type)
     $erreurs['message_erreur'] = "Type de souscription invalide: " . _request('type_souscription');
 
+
+  if(_request('recu_fiscal') || $type == "adhesion") {
+    foreach(array('prenom', 'nom', 'adresse', 'code_postal', 'ville') as $obligatoire) {
+      if(!_request($obligatoire)) {
+        if($type == "adhesion") {
+          $erreurs[$obligatoire] = "Ce champ est obligatoire pour les adhésions";
+        }
+        else {
+          $erreurs[$obligatoire] = 'Ce champ est obligatoire (reçu fiscal demandé)';
+        }
+      }
+    }
+  }
+
   if ($e = _request('courriel') AND !email_valide($e))
     $erreurs['courriel'] = _T('form_prop_indiquer_email');
 
-  if ($e = _request('montant') AND !(ctype_digit($e))) {
-    /* FIXME: vérifier que le montant est compris dans les bornes. */
-    $erreurs['montant'] = "Montant invalide";
+  if ($e = _request('code_postal') AND !preg_match("/^(2[ABab]|0[1-9]|[1-9][0-9])[0-9]{3}$/", $e)) {
+    $erreurs['code_postal'] = "Code postal invalide";
+  }
+
+  if ($e = _request('montant')) {
+    if(!(ctype_digit($e)))
+      $erreurs['montant'] = "Montant invalide";
+    else {
+      $type_saisie = lire_config("souscription/${type}_type_sasie");
+
+      /* On ne vérifie strictement la valeur du montant que si on
+       * n'utilise pas le type de saisie « entrée libre » (input) pour
+       * le montant. */
+      if(($type_saisie != "input") AND !array_key_exists($e, lire_config("souscription/${type}_montants")))
+        $erreurs['montant'] = "Le montant spécifié est invalide";
+    }
   }
 
   return $erreurs;