Correction de la validité du montant.
[spip_souscriptions.git] / formulaires / souscription.php
index 6d269c5..f288fa2 100644 (file)
@@ -15,32 +15,6 @@ 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é
- *
- * @param int|string $id_souscription
- *     Identifiant du souscription. 'new' pour un nouveau souscription.
- * @param string $retour
- *     URL de redirection après le traitement
- * @param int $lier_trad
- *     Identifiant éventuel d'un souscription source d'une traduction
- * @param string $config_fonc
- *     Nom de la fonction ajoutant des configurations particulières au formulaire
- * @param array $row
- *     Valeurs de la ligne SQL du souscription, si connu
- * @param string $hidden
- *     Contenu HTML ajouté en même temps que les champs cachés du formulaire.
- * @return string
- *     Hash du formulaire
- */
-function formulaires_souscription_identifier_dist($id_souscription = 'new',
-                                                  $retour = '',
-                                                  $lier_trad = 0,
-                                                  $config_fonc = '',
-                                                  $row = array(),
-                                                  $hidden = ''){
-       return serialize(array(intval($id_souscription)));
-}
 
 /**
  * Chargement du formulaire d'édition de souscription
@@ -49,18 +23,8 @@ function formulaires_souscription_identifier_dist($id_souscription = 'new',
  *
  * @uses formulaires_editer_objet_charger()
  *
- * @param int|string $id_souscription
- *     Identifiant du souscription. 'new' pour un nouveau souscription.
- * @param string $retour
- *     URL de redirection après le traitement
- * @param int $lier_trad
- *     Identifiant éventuel d'un souscription source d'une traduction
- * @param string $config_fonc
- *     Nom de la fonction ajoutant des configurations particulières au formulaire
- * @param array $row
- *     Valeurs de la ligne SQL du souscription, si connu
- * @param string $hidden
- *     Contenu HTML ajouté en même temps que les champs cachés du formulaire.
+ * @param int$id_souscription_campagne
+ *     Identifiant de la campagne de souscription
  * @return array
  *     Environnement du formulaire
  */
@@ -76,7 +40,7 @@ function formulaires_souscription_charger_dist($id_souscription_campagne){
 
        $type = $campagne['type_objectif'];
 
-       $recu_fiscal = "";
+       $recu_fiscal = "off";
        if ($type=="adhesion")
                $recu_fiscal = "on";
 
@@ -112,56 +76,34 @@ function formulaires_souscription_charger_dist($id_souscription_campagne){
        );
 }
 
+
 /**
- * Vérifications du formulaire d'édition de souscription
+ * Chargement du formulaire d'édition de souscription
  *
- * Vérifier les champs postés et signaler d'éventuelles erreurs
+ * Déclarer les champs postés et y intégrer les valeurs par défaut
  *
- * @uses formulaires_editer_objet_verifier()
+ * @uses formulaires_editer_objet_charger()
  *
- * @param int|string $id_souscription
- *     Identifiant du souscription. 'new' pour un nouveau souscription.
- * @param string $retour
- *     URL de redirection après le traitement
- * @param int $lier_trad
- *     Identifiant éventuel d'un souscription source d'une traduction
- * @param string $config_fonc
- *     Nom de la fonction ajoutant des configurations particulières au formulaire
- * @param array $row
- *     Valeurs de la ligne SQL du souscription, si connu
- * @param string $hidden
- *     Contenu HTML ajouté en même temps que les champs cachés du formulaire.
+ * @param int$id_souscription_campagne
+ *     Identifiant de la campagne de souscription
  * @return array
  *     Tableau des erreurs
  */
 function formulaires_souscription_verifier_dist($id_souscription_campagne){
-       $campagne = _request('id_souscription_campagne');
 
-       $erreurs = formulaires_editer_objet_verifier('souscription', 'new',
-               array('courriel',
-                       'montant',
-                       'id_souscription_campagne'));
-
-       if (!$id_souscription_campagne || intval($id_souscription_campagne)!=intval($campagne)){
-               $erreurs['message_erreur'] = "Campagne invalide";
-       }
+       $erreurs = formulaires_editer_objet_verifier('souscription', 'new', array('courriel','montant'));
 
        $campagne = sql_fetsel(array("type_objectif", "configuration_specifique", "type_saisie", "montants"),
-               "spip_souscription_campagnes", "id_souscription_campagne=$id_souscription_campagne");
+               "spip_souscription_campagnes", "id_souscription_campagne=".intval($id_souscription_campagne));
 
        $type_campagne = $campagne['type_objectif'];
 
-       /* Le champ 'type' (hidden) doit être le même que celui défini dans
-        * la campagne. */
-       if (_request('type_souscription')!=$type_campagne)
-               $erreurs['message_erreur'] = "Type de souscription invalide : " . _request('type_souscription');
-
        if (!verifier_campagne($id_souscription_campagne, $type_campagne)){
                $erreurs['message_erreur'] = "La campagne à laquelle est associée cette souscription est invalide";
        }
 
 
