X-Git-Url: http://olivier.miskin.fr/git/?a=blobdiff_plain;f=formulaires%2Fsouscription.php;h=f4aed2738ae9b1042bd6c8c4a18fe489512146d4;hb=df311c2591aa33a1a73fea988d05ac1b10267617;hp=37d42353cb5eefcdd983417ad0e033846875401b;hpb=cf46f25623c27c4f86688a09a645db0c96c10f1a;p=spip_souscriptions.git

diff --git a/formulaires/souscription.php b/formulaires/souscription.php
index 37d4235..f4aed27 100644
--- 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é
@@ -37,8 +38,7 @@ function formulaires_souscription_identifier_dist($id_souscription='new',
                                                       $lier_trad=0,
                                                       $config_fonc='',
                                                       $row=array(),
-                                                      $hidden='')
-{
+                                                      $hidden='') {
   return serialize(array(intval($id_souscription)));
 }
 
@@ -88,6 +88,8 @@ function formulaires_souscription_charger_dist($id_souscription_campagne) {
                'adresse' => '',
                'code_postal' => '',
                'ville' => '',
+               'pays' => 'FR',
+               'telephone' => '',
                'id_souscription_campagne' => $id_souscription_campagne,
                'type_souscription' => $type,
                );
@@ -115,8 +117,7 @@ function formulaires_souscription_charger_dist($id_souscription_campagne) {
  * @return array
  *     Tableau des erreurs
  */
-function formulaires_souscription_verifier_dist($id_souscription_campagne)
-{
+function formulaires_souscription_verifier_dist($id_souscription_campagne) {
   $campagne = _request('id_souscription_campagne');
 
   $erreurs = formulaires_editer_objet_verifier('souscription', 'new',
@@ -131,7 +132,7 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne)
   if(!$id_souscription_campagne || intval($id_souscription_campagne) != intval($campagne)) {
       $erreurs['message_erreur'] = "Campagne invalide";
   }
-    
+
   /* La campagne doit être valide (définie dans la base) et doit
    * accepter les dons. */
   $type = sql_getfetsel("type_objectif",
@@ -148,7 +149,7 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne)
 
 
   if(_request('recu_fiscal') || $type == "adhesion") {
-    foreach(array('prenom', 'nom', 'adresse', 'code_postal', 'ville') as $obligatoire) {
+    foreach(array('prenom', 'nom', 'adresse', 'code_postal', 'ville', 'pays') as $obligatoire) {
       if(!_request($obligatoire)) {
         if($type == "adhesion") {
           $erreurs[$obligatoire] = "Ce champ est obligatoire pour les adhésions";
@@ -163,9 +164,41 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne)
   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('pays')) {
+    $ret = sql_select('nom', 'spip_pays', "code='${e}'");
+
+    if(sql_count($ret) != 1)
+      $erreurs['pays'] = "Pays invalide";
+
+    /* Le code postal n'est vérifié que si on est dans le cas de la France */
+    elseif($e = _request('pays') AND $e == "FR") {
+      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('telephone') AND !preg_match("/^[0-9\+ \.]+$/", $e)) {
+    $erreurs['telephone'] = "Numéro de téléphone incorrect";
+  }
+
+
+  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";
+    }
+  }
+
+  if(count($erreurs) > 0) {
+    $erreurs['message_erreur'] = "Le formulaire contient des erreurs";
   }
 
   return $erreurs;
@@ -193,9 +226,7 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne)
  * @return array
  *     Retours des traitements
  */
-function formulaires_souscription_traiter_dist($id_souscription_campagne)
-{
-
+function formulaires_souscription_traiter_dist($id_souscription_campagne) {
   $lier_trad=0;
   $config_fonc='';
   $row=array();
@@ -211,19 +242,19 @@ function formulaires_souscription_traiter_dist($id_souscription_campagne)
                                           $hidden);
 
   $redirect = "";
-  $row = sql_fetsel("transaction_hash,id_transaction", /* $select */
-                    "spip_transactions LEFT JOIN spip_souscriptions USING(id_transaction)", /* $from */
-                    "id_souscription=".$ret['id_souscription']); /* $where */
-
-  print_r($row);
+  $row = sql_fetsel("transaction_hash,id_transaction",
+                    "spip_transactions LEFT JOIN spip_souscriptions USING(id_transaction)",
+                    "id_souscription=".$ret['id_souscription']);
 
   if(!$row) {
+    spip_log(sprintf("Erreur lors de la création de la transaction liée à la souscription [%s].", $ret['id_souscription']), "souscription");
     $ret['message_erreur'] = "Echec creation de la transaction";
   }
   else {
+    spip_log(sprintf("La souscription [%s], associée à la transaction [%s] a bien été crée.", $ret['id_souscription'], $row['id_transaction']), "souscription");
     $hash = $row['transaction_hash'];
     $id_transaction = $row['id_transaction'];
-    $redirect = generer_url_public("payer", "id_transaction=$id_transaction&transaction_hash=$hash", false, false);
+    $redirect = generer_url_public("payer-acte", "id_transaction=$id_transaction&transaction_hash=$hash", false, false);
     $ret['redirect'] = $redirect;
   }