Changeset 90429 in spip-zone

Timestamp:

Jun 22, 2015, 1:02:15 PM (4 years ago)

Author:

rastapopoulos@…

Message:

Les dérogations pour des dates de fin différentes ne doivent se faire que s'il s'agit de renouvellements automatiques : pas juste car CB ou SEPA. En effet, on peut payer CB ou SEPA aussi pour des paiements uniques, sans renouvellement. Donc il faut tester s'il y a abo_uid. Au passage, on déplace toute la partie Commande/Bank? dans une fonction dédiée à part, comme ça on peut la réutiliser pour d'autres choses, et notamment le renouvellement ! En effet, pour l'instant ces dates de fin dérogatoires ne sont changées que pour l'initialisation, lorsqu'on crée un nouvel abonnement. Sauf que si on renouvèle un abonnement existant avec renouvellement auto, il faudrait aussi changer cette fin (ou bien on peut choisir de faire un nouvel abonnement obligatoirement peut-être).

Location:

_plugins_/abonnements/trunk

Files:

• inc/abonnements.php (modified) (2 diffs)
• paquet.xml (modified) (1 diff)

_plugins_/abonnements/trunk/inc/abonnements.php

@@ -43 +43 @@
         $modifs['date_fin'] = $modifs['date_echeance'];
         
-        // Mais s'il y a le plugin Commandes et Bank et qu'on trouve commande et transaction
-        if (
-                _DIR_PLUGIN_COMMANDES
-                and _DIR_PLUGIN_BANK
-                and include_spip('action/editer_liens')
-                and $lien_commande = objet_trouver_liens(array('commande' => '*'), array('abonnement' => $abonnement['id_abonnement']))
-                and is_array($lien_commande)
-                // On prend juste la première commande qu'on trouve
-                and $id_commande = intval($lien_commande[0]['id_commande'])
-                // On cherche le dernier paiement bien payé pour cette commande
-                and $transaction = sql_fetsel(
-                        '*', 'spip_transactions', array('id_commande = '.$id_commande, 'statut = "ok"')
-                )
-        ) {
-                // On a trouvé la transaction qui a activé la commande qui a activé l'abonnement
-                // Si on détecte un prélèvement SEPA, on annule la date de fin !
-                if ($refcb = $transaction['refcb'] and strpos($refcb, 'SEPA') === 0) {
-                        $modifs['date_fin'] = '0000-00-00 00:00:00';
-                }
-                // Si ya une fin de validité de carte bleue on en déduit une fin d'abonnement !
-                elseif ($validite = $transaction['validite']) {
-                        include_spip('inc/bank');
-                        list($year, $month) = explode('-', $validite);
-                        $modifs['date_fin'] = bank_date_fin_mois($year, $month);
-                }
+        // Mais si c'est un renouvellement auto avec Commandes et Bank
+        if ($date_fin = abonnements_bank_date_fin($abonnement['id_abonnnement'])) {
+                $modifs['date_fin'] = $date_fin;
         }
         
@@ -81 +59 @@
 }
 
+/**
+ * Trouver la date de fin d'un renouvellement automatique éventuel
+ * 
+ * @param int $id_abonnement
+ *              Identifiant de l'abonnement dont on veut trouver la date de fin
+ * @param int $id_commande
+ *              Possibilité de donner la commande pour éviter une requête SQL
+ * @return bool|datetime
+ *              Retourne la date de fin du renouvellement si on trouve, sinon false pour ne rien faire
+ **/
+function abonnements_bank_date_fin($id_abonnement, $id_commande=0){
+        $date_fin = false;
+        
+        // On teste si on trouve un renouvellement auto
+        if (
+                _DIR_PLUGIN_COMMANDES
+                and _DIR_PLUGIN_BANK
+                and (
+                        // Soit on a déjà une commande sous la main
+                        (
+                                $id_commande = intval($id_commande)
+                                and $id_commande > 0
+                        )
+                        // Soit on va chercher une commande liée à l'abonnement
+                        or
+                        (
+                                include_spip('action/editer_liens')
+                                and $lien_commande = objet_trouver_liens(array('commande' => '*'), array('abonnement' => $id_abonnement))
+                                and is_array($lien_commande)
+                                // On prend juste la première commande qu'on trouve
+                                and $id_commande = intval($lien_commande[0]['id_commande'])
+                        )
+                )
+                // On cherche un paiement bien payé pour cette commande
+                and $transaction = sql_fetsel(
+                        '*', 'spip_transactions', array('id_commande = '.$id_commande, 'statut = "ok"')
+                )
+                // Et que c'est un renouvellement auto !
+                and $transaction['abo_uid']
+        ) {
+                // On a trouvé la transaction qui a activé la commande qui a activé l'abonnement
+                // Si on détecte un prélèvement SEPA, on annule la date de fin !
+                if ($refcb = $transaction['refcb'] and strpos($refcb, 'SEPA') === 0) {
+                        $date_fin = '0000-00-00 00:00:00';
+                }
+                // Si ya une fin de validité de carte bleue on en déduit une fin d'abonnement !
+                elseif ($validite = $transaction['validite']) {
+                        include_spip('inc/bank');
+                        list($year, $month) = explode('-', $validite);
+                        $date_fin = bank_date_fin_mois($year, $month);
+                }
+        }
+        
+        return $date_fin;
+}
 
 /*

_plugins_/abonnements/trunk/paquet.xml

@@ -2 +2 @@
         prefix="abonnements"
         categorie="auteur"
-        version="3.0.3"
+        version="3.0.4"
         schema="2.2.0"
         etat="dev"