-       if (_request('recu_fiscal') || $type_campagne=="adhesion"){
+       if (_request('recu_fiscal')==="on" || $type_campagne=="adhesion"){
                foreach (array('prenom', 'nom', 'adresse', 'code_postal', 'ville', 'pays') as $obligatoire){
                        if (!_request($obligatoire)){
                                if ($type_campagne=="adhesion"){
@@ -204,18 +146,18 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne){
                        $erreurs['montant'] = "Montant invalide";
                else {
                        if ($campagne['configuration_specifique']!=='on'){
-                               $montant_type = lire_config("souscription/{$type}_type_saisie", 'input');
-                               $montant_datas = lire_config("souscription/${type}_montants", array());
+                               $montant_type = lire_config("souscription/{$type_campagne}_type_saisie", 'input');
+                               $montant_datas = lire_config("souscription/{$type_campagne}_montants", array());
                        } else {
                                $montant_type = $campagne['type_saisie'];
-                               $montant_datas = montants_str2array($campagne['montants']);
+                               $montant_datas = $campagne['montants'];
                        }
 
                        /* 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 (($montant_type!="input") AND !array_key_exists($e, $montant_datas))
-                               $erreurs['montant'] = "Le montant spécifié est invalide" . var_export($campagne, true);
+                       if (($montant_type!="input") AND !array_key_exists($e, montants_str2array($montant_datas)))
+                          $erreurs['montant'] = "Le montant spécifié est invalide";
                }
        }
 
@@ -227,24 +169,14 @@ function formulaires_souscription_verifier_dist($id_souscription_campagne){
 }
 
 /**
- * Traitement du formulaire d'édition de souscription
+ * Chargement du formulaire d'édition de souscription
  *
- * Traiter les champs postés
+ * Déclarer les champs postés et y intégrer les valeurs par défaut
  *
- * @uses formulaires_editer_objet_traiter()
+ * @uses formulaires_editer_objet_charger()
  *
- * @param int|string $id_souscription
- *     Identifiant du souscription. 'new' pour un nouveau souscription.
- * @param string $retour
- *     URL de redirection après le traitement
- * @param int $lier_trad
- *     Identifiant éventuel d'un souscription source d'une traduction
- * @param string $config_fonc
- *     Nom de la fonction ajoutant des configurations particulières au formulaire
- * @param array $row
- *     Valeurs de la ligne SQL du souscription, si connu
- * @param string $hidden
- *     Contenu HTML ajouté en même temps que les champs cachés du formulaire.
+ * @param int$id_souscription_campagne
+ *     Identifiant de la campagne de souscription
  * @return array
  *     Retours des traitements
  */
@@ -253,32 +185,56 @@ function formulaires_souscription_traiter_dist($id_souscription_campagne){
        $config_fonc = '';
        $row = array();
        $hidden = '';
+       $retour = '';
+       $ret = array();
 
-       $ret = formulaires_editer_objet_traiter('souscription',
-               'new',
-               '',
-               $lier_trad,
-               $retour,
-               $config_fonc,
-               $row,
-               $hidden);
-
-       $redirect = "";
-       $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-acte", "id_transaction=$id_transaction&transaction_hash=$hash", false, false);
-               $ret['redirect'] = $redirect;
+       $campagne = sql_fetsel(array("type_objectif", "configuration_specifique", "type_saisie", "montants"),
+               "spip_souscription_campagnes", "id_souscription_campagne=".intval($id_souscription_campagne));
+       set_request("id_souscription_campagne",$id_souscription_campagne);
+       set_request('type_souscription',$campagne['type_objectif']);
+
+       // generer la transaction et l'associer a la souscription
+       $inserer_transaction = charger_fonction('inserer_transaction', 'bank');
+       $montant = _request('montant');
+       $id_auteur = (isset($GLOBALS['visiteur_session']['id_auteur'])?$GLOBALS['visiteur_session']['id_auteur']:0);
+       $id_transaction = $inserer_transaction($montant,
+               $montant, /* montant_ht */
+               $id_auteur, /* id_auteur */
+               '', /* auteur_id */
+               _request('courriel'));
+
+       if (!$id_transaction){
+               $ret['message_erreur'] = "Erreur technique : impossible de preparer la transaction..."; /* FIXME: à rendre traduisible. */
+       }
+       else {
+
+               set_request("id_transaction",$id_transaction);
+
+               $ret = formulaires_editer_objet_traiter('souscription',
+                       'new',
+                       '',
+                       $lier_trad,
+                       $retour,
+                       $config_fonc,
+                       $row,
+                       $hidden);
+
+               $redirect = "";
+               $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-acte", "id_transaction=$id_transaction&transaction_hash=$hash", false, false);
+                       $ret['redirect'] = $redirect;
+               }
        }
-
        return $ret;
 }