From: Olivier Tétard Date: Mon, 18 Mar 2013 09:17:44 +0000 (+0100) Subject: Nettoyage X-Git-Url: http://olivier.miskin.fr/git/?a=commitdiff_plain;h=aa969a4df110bd0e5a21646cd5f59b694ef601b1;p=spip_souscriptions.git Nettoyage --- diff --git a/formulaires/souscription.php b/formulaires/souscription.php index 0ee265d..7335428 100644 --- a/formulaires/souscription.php +++ b/formulaires/souscription.php @@ -38,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))); } @@ -116,8 +115,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', @@ -132,7 +130,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", @@ -207,9 +205,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(); @@ -225,16 +221,16 @@ 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); diff --git a/souscription_autorisations.php b/souscription_autorisations.php index e234789..c71373d 100644 --- a/souscription_autorisations.php +++ b/souscription_autorisations.php @@ -34,7 +34,6 @@ function autoriser_souscriptions_menu_dist($faire, $type, $id, $qui, $opt){ * Autorisation de voir le bouton d'accès rapide de création (souscription) **/ function autoriser_souscriptioncreer_menu_dist($faire, $type, $id, $qui, $opt){ - /* return autoriser('creer', 'souscription', '', $qui, $opt); */ return false; } diff --git a/souscription_fonctions.php b/souscription_fonctions.php deleted file mode 100644 index c3be168..0000000 --- a/souscription_fonctions.php +++ /dev/null @@ -1,15 +0,0 @@ - diff --git a/souscription_options.php b/souscription_options.php deleted file mode 100644 index 05911bb..0000000 --- a/souscription_options.php +++ /dev/null @@ -1,15 +0,0 @@ -