Handles canceled payment transactions.
[platal.git] / modules / payment.php
index 543eba1..4fa0641 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /***************************************************************************
- *  Copyright (C) 2003-2010 Polytechnique.org                              *
+ *  Copyright (C) 2003-2014 Polytechnique.org                              *
  *  http://opensource.polytechnique.org/                                   *
  *                                                                         *
  *  This program is free software; you can redistribute it and/or modify   *
  ***************************************************************************/
 
 /* sort en affichant une erreur */
-function cb_erreur($text) {
+function cb_erreur($text, $conf_title="") {
     global $globals;
-    echo "Error.\n";
+    echo "Error: ".$text."\n";
     $mymail = new PlMailer();
     $mymail->addTo($globals->money->email);
     $mymail->setFrom("webmaster@" . $globals->mail->domain);
     $mymail->setSubject("erreur lors d'un télépaiement (CyberPaiement)");
-    $mymail->setTxtBody("\n\n".var_export($_REQUEST,true));
+    $content = "raison de l'erreur : " . $text . "\n";
+    if ($conf_title != "") {
+        $content = $content."paiement : ".$conf_title."\n";
+    }
+    $content = $content . "dump de REQUEST :\n" . var_export($_REQUEST, true);
+    $mymail->setTxtBody($content);
     $mymail->send();
     echo "Notification sent.\n";
     exit;
@@ -45,7 +50,10 @@ function paypal_erreur($text, $send=true)
     $mymail->addTo($globals->money->email);
     $mymail->setFrom("webmaster@" . $globals->mail->domain);
     $mymail->setSubject("erreur lors d'un télépaiement (PayPal)");
-    $mymail->setTxtBody("\n\n".var_export($_REQUEST,true));
+    $mymail->setTxtBody("raison de l'erreur : ".$text."\n".
+                        "paiement : $conf_title \n\n".
+                        "dump de REQUEST :\n".
+                        var_export($_REQUEST,true));
     $mymail->send();
 
     Platal::page()->trigError($text);
@@ -55,23 +63,23 @@ function paypal_erreur($text, $send=true)
 function luhn($nombre) {
     $s = strrev($nombre);
     $sum = 0;
-    for ($i = 0; $i < strlen($s); $i++) {
+    for ($i = 0; $i < strlen($s); ++$i) {
     $dgt = $s{$i};
-        $sum += ($i % 2) ? (2*$dgt) % 9 : $dgt;
+        $sum += ($i % 2) ? (2 * $dgt) % 9 : $dgt;
     }
     return $sum % 10;
 }
 
 /* calcule la clé d'acceptation a partir de 5 champs */
-function cle_accept($d1,$d2,$d3,$d4,$d5)
+function cle_accept($d1, $d2, $d3, $d4, $d5)
 {
-    $m1 = luhn($d1.$d5);
-    $m2 = luhn($d2.$d5);
-    $m3 = luhn($d3.$d5);
-    $m4 = luhn($d4.$d5);
+    $m1 = luhn($d1 . $d5);
+    $m2 = luhn($d2 . $d5);
+    $m3 = luhn($d3 . $d5);
+    $m4 = luhn($d4 . $d5);
     $n = $m1 + $m2 + $m3 + $m4;
-    $alpha = "ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
-    return $alpha{$n-1}.$m1.$m2.$m3.$m4;
+    $alpha = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789';
+    return $alpha{$n-1} . $m1 . $m2 . $m3 . $m4;
 }
 
 /* decode the comment */
@@ -88,16 +96,16 @@ function comment_decode($comment) {
 function check_rib($rib)
 {
     if(strlen($rib) != 23) return false;
-    
+
     // extract fields
     $rib = strtr(strtoupper($rib),'ABCDEFGHIJKLMNOPQRSTUVWXYZ','12345678912345678923456789');
     $bank    = substr($rib,0,5);
     $counter = substr($rib,5,5);
     $account = substr($rib,10,11);
     $key     = substr($rib,21,2);
-    
+
     // check
-    return 0 == fmod(89 * $bank + 15 * $counter + 3 * $account + $key, 97);
+    return (0 == fmod(89 * $bank + 15 * $counter + 3 * $account + $key, 97));
 }
 
 class PaymentModule extends PLModule
@@ -105,193 +113,133 @@ class PaymentModule extends PLModule
     function handlers()
     {
         return array(
-            'payment'                    => $this->make_hook('payment',       AUTH_MDP,    'payment'),
-            'payment/cyber_return'       => $this->make_hook('cyber_return',  AUTH_PUBLIC, 'user', NO_HTTPS),
-            'payment/cyber2_return'      => $this->make_hook('cyber2_return', AUTH_PUBLIC, 'user', NO_HTTPS),
-            'payment/paypal_return'      => $this->make_hook('paypal_return', AUTH_PUBLIC, 'user', NO_HTTPS),
-            '%grp/paiement'              => $this->make_hook('xnet_payment',  AUTH_MDP),
-            '%grp/payment'               => $this->make_hook('xnet_payment',  AUTH_MDP),
-            '%grp/payment/cyber_return'  => $this->make_hook('cyber_return',  AUTH_PUBLIC, 'user', NO_HTTPS),
-            '%grp/payment/cyber2_return' => $this->make_hook('cyber2_return', AUTH_PUBLIC, 'user', NO_HTTPS),
-            '%grp/payment/paypal_return' => $this->make_hook('paypal_return', AUTH_PUBLIC, 'user', NO_HTTPS),
-            'admin/payments'             => $this->make_hook('admin',         AUTH_MDP,    'admin'),
-            'admin/payments/methods'     => $this->make_hook('adm_methods',   AUTH_MDP,    'admin'),
-            'admin/payments/transactions'=> $this->make_hook('adm_transactions', AUTH_MDP, 'admin'),
-            'admin/reconcile'            => $this->make_hook('adm_reconcile', AUTH_MDP,    'admin'),
-            'admin/reconcile/importlogs' => $this->make_hook('adm_importlogs', AUTH_MDP,    'admin'),
-            'admin/reconcile/transfers'  => $this->make_hook('adm_transfers', AUTH_MDP,    'admin'),
-            'admin/reconcile/bankaccounts'=> $this->make_hook('adm_bankaccounts', AUTH_MDP, 'admin'),
+            'payment'                      => $this->make_hook('payment',          AUTH_PUBLIC, 'user'),
+            'payment/cyber2_return'        => $this->make_hook('cyber2_return',    AUTH_PUBLIC, 'user', NO_HTTPS),
+            'payment/paypal_return'        => $this->make_hook('paypal_return',    AUTH_PUBLIC, 'user', NO_HTTPS),
+            '%grp/paiement'                => $this->make_hook('xnet_payment',     AUTH_PUBLIC, 'user'),
+            '%grp/payment'                 => $this->make_hook('xnet_payment',     AUTH_PUBLIC, 'user'),
+            '%grp/payment/csv'             => $this->make_hook('payment_csv',      AUTH_PASSWD, 'groupadmin'),
+            '%grp/payment/cyber2_return'   => $this->make_hook('cyber2_return',    AUTH_PUBLIC, 'user', NO_HTTPS),
+            '%grp/payment/paypal_return'   => $this->make_hook('paypal_return',    AUTH_PUBLIC, 'user', NO_HTTPS),
+            'admin/payments'               => $this->make_hook('admin',            AUTH_PASSWD, 'admin'),
+            'admin/payments/methods'       => $this->make_hook('adm_methods',      AUTH_PASSWD, 'admin'),
+            'admin/payments/transactions'  => $this->make_hook('adm_transactions', AUTH_PASSWD, 'admin'),
+            'admin/reconcile'              => $this->make_hook('adm_reconcile',    AUTH_PASSWD, 'admin'),
+            'admin/reconcile/importlogs'   => $this->make_hook('adm_importlogs',   AUTH_PASSWD, 'admin'),
+            'admin/reconcile/transfers'    => $this->make_hook('adm_transfers',    AUTH_PASSWD, 'admin'),
+            'admin/payments/bankaccounts'  => $this->make_hook('adm_bankaccounts', AUTH_PASSWD, 'admin'),
         );
     }
 
-    function handler_payment(&$page, $ref = -1)
+    function handler_payment($page, $ref = -1)
     {
-        global $globals;
-
+        $page->changeTpl('payment/payment.tpl');
+        $page->setTitle('Télépaiement');
         $this->load('money.inc.php');
 
-        if (!empty($GLOBALS['IS_XNET_SITE'])) {
-            if (!$globals->asso('id')) {
-                return PL_NOT_FOUND;
-            }
-            $res = XDB::query("SELECT asso_id
-                                 FROM payments
-                                WHERE asso_id = {?} AND id = {?}",
-                              $globals->asso('id'), $ref);
-            if (!$res->numRows()) {
-                return PL_FORBIDDEN;
-            }
-        }
-        $page->changeTpl('payment/index.tpl');
-        $page->setTitle('Télépaiements');
-
-        // initialisation
-        $op   = Env::v('op', 'select');
         $meth = new PayMethod(Env::i('methode', -1));
         $pay  = new Payment($ref);
 
-        if($pay->flags->hasflag('old')){
-            $page->trigError("La transaction selectionnée est périmée.");
-            $pay = new Payment();
-        }
-        $val = Env::v('montant') != 0 ? Env::v('montant') : $pay->amount_def;
-
-        if (($e = $pay->check($val)) !== true) {
-            $page->trigError($e);
-        }
-
-        if ($op=='submit') {
-            $pay->init($val, $meth);
-            $pay->prepareform($pay);
+        if (!$pay->flags->hasflag('public') && (!S::user() || !S::logged())) {
+            $page->kill("Vous n'avez pas les permissions nécessaires pour accéder à cette page.");
         } else {
-            $res = XDB::iterator("SELECT  timestamp, amount
-                                    FROM  payment_transactions
-                                   WHERE  uid = {?} AND ref = {?}
-                                ORDER BY  timestamp DESC",
-                                 S::v('uid', -1), $ref);
-
-            if ($res->total()) $page->assign('transactions', $res);
+            $page->assign('public', true);
         }
 
-        $val = floor($val).".".substr(floor(($val - floor($val))*100+100),1);
-        $page->assign('montant',$val);
-        $page->assign('comment',Env::v('comment'));
-
-        $page->assign('meth', $meth);
-        $page->assign('pay',  $pay);
-        $page->assign('evtlink', $pay->event());
-        $page->assign('sex', S::user()->isFemale());
-    }
-
-    function handler_cyber_return(&$page, $uid = null)
-    {
-        /* reference banque (numero de transaction) */
-        $champ901 = Env::s('CHAMP901');
-        /* cle d'acceptation */
-        $champ905 = Env::s('CHAMP905');
-        /* code retour */
-        $champ906 = Env::s('CHAMP906');
-        /* email renvoye par la banque */
-        $champ104 = Env::s('CHAMP104');
-        /* reference complete de la commande */
-        $champ200 = Env::s('CHAMP200');
-        /* montant de la transaction */
-        $champ201 = Env::s('CHAMP201');
-        /* devise */
-        $champ202 = Env::s('CHAMP202');
-        $montant = "$champ201 $champ202";
-
-        /* on extrait les informations sur l'utilisateur */
-        $user = User::get($uid);
-        if (!$user) {
-            cb_erreur("uid invalide");
+        if ($pay->flags->hasflag('old')) {
+            $page->kill('La transaction selectionnée est périmée.');
         }
 
-
-        /* on extrait la reference de la commande */
-        if (!ereg('-xorg-([0-9]+)$', $champ200, $matches)) {
-            cb_erreur("référence de commande invalide");
+        if (Env::has('montant')) {
+            $pay->amount_def = Env::v('montant');
         }
+        $val = (Post::v('amount') != 0) ? Post::v('amount') : $pay->amount_def;
 
-        echo ($ref = $matches[1]);
-        $res = XDB::query("SELECT  mail, text, confirmation
-                             FROM  payments
-                            WHERE  id={?}", $ref);
-        if (!list($conf_mail, $conf_title, $conf_text) = $res->fetchOneRow()) {
-            cb_erreur("référence de commande inconnue");
+        if (($error = $pay->check($val)) !== true) {
+            $page->trigError($error);
         }
 
-        /* on extrait le code de retour */
-        if ($champ906 != "0000") {
-            $res = XDB::query('SELECT  rcb.text, c.id, c.text
-                                 FROM  payment_codeRCB AS rcb
-                            LEFT JOIN  payment_codeC   AS c ON (rcb.codeC = c.id)
-                                WHERE  rcb.id = {?}', $champ906);
-            if (list($rcb_text, $c_id, $c_text) = $res->fetchOneRow()) {
-                cb_erreur("erreur lors du paiement : $c_text ($c_id)");
-            } else{
-                cb_erreur("erreur inconnue lors du paiement");
+        if (Post::has('op') && Post::v('op', 'select') == 'submit') {
+            if (S::logged()) {
+                $user = S::user();
+            } else {
+                $user = User::getSilent(Post::t('login'));
             }
-        }
 
-        /* on fait l'insertion en base de donnees */
-        XDB::execute("INSERT INTO  payment_transactions (id, uid, ref, fullref, amount, pkey, comment)
-                           VALUES  ({?}, {?}, {?}, {?}, {?}, {?}, {?})",
-                     $champ901, $user->id(), $ref, $champ200, $montant, $champ905, Env::v('comment'));
+            if (is_null($user)) {
+                $page->trigError("L'identifiant est erroné.");
+                $page->assign('login_error', true);
+                $page->assign('login', Post::t('login'));
+            } else {
+                $pay->init($val, $meth);
+                $pay->prepareform($user);
+                $page->assign('full_name', $user->fullName(true));
+                $page->assign('sex', $user->isFemale());
+            }
+        } elseif (S::logged()) {
+            $res = XDB::iterator('SELECT  ts_confirmed, amount
+                                    FROM  payment_transactions
+                                   WHERE  uid = {?} AND ref = {?}
+                                ORDER BY  ts_confirmed DESC',
+                                 S::v('uid', -1), $pay->id);
 
-        // We check if it is an Xnet payment and then update the related ML.
-        $res = XDB::query('SELECT  eid
-                             FROM  group_events
-                            WHERE  paiement_id = {?}', $ref);
-        if ($eid = $res->fetchOneCell()) {
-            require_once dirname(__FILE__) . '/xnetevents/xnetevents.inc.php';
-            $evt = get_event_detail($eid);
-            subscribe_lists_event($uid, $evt, 1, $montant, true);
-        }
+            if ($res->total()) {
+                $page->assign('transactions', $res);
+            }
 
-        /* on genere le mail de confirmation */
-        $conf_text = str_replace(
-            array('<prenom>', '<nom>', '<promo>', '<montant>', '<salutation>', '<cher>', '<comment>'),
-            array($user->firstName(), $user->lastName(), $user->promo(), $montant,
-                  $user->isFemale() ? 'Chère' : 'Cher', $user->isFemale() ? 'Chère' : 'Cher',
-                  Env::v('comment')), $conf_text);
+            // Only if $id = -1, meaning only for donation the site's association
+            if ($ref == -1) {
+                $biggest_donations = XDB::fetchAllAssoc('SELECT  IF(p.display,
+                                                                    IF(ap.pid IS NOT NULL, CONCAT(a.full_name, \' (\', pd.promo, \')\'), a.full_name),
+                                                                    \'XXXX\') AS name, p.amount, p.ts_confirmed
+                                                           FROM  payment_transactions AS p
+                                                     INNER JOIN  accounts             AS a  ON (a.uid = p.uid)
+                                                      LEFT JOIN  account_profiles     AS ap ON (a.uid = ap.uid AND FIND_IN_SET(\'owner\', ap.perms))
+                                                      LEFT JOIN  profile_display      AS pd ON (ap.pid = pd.pid)
+                                                          WHERE  p.ref = {?}
+                                                       ORDER BY  LENGTH(p.amount) DESC, p.amount DESC, name
+                                                          LIMIT  10',
+                                                        $pay->id);
+
+                $donations = XDB::fetchAllAssoc('(SELECT  SUM(amount) AS amount, YEAR(ts_confirmed) AS year, MONTH(ts_confirmed) AS month, ts_confirmed
+                                                    FROM  payment_transactions
+                                                   WHERE  ref = {?} AND YEAR(ts_confirmed) = YEAR(CURDATE())
+                                                GROUP BY  month)
+                                                 UNION
+                                                 (SELECT  SUM(amount) AS amount, YEAR(ts_confirmed) AS year, 0 AS month, ts_confirmed
+                                                    FROM  payment_transactions
+                                                   WHERE  ref = {?} AND YEAR(ts_confirmed) < YEAR(CURDATE())
+                                                GROUP BY  year)
+                                                ORDER BY  year DESC, month DESC',
+                                                $pay->id, $pay->id);
+
+                $page->assign('biggest_donations', $biggest_donations);
+                $page->assign('donations', $donations);
+                $page->assign('donation', true);
+            }
+        }
 
-        global $globals;
-        $mymail = new PlMailer();
-        $mymail->setFrom($conf_mail);
-        $mymail->addCc($conf_mail);
-        $mymail->setSubject($conf_title);
-        $mymail->setWikiBody($conf_text);
-        $mymail->sendTo($user);
+        $val = floor($val * 100) / 100;
+        $page->assign('amount', $val);
+        $page->assign('comment', Env::v('comment'));
 
-        /* on envoie les details de la transaction à telepaiement@ */
-        $mymail = new PlMailer();
-        $mymail->setFrom("webmaster@" . $globals->mail->domain);
-        $mymail->addTo($globals->money->email);
-        $mymail->setSubject($conf_title);
-        $msg = 'utilisateur : ' . $user->login() . ' (' . $user->id() . ')' . "\n" .
-               'mail : ' . $user->forlifeEmail() . "\n\n" .
-               "paiement : $conf_title ($conf_mail)\n".
-               "reference : $champ200\n".
-               "montant : $montant\n\n".
-               "dump de REQUEST:\n".
-               var_export($_REQUEST,true);
-        $mymail->setTxtBody($msg);
-        $mymail->send();
-        exit;
+        $page->assign('meth', $meth);
+        $page->assign('pay', $pay);
+        $page->assign('evtlink', $pay->event());
     }
 
-    function handler_cyber2_return(&$page, $uid = null)
+    function handler_cyber2_return($page, $uid = null)
     {
         global $globals, $platal;
 
         /* on vérifie la signature */
         $vads_params = array();
         foreach($_REQUEST as $key => $value)
-            if(substr($key,0,5) == "vads_")
+            if(substr($key,0,5) == 'vads_') {
                 $vads_params[$key] = $value;
+            }
         ksort($vads_params);
-        $signature = sha1(join('+',$vads_params).'+'.$globals->money->cyperplus_key);
+        $signature = sha1(join('+', $vads_params) . '+' . $globals->money->cyperplus_key);
         //if($signature != Env::v('signature')) {
         //    cb_erreur("signature invalide");
         //}
@@ -303,36 +251,35 @@ class PaymentModule extends PLModule
         }
 
         /* on extrait la reference de la commande */
-        if (!ereg('-([0-9]+)$', Env::v('vads_order_id'), $matches)) {
+        if (!preg_match('/-([0-9]+)$/', Env::v('vads_order_id'), $matches)) {
             cb_erreur("référence de commande invalide");
         }
 
         $ref = $matches[1];
-        $res = XDB::query("SELECT  mail, text, confirmation
+        $res = XDB::query('SELECT  mail, text, confirmation
                              FROM  payments
-                            WHERE  id={?}", $ref);
+                            WHERE  id={?}', $ref);
         if ($res->numRows() != 1) {
             cb_erreur("référence de commande inconnue");
         }
         list($conf_mail, $conf_title, $conf_text) = $res->fetchOneRow();
 
         /* on extrait le montant */
-        if (Env::v('vads_currency') != "978") {
+        if (Env::v('vads_currency') != '978') {
             cb_erreur("monnaie autre que l'euro");
         }
-        $amount = ((float)Env::i('vads_amount')) / 100;
-        $montant = sprintf("%.02f EUR", $amount);
+        $montant = ((float)Env::i('vads_amount')) / 100;
 
         /* on extrait le code de retour */
-        if (Env::v('vads_result') != "00") {
-            cb_erreur("erreur lors du paiement : ?? (".Env::v('vads_result').")");
+        if (Env::v('vads_result') != '00') {
+            cb_erreur('erreur lors du paiement : ?? (' . Env::v('vads_result') . ')', $conf_title);
         }
 
         /* on fait l'insertion en base de donnees */
-        XDB::execute("INSERT INTO  payment_transactions (id, uid, ref, fullref, amount, pkey, comment)
-                           VALUES  ({?}, {?}, {?}, {?}, {?}, {?}, {?})",
-                     Env::v('vads_trans_date'), $user->id(), $ref, Env::v('vads_order_id'), $montant, "", Env::v('vads_order_info'));
-        echo "Paiement stored.\n";
+        XDB::execute('INSERT INTO  payment_transactions (id, method_id, uid, ref, fullref, ts_confirmed, amount, pkey, comment, status, display)
+                           VALUES  ({?}, 2, {?}, {?}, {?}, NOW(), {?}, {?}, {?}, "confirmed", {?})',
+                     Env::v('vads_trans_date'), $user->id(), $ref, Env::v('vads_order_id'), $montant, '', Env::v('vads_order_info'), Env::i('vads_order_info2'));
+        echo "Payment stored.\n";
 
         // We check if it is an Xnet payment and then update the related ML.
         $res = XDB::query('SELECT  eid, asso_id
@@ -342,7 +289,7 @@ class PaymentModule extends PLModule
             list($eid, $asso_id) = $res->fetchOneRow();
             require_once dirname(__FILE__) . '/xnetevents/xnetevents.inc.php';
             $evt = get_event_detail($eid, false, $asso_id);
-            subscribe_lists_event($user->id(), $evt, 1, $amount, true);
+            subscribe_lists_event($user->id(), $evt['short_name'], 1, $montant, true);
         }
 
         /* on genere le mail de confirmation */
@@ -350,7 +297,7 @@ class PaymentModule extends PLModule
             array('<prenom>', '<nom>', '<promo>', '<montant>', '<salutation>', '<cher>', '<comment>'),
             array($user->firstName(), $user->lastName(), $user->promo(), $montant,
                   $user->isFemale() ? 'Chère' : 'Cher', $user->isFemale() ? 'Chère' : 'Cher',
-                  Env::v('comment')), $conf_text);
+                  Env::v('vads_order_info')), $conf_text);
 
         global $globals;
         $mymail = new PlMailer();
@@ -378,7 +325,7 @@ class PaymentModule extends PLModule
         exit;
     }
 
-    function handler_paypal_return(&$page, $uid = null)
+    function handler_paypal_return($page, $uid = null)
     {
         $page->changeTpl('payment/retour_paypal.tpl');
 
@@ -389,21 +336,23 @@ class PaymentModule extends PLModule
         /* code retour */
         $status = Env::s('st');
         /* raison */
-        $reason = ($status == 'Pending')? Env::s('pending_reason'): Env::s('reason_code');
+        $reason = ($status == 'Pending') ? Env::s('pending_reason') : Env::s('reason_code');
         /* reference complete de la commande */
-        $fullref = Env::s('cm');
+        $fullref = str_replace('%2d','-',Env::s('cm'));
         /* montant de la transaction */
-        $montant_nb = Env::s('amt');
+        $montant = Env::s('amt');
         /* devise */
-        $montant_dev = Env::s('cc');
-        $montant = "$montant_nb $montant_dev";
+        if (Env::s('cc') != 'EUR') {
+            cb_erreur("monnaie autre que l'euro");
+        }
 
         /* on extrait le code de retour */
         if ($status != "Completed") {
-            if ($status)
+            if ($status) {
                 paypal_erreur("erreur lors du paiement : $status - $reason");
-            else
+            } else {
                 paypal_erreur("Paiement annulé", false);
+            }
         }
 
         /* on extrait les informations sur l'utilisateur */
@@ -413,38 +362,39 @@ class PaymentModule extends PLModule
         }
 
         /* on extrait la reference de la commande */
-        if (!ereg('-xorg-([0-9]+)$', $fullref, $matches)) {
+        if (!preg_match('/-xorg-([0-9]+)$/', $fullref, $matches)) {
             paypal_erreur("référence de commande invalide");
         }
 
         $ref = $matches[1];
-        $res = XDB::query("SELECT  mail, text, confirmation
+        $res = XDB::query('SELECT  mail, text, confirmation
                              FROM  payments
-                            WHERE  id = {?}", $ref);
-        if (!list($conf_mail,$conf_title,$conf_text) = $res->fetchOneRow()) {
-            paypal_erreur("référence de commande inconnue");
+                            WHERE  id = {?}', $ref);
+        if (!list($conf_mail, $conf_title, $conf_text) = $res->fetchOneRow()) {
+            paypal_erreur('référence de commande inconnue');
         }
 
         /* on fait l'insertion en base de donnees */
-        XDB::execute("INSERT INTO  payment_transactions (id, uid, ref, fullref, amount, pkey, comment)
-                           VALUES  ({?}, {?}, {?}, {?}, {?}, {?}, {?})",
-                    $no_transaction, $user->id(), $ref, $fullref, $montant, $clef, Env::v('comment'));
+        XDB::execute("INSERT INTO  payment_transactions (id, method_id, uid, ref, fullref, ts_confirmed, amount, pkey, comment, status, display)
+                           VALUES  ({?}, 1, {?}, {?}, {?}, NOW(), {?}, {?}, {?}, 'confirmed', {?})",
+                    $no_transaction, $user->id(), $ref, $fullref, $montant, $clef, Env::v('comment'), Get::i('display'));
 
         // We check if it is an Xnet payment and then update the related ML.
-        $res = XDB::query('SELECT  eid
+        $res = XDB::query('SELECT  eid, asso_id
                              FROM  group_events
                             WHERE  paiement_id = {?}', $ref);
-        if ($eid = $res->fetchOneCell()) {
+        if ($res->numRows() == 1) {
+            list($eid, $asso_id) = $res->fetchOneRow();
             require_once dirname(__FILE__) . '/xnetevents/xnetevents.inc.php';
-            $evt = get_event_detail($eid);
-            subscribe_lists_event($user->id(), $evt, 1, $montant, true);
+            $evt = get_event_detail($eid, false, $asso_id);
+            subscribe_lists_event($user->id(), $evt['short_name'], 1, $montant, true);
         }
 
         /* on genere le mail de confirmation */
-        $conf_text = str_replace(array('<prenom>', '<nom>', '<promo>', '<montant>', '<salutation>', '<cher>'),
+        $conf_text = str_replace(array('<prenom>', '<nom>', '<promo>', '<montant>', '<salutation>', '<cher>', '<comment>'),
                                  array($user->firstName(), $user->lastName(), $user->promo(), $montant,
-                                       $user->isFemale() ? 'Chère' : 'Cher',
-                                       $user->isFemale() ? 'Chère' : 'Cher'), $conf_text);
+                                       $user->isFemale() ? 'Chère' : 'Cher', $user->isFemale() ? 'Chère' : 'Cher',
+                                       Env::v('comment')), $conf_text);
 
         global $globals;
         $mymail = new PlMailer();
@@ -473,22 +423,30 @@ class PaymentModule extends PLModule
         $page->assign('erreur', $erreur);
     }
 
-    function handler_xnet_payment(&$page, $pid = null)
+    function handler_xnet_payment($page, $pid = null)
     {
         global $globals;
 
         $perms = S::v('perms');
-        if (!$perms->hasFlag('groupmember')) {
-            if (is_null($pid)) {
+        if (is_null($pid)) {
+            if (!(S::identified() && $perms->hasFlag('groupadmin'))) {
                 return PL_FORBIDDEN;
             }
-            $res = XDB::query("SELECT  1
-                                 FROM  group_events AS e
-                           INNER JOIN  group_event_participants AS ep ON (ep.eid = e.eid AND e.uid = {?})
-                                WHERE  e.paiement_id = {?} AND e.asso_id = {?}",
-                              S::i('uid'), $pid, $globals->asso('id'));
-            if ($res->numRows() == 0) {
-                return PL_FORBIDDEN;
+        } else {
+            if (!(S::identified() && $perms->hasFlag('groupmember'))) {
+                $res = XDB::query("SELECT  1
+                                     FROM  group_events AS e
+                               INNER JOIN  group_event_participants AS ep ON (ep.eid = e.eid AND ep.uid = {?})
+                                    WHERE  e.paiement_id = {?} AND e.asso_id = {?}",
+                                  S::i('uid'), $pid, $globals->asso('id'));
+                $public = XDB::query("SELECT  1
+                                        FROM  payments     AS p
+                                  INNER JOIN  group_events AS g ON (g.paiement_id = p.id)
+                                       WHERE  g.asso_id = {?} AND p.id = {?} AND FIND_IN_SET('public', p.flags)",
+                                     $globals->asso('id'), $pid);
+                if ($res->numRows() == 0 && $public->numRows() == 0) {
+                    return PL_FORBIDDEN;
+                }
             }
         }
 
@@ -503,33 +461,60 @@ class PaymentModule extends PLModule
                   WHERE  asso_id = {?} AND NOT FIND_IN_SET('old', flags)
                ORDER BY  id DESC", $globals->asso('id'));
         $tit = $res->fetchAllAssoc();
-        $page->assign('titres', $tit);
+        $page->assign('titles', $tit);
 
-
-        // TODO: replug sort.
         $trans = array();
         $event = array();
+        if (may_update()) {
+            static $orders = array('ts_confirmed' => 'p', 'directory_name' => 'a', 'promo' => 'pd', 'comment' => 'p', 'amount' => 'p');
+
+            if (Get::has('order_id') && Get::has('order') && array_key_exists(Get::v('order'), $orders)) {
+                $order_id = Get::i('order_id');
+                $order = Get::v('order');
+                $ordering = ' ORDER BY ' . $orders[$order] . '.' . $order;
+                if (Get::has('order_inv') && Get::i('order_inv') == 1) {
+                    $ordering .= ' DESC';
+                    $page->assign('order_inv', 0);
+                } else {
+                    $page->assign('order_inv', 1);
+                }
+                $page->assign('order_id', $order_id);
+                $page->assign('order', $order);
+                $page->assign('anchor', 'legend_' . $order_id);
+            } else {
+                $order_id = false;
+                $ordering = '';
+                $page->assign('order', false);
+            }
+        } else {
+            $ordering = '';
+            $page->assign('order', false);
+        }
         foreach($tit as $foo) {
             $pid = $foo['id'];
             if (may_update()) {
-                $res = XDB::query('SELECT  t.uid, timestamp AS `date`, t.comment, amount
-                                     FROM  payment_transactions AS t
-                                    WHERE  t.ref = {?}', $pid);
+                $res = XDB::query('SELECT  p.uid, IF(p.ts_confirmed = \'0000-00-00\', 0, p.ts_confirmed) AS date, p.comment, p.amount
+                                     FROM  payment_transactions AS p
+                               INNER JOIN  accounts             AS a  ON (a.uid = p.uid)
+                                LEFT JOIN  account_profiles     AS ap ON (ap.uid = p.uid AND FIND_IN_SET(\'owner\', ap.perms))
+                                LEFT JOIN  profile_display      AS pd ON (ap.pid = pd.pid)
+                                    WHERE  p.ref = {?}' . (($order_id == $pid) ? $ordering : ''),
+                                  $pid);
                 $trans[$pid] = User::getBulkUsersWithUIDs($res->fetchAllAssoc(), 'uid', 'user');
                 $sum = 0;
                 foreach ($trans[$pid] as $i => $t) {
-                    $sum += strtr(substr($t['amount'], 0, strpos($t['amount'], 'EUR')), ',', '.');
-                    $trans[$pid][$i]['amount'] = str_replace('EUR', '€', $t['amount']);
+                    $sum += $t['amount'];
+                    $trans[$pid][$i]['amount'] = $t['amount'];
                 }
-                $trans[$pid][] = array('nom' => 'somme totale',
-                                       'amount' => strtr($sum, '.', ',').' €');
+                $trans[$pid][] = array('limit'  =>  true,
+                                       'amount' => $sum);
             }
-            $res = XDB::iterRow("SELECT e.eid, e.short_name, e.intitule, ep.nb, ei.montant, ep.paid
-                                   FROM group_events AS e
-                              LEFT JOIN group_event_participants AS ep ON (ep.eid = e.eid AND ep.uid = {?})
-                             INNER JOIN group_event_items AS ei ON (ep.eid = ei.eid AND ep.item_id = ei.item_id)
-                                  WHERE e.paiement_id = {?}",
-                                 S::v('uid'), $pid);
+            $res = XDB::iterRow("SELECT  e.eid, e.short_name, e.intitule, ep.nb, ei.montant, ep.paid
+                                   FROM  group_events             AS e
+                              LEFT JOIN  group_event_participants AS ep ON (ep.eid = e.eid AND ep.uid = {?})
+                             INNER JOIN  group_event_items        AS ei ON (ep.eid = ei.eid AND ep.item_id = ei.item_id)
+                                  WHERE  e.paiement_id = {?}",
+                                S::v('uid'), $pid);
             $event[$pid] = array();
             $event[$pid]['paid'] = 0;
             if ($res->total()) {
@@ -543,244 +528,306 @@ class PaymentModule extends PLModule
                     $event[$pid]['paid']      = $paid;
                 }
             }
-            $res = XDB::query("SELECT amount
-                                 FROM payment_transactions AS t
-                                WHERE ref = {?} AND uid = {?}", $pid, S::v('uid'));
-            $montants = $res->fetchColumn();
-
-            foreach ($montants as $m) {
-                $p = strtr(substr($m, 0, strpos($m, 'EUR')), ',', '.');
-                $event[$pid]['paid'] += trim($p);
-            }
+            $res = XDB::query('SELECT  SUM(amount) AS sum_amount
+                                 FROM  payment_transactions
+                                WHERE  ref = {?} AND uid = {?}', $pid, S::v('uid'));
+            $event[$pid]['paid'] = $res->fetchOneCell();
         }
         $page->register_modifier('decode_comment', 'decode_comment');
         $page->assign('trans', $trans);
         $page->assign('event', $event);
     }
 
-    function handler_admin(&$page, $action = 'list', $id = null) {
+    function handler_payment_csv($page, $pid = null)
+    {
+        if (is_null($pid)) {
+            pl_redirect('payment');
+        }
+        if (substr($pid, -4) == '.vcf') {
+            $pid = substr($pid, 0, strlen($pid) - 4);
+        }
+
+        $res = XDB::fetchAllAssoc('SELECT  uid, IF(ts_confirmed = \'0000-00-00\', 0, ts_confirmed) AS date, comment, amount
+                                     FROM  payment_transactions
+                                    WHERE  ref = {?}
+                                 ORDER BY  ts_confirmed',
+                                  $pid);
+        if (is_null($res)) {
+            pl_redirect('payment');
+        }
+
+        $users = User::getBulkUsersWithUIDs($res, 'uid', 'user');
+        $sum = 0;
+
+        pl_cached_content_headers('text/x-csv', 'iso-8859-1', 1);
+        $csv = fopen('php://output', 'w');
+        fputcsv($csv, array('Date', 'Nom', utf8_decode('Prénom'), 'Sexe', 'Promotion', 'Email', 'Commentaire', 'Montant'), ';');
+        foreach ($users as $item) {
+            $user = $item['user'];
+            $sum += $item['amount'];
+            fputcsv($csv, array(format_datetime($item['date'], '%d/%m/%y'), utf8_decode($user->lastName()), utf8_decode($user->firstName()),
+                                ($user->isFemale()) ? 'F' : 'M', $user->promo(), $user->ForlifeEmail(),
+                                utf8_decode($item['comment']), strtr($item['amount'], '.', ',') . ' EUR' ), ';');
+        }
+        fputcsv($csv, array(date('d/m/y'), 'Total', '', '', '' , '', '', strtr($sum, '.', ',') . ' EUR'), ';');
+
+        fclose($csv);
+        exit;
+    }
+
+    function handler_admin($page, $action = 'list', $id = null) {
         $page->setTitle('Administration - Paiements');
         $page->assign('title', 'Gestion des télépaiements');
         $table_editor = new PLTableEditor('admin/payments','payments','id');
-        //$table_editor->add_join_table('payment_transactions','ref',true); => on ne supprime jamais une transaction
         $table_editor->add_sort_field('flags');
         $table_editor->add_sort_field('id', true, true);
         $table_editor->on_delete("UPDATE payments SET flags = 'old' WHERE id = {?}", "Le paiement a été archivé");
-        $table_editor->describe('text','intitulé',true);
-        $table_editor->describe('url','site web',false);
-        $table_editor->describe('amount_def','montant par défaut',false);
-        $table_editor->describe('amount_min','montant minimum',false);
-        $table_editor->describe('amount_max','montant maximum',false);
-        $table_editor->describe('mail','email contact',true);
-        $table_editor->describe('confirmation','message confirmation',false);
-
+        $table_editor->describe('text', 'intitulé', true);
+        $table_editor->describe('url', 'site web', false, true);
+        $table_editor->describe('amount_def', 'montant par défaut', false, true);
+        $table_editor->describe('amount_min', 'montant minimum', false, true);
+        $table_editor->describe('amount_max', 'montant maximum', false, true);
+        $table_editor->describe('mail', 'email contact', true);
+        $table_editor->describe('confirmation', 'message confirmation', false, true);
+        $table_editor->describe('rib_id', 'RIB', false, true);
         // adds a column with the start date of the linked event if there is one
-        $table_editor->add_option_table('group_events','group_events.paiement_id = t.id');
-        $table_editor->add_option_field('group_events.debut', 'related_event', 'évènement', 'timestamp');
+        $table_editor->add_option_table('group_events', 'group_events.paiement_id = t.id');
+        $table_editor->add_option_field('group_events.archive', 'related_event', 'évènement archivé ?', 'tinyint');
+        // adds a column with the linked rib if there is one
+        $table_editor->add_option_table('payment_bankaccounts', 'payment_bankaccounts.id = t.rib_id');
+        $table_editor->add_option_field('payment_bankaccounts.owner', 'linked_rib_owner', 'rib associé', 'varchar');
+        // adds a link to the table of all the transactions
+        $table_editor->addLink('id', "admin/payments/transactions/");
 
         $table_editor->apply($page, $action, $id);
     }
-    
-    function handler_adm_transactions(&$page, $payment_id = null, $action = "list", $id = null) {
+
+    function handler_adm_transactions($page, $payment_id = null, $action = "list", $id = null) {
         // show transactions. FIXME: should not be modifiable
         $page->setTitle('Administration - Paiements - Transactions');
         $page->assign('title', "Liste des transactions pour le paiement {$payment_id}");
-        
+
         if ($payment_id == null)
             $page->trigError("Aucun ID de paiement fourni.");
-        
-        $table_editor = new PLTableEditor("admin/transactions/{$payment_id}",'payment_transactions','id');
+
+        $table_editor = new PLTableEditor("admin/payments/transactions/{$payment_id}",'payment_transactions','id', true);
         $table_editor->set_where_clause(XDB::format('ref = {?}', $payment_id));
-        $table_editor->apply($page, 'list', $id); // only the 'list' action is allowed
-        $page->assign("readonly","readonly");     // don't show modification features
+        $table_editor->add_sort_field('id', true);
+        $table_editor->describe('ts_initiated', 'ts_initiated', true, false);
+        $table_editor->describe('commission', 'commission', true, false);
+        $table_editor->describe('pkey', 'pkey', true, true);
+        $table_editor->describe('comment', 'comment', true, true);
+        $table_editor->describe('recon_id', 'recon_id', true, false);
+        $table_editor->describe('display', 'display', true, true);
+        $table_editor->apply($page, $action, $id);
+        $page->assign('addonly', 'addonly');     // don't show modification features, only add feature
     }
-    
-    function handler_adm_bankaccounts(&$page, $action = "list", $id = null) {
+
+    function handler_adm_bankaccounts($page, $action = "list", $id = null) {
         // managment of bank account used for money transfert
         $page->setTitle('Administration - Paiements - RIBs');
         $page->assign('title', "Liste des RIBs");
-        
-        $table_editor = new PLTableEditor("admin/payments/bankaccounts",'payment_bankaccounts','id');
-        $table_editor->describe('asso_id','ID du groupe',false);
-        $table_editor->describe('owner','titulaire',true);
-        $table_editor->add_option_table('groups','groups.id = t.asso_id');
-        $table_editor->add_option_field('groups.diminutif', 'group_name', 'groupe', 'varchar','account');
-        
-        // check RIB key
-        if ($action == "update" && Post::has("account") && !check_rib(Post::v("account"))) {
+
+        $table_editor = new PLTableEditor('admin/payments/bankaccounts', 'payment_bankaccounts', 'id');
+        $table_editor->describe('asso_id', 'ID du groupe', false, true);
+        $table_editor->describe('owner', 'titulaire', true);
+        $table_editor->add_option_table('groups', 'groups.id = t.asso_id');
+        $table_editor->add_option_field('groups.diminutif', 'group_name', 'groupe', 'varchar','iban');
+
+        /* check RIB key     FIXME: the column format (and name) changed
+        if ($action == 'update' && Post::has('account') && !check_rib(Post::v('account'))) {
             $page->trigError("Le RIB n'est pas valide");
-            $table_editor->apply($page, "edit", $id);
+            $table_editor->apply($page, 'edit', $id);
             return;
         }
-        
+        */
+
         $table_editor->apply($page, $action, $id);
     }
-    
-    function handler_adm_methods(&$page, $action = "list", $id = null) {
+
+    function handler_adm_methods($page, $action = "list", $id = null) {
         // show and edit payment methods
         $page->setTitle('Administration - Paiements - Méthodes');
-        $page->assign('title', "Méthodes de paiement");
-        $table_editor = new PLTableEditor("admin/payments/methods",'payment_methods','id');
+        $page->assign('title', 'Méthodes de paiement');
+        $table_editor = new PLTableEditor('admin/payments/methods', 'payment_methods', 'id');
         $table_editor->apply($page, $action, $id);
     }
 
-    function handler_adm_reconcile(&$page, $step = 'list', $param = null) {
+    function handler_adm_reconcile($page, $step = 'list', $param = null) {
         // reconciles logs with transactions
         // FIXME: the admin is considered to be fair => he doesn't hack the $step value, nor other params
         $page->setTitle('Administration - Paiements - Réconciliations');
         $page->changeTpl('payment/reconcile.tpl');
         $page->assign('step', $step);
         $list = true;
-        
+
         // actions
         if ($step == 'delete' && $param != null) {
             S::assert_xsrf_token();
-            XDB::execute("DELETE FROM payment_reconcilations WHERE id={?}", $param);
+            XDB::execute('DELETE FROM payment_reconcilations WHERE id = {?}', $param);
             // FIXME: hardcoding !!!
-            XDB::execute("UPDATE payment_transactions SET recon_id=NULL,commission=NULL WHERE recon_id={?} AND method_id=2", $param);
-            XDB::execute("UPDATE payment_transactions SET recon_id=NULL WHERE recon_id={?} AND method_id=1", $param);
-            $page->trigSuccess("L'entrée ".$param." a été supprimée.");
-        
+            XDB::execute('UPDATE payment_transactions SET recon_id = NULL,commission = NULL WHERE recon_id = {?} AND method_id = 2', $param);
+            XDB::execute('UPDATE payment_transactions SET recon_id = NULL WHERE recon_id = {?} AND method_id = 1', $param);
+            $page->trigSuccess("L'entrée " . $param . ' a été supprimée.');
+
         } elseif ($step == 'edit') {
             $page->trigError("L'édition n'est pas implémentée.");
-            
+
         } elseif ($step == 'step5') {
             $page->trigSuccess("La réconciliation est terminée. Il est maintenant nécessaire de générer les virements.");
-        
+
         }
-        
+
         if($list) {
             // show list of reconciliations, with a "add" button
-            $page->assign('title', "Réconciliation - Liste");
+            $page->assign('title', 'Réconciliation - Liste');
             $page->assign('step', 'list');
-            
+
             $recongps = array();
-    
+
+            // récupère les réconciliations non groupées
             $res = XDB::query("SELECT  r.id, short_name AS method, period_start, period_end, status,
                                        payment_count, sum_amounts, sum_commissions
                                  FROM  payment_reconcilations AS r
-                            LEFT JOIN  payment_methods AS m ON r.method_id=m.id
+                            LEFT JOIN  payment_methods        AS m ON (r.method_id = m.id)
                                 WHERE  recongroup_id IS NULL
                              ORDER BY  period_end DESC, period_start DESC");
             foreach ($res->fetchAllAssoc() as $recon)
                 $recongps[] = array('recons' => array($recon), 'transfers' => array());
-            
+
+            // ne récupère que les 18 derniers groupements
             $res = XDB::query("SELECT  recongroup_id AS id
                                  FROM  payment_reconcilations
                              GROUP BY  recongroup_id
-                             ORDER BY  MAX(period_end) DESC, MIN(period_start) DESC");
+                             ORDER BY  MAX(period_end) DESC, MIN(period_start) DESC
+                                LIMIT  18");
             foreach ($res->fetchAllAssoc() as $recongp) {
                 $res = XDB::query("SELECT  r.id, short_name AS method, period_start, period_end, status,
                                            payment_count, sum_amounts, sum_commissions
                                      FROM  payment_reconcilations AS r
-                                LEFT JOIN  payment_methods AS m ON r.method_id=m.id
-                                    WHERE  recongroup_id={?}
+                                LEFT JOIN  payment_methods        AS m ON (r.method_id = m.id)
+                                    WHERE  recongroup_id = {?}
                                  ORDER BY  period_end DESC, period_start DESC",
                                   $recongp['id']);
                 $recongp['recons'] = $res->fetchAllAssoc();
 
-                $res = XDB::query("SELECT  id, payment_id, amount, account_id, message, date
-                                     FROM  payment_transfers
-                                    WHERE  recongroup_id={?}",
+                $res = XDB::query('SELECT  t.id, t.payment_id, t.amount, t.message, t.date
+                                     FROM  payment_transfers    AS t
+                                    WHERE  recongroup_id = {?}',
                                   $recongp['id']);
-                $recongp['transfers'] = $res->fetchAllAssoc();
-                        
+                $transfers = $res->fetchAllAssoc();
+                foreach ($transfers as $id => $t) {
+                    if ($t['date'] == NULL)  { // si le virement n'est pas fait, on va récupérer le rib associé au paiment
+                        $ownertmp = XDB::fetchOneCell('SELECT  b.owner
+                                                         FROM  payment_bankaccounts AS b
+                                                    LEFT JOIN  payments             AS p ON (p.rib_id = b.id)
+                                                        WHERE  p.id = {?}', $t['payment_id']);
+                    } else { // sinon on prend celui associé au virement
+                        $ownertmp = XDB::fetchOneCell('SELECT  b.owner
+                                                         FROM  payment_bankaccounts AS b
+                                                    LEFT JOIN  payment_transfers    AS t ON (t.account_id = b.id)
+                                                        WHERE  t.id = {?}', $t['id']);
+                    }
+                    $transfers[$id]['owner'] = $ownertmp;
+                }
+                $recongp['transfers'] = $transfers;
+
                 $recongps[] = $recongp;
             }
             $page->assign_by_ref('recongps', $recongps);
         }
     }
-    
-    function handler_adm_importlogs(&$page, $step, $param = null) {
+
+    function handler_adm_importlogs($page, $step, $param = null) {
         $page->setTitle('Administration - Paiements - Réconciliations');
         $page->changeTpl('payment/reconcile.tpl');
         $page->assign('step', $step);
-        
+
         if (isset($_SESSION['paymentrecon_data'])) {
             // create temporary table with imported data
-            XDB::execute("CREATE TEMPORARY TABLE payment_tmp (
+            XDB::execute('CREATE TEMPORARY TABLE payment_tmp (
                             reference VARCHAR(255) PRIMARY KEY,
                             date DATE,
                             amount DECIMAL(9,2),
                             commission DECIMAL(9,2)
-                          )");
+                          )');
             foreach ($_SESSION['paymentrecon_data'] as $i)
-                XDB::execute("INSERT INTO payment_tmp VALUES ({?},{?},{?},{?})",
+                XDB::execute('INSERT INTO payment_tmp VALUES ({?}, {?}, {?}, {?})',
                               $i['reference'], $i['date'], $i['amount'], $i['commission']);
         }
-        
+
         if ($step == 'step1') {
-            $page->assign('title', "Étape 1");
+            $page->assign('title', 'Étape 1');
             unset($_SESSION['paymentrecon_method']);
             unset($_SESSION['paymentrecon_data']);
             unset($_SESSION['paymentrecon_id']);
-                
+
             // was a payment method choosen ?
             if ($param != null) {
                 $_SESSION['paymentrecon_method'] = (int)$param;
-                pl_redirect("admin/reconcile/importlogs/step2");
+                pl_redirect('admin/reconcile/importlogs/step2');
 
             } else {
             // ask to choose a payment method
-                $res = XDB::query("SELECT id, text FROM payment_methods");
+                $res = XDB::query('SELECT id, text FROM payment_methods');
                 $page->assign('methods', $res->fetchAllAssoc());
             }
 
         } elseif ( $step == 'step2' ) {
-            $page->assign('title', "Étape 2");
-            
+            $page->assign('title', 'Étape 2');
+
             // import logs formated in CVS
-            $fields = array('date','reference','amount','commission');
+            $fields = array('date', 'reference', 'amount', 'commission');
             $importer = new PaymentLogsImporter();
             $importer->apply($page, 'admin/reconcile/importlogs/step2', $fields);
-            
+
             // if import is finished
             $result = $importer->get_result();
             if($result != null) {
                 $_SESSION['paymentrecon_data'] = $result;
-                pl_redirect("admin/reconcile/importlogs/step3");
+                pl_redirect('admin/reconcile/importlogs/step3');
             }
-            
+
         } elseif ($step == 'step3' ) {
-            $page->assign('title', "Étape 3");
-            
+            $page->assign('title', 'Étape 3');
+
             // compute reconcilation summary data
-            $res = XDB::query("SELECT  MIN(date) AS period_start, MAX(date) AS period_end,
+            $res = XDB::query('SELECT  MIN(date) AS period_start, MAX(date) AS period_end,
                                        count(*) AS payment_count, SUM(amount) AS sum_amounts,
                                        SUM(commission) AS sum_commissions
-                                 FROM  payment_tmp");
+                                 FROM  payment_tmp');
             $recon = $res->fetchOneAssoc();
             $recon['method_id'] = $_SESSION['paymentrecon_method'];
-            
+
             // create reconciliation item in database
             if(Post::has('next')) {
                 S::assert_xsrf_token();
-                
+
                 // get parameters
-                $recon['period_start'] = preg_replace('/([0-9]{1,2})\/([0-9]{1,2})\/([0-9]{4})/','\3-\2-\1',Post::v('period_start'));
-                $recon['period_end']   = preg_replace('/([0-9]{1,2})\/([0-9]{1,2})\/([0-9]{4})/','\3-\2-\1',Post::v('period_end'));
+                $recon['period_start'] = preg_replace('/([0-9]{1,2})\/([0-9]{1,2})\/([0-9]{4})/', '\3-\2-\1', Post::v('period_start'));
+                $recon['period_end']   = preg_replace('/([0-9]{1,2})\/([0-9]{1,2})\/([0-9]{4})/', '\3-\2-\1', Post::v('period_end'));
                     // FIXME: save checks to be done at next step
-                
+
                 // Create reconcilation item in database
                     // FIXME: check if period doesn't overlap with others for the same method_id
-                XDB::execute("INSERT INTO  payment_reconcilations (method_id, period_start, period_end,
+                XDB::execute('INSERT INTO  payment_reconcilations (method_id, period_start, period_end,
                                                                    payment_count, sum_amounts, sum_commissions)
-                                   VALUES  ({?}, {?}, {?}, {?}, {?}, {?})",
+                                   VALUES  ({?}, {?}, {?}, {?}, {?}, {?})',
                              $recon['method_id'], $recon['period_start'], $recon['period_end'],
                              $recon['payment_count'], $recon['sum_amounts'], $recon['sum_commissions']);
                 $_SESSION['paymentrecon_id'] = XDB::insertId();
-                
+
                 // reconcile simple cases (trans.commission n'est modifié que s'il vaut NULL)
                 XDB::execute("UPDATE  payment_transactions AS trans, payment_tmp AS tmp
-                                 SET  trans.recon_id={?}, trans.commission=tmp.commission
-                               WHERE  trans.fullref=tmp.reference
-                                      AND trans.amount=tmp.amount AND DATE(trans.ts_confirmed)=tmp.date
-                                      AND (trans.commission IS NULL OR trans.commission=tmp.commission)
-                                      AND method_id={?} AND recon_id IS NULL AND status='confirmed'",
+                                 SET  trans.recon_id = {?}, trans.commission=tmp.commission
+                               WHERE  trans.fullref = tmp.reference
+                                      AND trans.amount = tmp.amount AND DATE(trans.ts_confirmed) = tmp.date
+                                      AND (trans.commission IS NULL OR trans.commission = tmp.commission)
+                                      AND method_id = {?} AND recon_id IS NULL AND status = 'confirmed'",
                              $_SESSION['paymentrecon_id'], $recon['method_id']);
-                
-                pl_redirect("admin/reconcile/importlogs/step4");                
-                
+
+                pl_redirect("admin/reconcile/importlogs/step4");
+
             // show summary of the imported data + ask form start/end of reconcilation period
             } else {
                 $recon['period_start'] = preg_replace('/([0-9]{4})-([0-9]{2})-([0-9]{2})/', '\3/\2/\1', $recon['period_start']);
@@ -789,181 +836,204 @@ class PaymentModule extends PLModule
             }
 
         } elseif ($step == 'step4' ) {
-            $page->assign('title', "Étape 4");
-            
+            $page->assign('title', 'Étape 4');
+
             // get reconcilation summary informations
-            $res = XDB::query("SELECT * FROM payment_reconcilations WHERE id={?}", $_SESSION['paymentrecon_id']);
+            $res = XDB::query('SELECT * FROM payment_reconcilations WHERE id = {?}', $_SESSION['paymentrecon_id']);
             $recon = $res->fetchOneAssoc();
             $page->assign('recon', $recon);
 
             if (Post::has('force')) {
                 S::assert_xsrf_token();
                 foreach (Post::v('force') as $id => $value) {
-                    XDB::execute("UPDATE  payment_transactions AS trans, payment_tmp AS tmp
-                                     SET  trans.recon_id={?}, trans.commission=tmp.commission
-                                   WHERE  trans.id={?} AND trans.fullref=tmp.reference",
+                    XDB::execute('UPDATE  payment_transactions AS trans, payment_tmp AS tmp
+                                     SET  trans.recon_id = {?}, trans.commission = tmp.commission
+                                   WHERE  trans.id = {?} AND trans.fullref = tmp.reference',
                                  $_SESSION['paymentrecon_id'], $id);
                 }
-                $page->trigSuccess('La réconciliation a été forcée pour '.count(Post::v('force')).' transaction(s).');
-                
+                $page->trigSuccess('La réconciliation a été forcée pour ' . count(Post::v('force')) . ' transaction(s).');
+
             } elseif (Post::has('next')) {
-                if (strlen($recon['comments'])<3) {
-                    $page->trigError("Le commentaire doit contenir au moins 3 caractères.");
+                if (strlen($recon['comments'])< 3 ) {
+                    $page->trigError('Le commentaire doit contenir au moins 3 caractères.');
                 } else {
-                    XDB::execute("UPDATE payment_reconcilations SET status='transfering' WHERE id={?}", $_SESSION['paymentrecon_id']);
+                    XDB::execute("UPDATE payment_reconcilations SET status = 'transfering' WHERE id = {?}", $_SESSION['paymentrecon_id']);
                     pl_redirect('admin/reconcile/step5');
                 }
-                
+
             } elseif (Post::has('savecomments')) {
                 S::assert_xsrf_token();
                 $recon['comments'] = Post::v('comments');
                 $page->assign('recon', $recon);
-                XDB::execute("UPDATE payment_reconcilations SET comments={?} WHERE id={?}", $recon['comments'], $_SESSION['paymentrecon_id']);
+                XDB::execute('UPDATE payment_reconcilations SET comments = {?} WHERE id = {?}', $recon['comments'], $_SESSION['paymentrecon_id']);
                 $page->trigSuccess('Les commentaires ont été enregistrés.');
             }
-            
+
             // reconcilation results - ok
-            $res = XDB::query("SELECT  count(*),SUM(amount),SUM(commission)
-                                 FROM  payment_transactions WHERE recon_id={?}",
+            $res = XDB::query('SELECT  count(*), SUM(amount), SUM(commission)
+                                 FROM  payment_transactions
+                                WHERE recon_id = {?}',
                               $recon['id']);
-            list($ok_count,$ok_sum_amounts,$ok_sum_coms) = $res->fetchOneRow();
+            list($ok_count, $ok_sum_amounts, $ok_sum_coms) = $res->fetchOneRow();
             $page->assign('ok_count', $ok_count);
-            
+
             // reconcilation results - ref exists, but some data differs
-            $res = XDB::query("SELECT  id, fullref, method_id, ts_confirmed, trans.amount, trans.commission, status, recon_id,
+            $res = XDB::query('SELECT  id, fullref, method_id, ts_confirmed, trans.amount, trans.commission, status, recon_id,
                                        reference, date, tmp.amount as amount2, tmp.commission as commission2
                                  FROM  payment_transactions AS trans
-                           INNER JOIN  payment_tmp AS tmp ON trans.fullref=tmp.reference
-                                WHERE  trans.recon_id IS NULL OR trans.recon_id != {?}",
+                           INNER JOIN  payment_tmp          AS tmp ON (trans.fullref = tmp.reference)
+                                WHERE  trans.recon_id IS NULL OR trans.recon_id != {?}',
                               $recon['id']);
             $differs = $res->fetchAllAssoc();
             $page->assign_by_ref('differs', $differs);
             $page->assign('differ_count', count($differs));
-            
+
             // reconcilation results - ref doesn't exists in database
-            $res = XDB::query("SELECT  tmp.*
-                                 FROM  payment_tmp AS tmp
-                            LEFT JOIN  payment_transactions AS trans ON trans.fullref=tmp.reference
-                                WHERE  trans.fullref IS NULL");
+            $res = XDB::query('SELECT  tmp.*
+                                 FROM  payment_tmp          AS tmp
+                            LEFT JOIN  payment_transactions AS trans ON (trans.fullref = tmp.reference)
+                                WHERE  trans.fullref IS NULL');
             $only_import = $res->fetchAllAssoc();
             $page->assign_by_ref('only_import', $only_import);
             $page->assign('onlyim_count', count($only_import));
-            
+
             // reconcilation results - exists in database but not in import
-            $res = XDB::query("SELECT  trans.*
+            $res = XDB::query('SELECT  trans.*
                                  FROM  payment_transactions AS trans
-                            LEFT JOIN  payment_tmp AS tmp ON trans.fullref=tmp.reference
-                                WHERE  {?}<=DATE(trans.ts_confirmed) AND DATE(trans.ts_confirmed)<={?}
-                                       AND tmp.reference IS NULL AND method_id={?}",
+                            LEFT JOIN  payment_tmp          AS tmp ON (trans.fullref = tmp.reference)
+                                WHERE  {?} <= DATE(trans.ts_confirmed) AND DATE(trans.ts_confirmed) <= {?}
+                                       AND tmp.reference IS NULL AND method_id = {?}',
                               $recon['period_start'], $recon['period_end'], $recon['method_id']);
             $only_database = $res->fetchAllAssoc();
             $page->assign_by_ref('only_database', $only_database);
             $page->assign('onlydb_count', count($only_database));
         }
     }
-    
-    function handler_adm_transfers(&$page, $action = null, $id = null) {
+
+    function handler_adm_transfers($page, $action = null, $id = null) {
         // list/log all bank transfers and link them to individual transactions
-        
+
         if (Post::has('generate')) {
             $recon_ids = array_keys(Post::v('recon_id'));
-            
+
             // generate a new reconcilation group ID
             $res = XDB::query("SELECT MAX(recongroup_id)+1 FROM payment_reconcilations");
             $recongp_id = $res->fetchOneCell();
             if ($recongp_id == null) $recongp_id = 1;
-            
+
             // add reconcilations to group
                 // FIXME: should check if reconcilations are in good status
-            XDB::execute("UPDATE payment_reconcilations SET recongroup_id={?}, status='closed'
-                          WHERE id IN {?}",
-                          $recongp_id, $recon_ids);
-            
+            XDB::execute("UPDATE  payment_reconcilations
+                             SET  recongroup_id = {?}, status = 'closed'
+                           WHERE  id IN {?}",
+                         $recongp_id, $recon_ids);
+
             // create transfers
-            XDB::execute("INSERT INTO  payment_transfers
+            XDB::execute('INSERT INTO  payment_transfers
                                SELECT  NULL, {?}, t.ref, SUM(t.amount+t.commission), NULL, p.text, NULL
                                  FROM  payment_transactions AS t
-                            LEFT JOIN  payments AS p ON t.ref = p.id
-                            LEFT JOIN  groups AS g ON p.asso_id = g.id
-                                WHERE  t.recon_id IN {?}
-                             GROUP BY  t.ref",
-                              $recongp_id, $recon_ids);
-            
+                            LEFT JOIN  payments             AS p ON (t.ref = p.id)
+                            LEFT JOIN  groups               AS g ON (p.asso_id = g.id)
+                                WHERE  t.recon_id IN {?} AND t.status = "confirmed"
+                             GROUP BY  t.ref',
+                         $recongp_id, $recon_ids);
+
             //$res = XDB::query("SELECT * FROM  payment_reconcilations WHERE id IN {?}", $recon_ids);
             //$recons = $res->fetchAllAssoc();
-            
-            $page->trigSuccess("Les virements ont été générés pour ".count($recon_ids)." réconciliations.");
+
+            $page->trigSuccess('Les virements ont été générés pour ' . count($recon_ids) . ' réconciliations.');
             $this->handler_adm_reconcile($page);
-            
-        } elseif ($action == "delgroup") {
+
+        } elseif ($action == 'delgroup') {
             S::assert_xsrf_token();
-            XDB::execute("UPDATE payment_reconcilations SET status='transfering', recongroup_id=NULL WHERE recongroup_id={?}", $id);
-            XDB::execute("DELETE FROM payment_transfers WHERE recongroup_id={?} AND date IS NULL", $id);
-            
+            XDB::execute("UPDATE  payment_reconcilations
+                             SET  status = 'transfering', recongroup_id = NULL
+                           WHERE  recongroup_id = {?}", $id);
+            XDB::execute("DELETE FROM  payment_transfers
+                                WHERE  recongroup_id = {?} AND date IS NULL", $id);
+
             $page->trigSuccess("Les virements non réalisés ont été supprimé du groupe ".$id.".");
             $this->handler_adm_reconcile($page);
-            
+
         } elseif ($action == "confirm") {
             S::assert_xsrf_token();
-            XDB::execute("UPDATE payment_transfers SET date=NOW() WHERE id={?}", $id);
-            
-            $page->trigSuccess("Virement ".$id." confirmé.");
+            $account_id = XDB::fetchOneCell('SELECT  rib_id
+                                               FROM  payments          AS p 
+                                          LEFT JOIN  payment_transfers AS t ON (t.payment_id = p.id)
+                                              WHERE  t.id = {?}', $id);
+            XDB::execute('UPDATE  payment_transfers
+                             SET  date = NOW(), account_id = {?}
+                           WHERE  id = {?}', $account_id, $id);
+
+            $page->trigSuccess('Virement ' . $id . ' confirmé.');
             $this->handler_adm_reconcile($page);
-            
+
         } else {
-            pl_redirect("admin/reconcile");
+            pl_redirect('admin/reconcile');
         }
     }
 }
 
 class PaymentLogsImporter extends CSVImporter {
     protected $result;
-    
+
     public function __construct() {
         parent::__construct('');
-        $this->registerFunction('systempay_commission', 'Compute BPLC commission', array($this,"compute_systempay_commission"));
-        $this->registerFunction('payment_id', 'Autocompute payment ID', array($this,"compute_payment_id"));
+        $this->registerFunction('systempay_commission', 'Compute BPLC commission', array($this, 'compute_systempay_commission'));
+        $this->registerFunction('payment_id', 'Autocompute payment ID', array($this, 'compute_payment_id'));
         //$this->forceValue('payment_id','func_payment_id');
     }
-    
+
     public function run($action = null, $insert_relation = null, $update_relation = null) {
         $this->result = array();
         foreach ($this->data as $line) {
             $a = $this->makeAssoc($line, $insert_relation);
-            $a['date'] = preg_replace('/([0-9]{2})\/([0-9]{2})\/([0-9]{4}).*/','\3-\2-\1', $a['date']);
-            $a['amount'] = str_replace(',','.',$a['amount']);
-            $a['commission'] = str_replace(',','.',$a['commission']);
+            // convert date
+            $a['date'] = preg_replace('/([0-9]{2})\/([0-9]{2})\/([0-9]{4}).*/', '\3-\2-\1', $a['date']);
+            $a['date'] = preg_replace('/T.*/','', $a['date']);
+
+            // convert money
+            $a['amount'] = str_replace(',', '.', $a['amount']);
+            $a['commission'] = str_replace(',', '.', $a['commission']);
             $this->result[] = $a;
         }
     }
-    
+
     public function get_result() {
         return $this->result;
     }
-    
+
     static public function compute_systempay_commission($line, $key, $relation) {
-        static $EEE_countries = array('France','Allemagne','Autriche','Belgique','Bulgarie','Chypre',
-            'Danemark','Espagne','Estonie','Finlande','Grèce','Hongrie','Irlande','Islande','Italie',
-            'Lettonie','Liechtenstein','Lituanie','Luxembourg','Malte','Norvège','Pays-Bas','Pologne',
-            'Portugal','Roumanie','Royaume-Uni','Slovaquie','Slovénie','Suède','République Tchèque');
-        
-        if($key!='commission' || !array_key_exists('carte',$line)) return null;
+        static $EEE_countries = array(
+            'France', 'Allemagne', 'Autriche', 'Belgique', 'Bulgarie', 'Chypre', 'Croatie',
+            'Danemark', 'Espagne', 'Estonie', 'Finlande', 'Grèce', 'Hongrie', 'Irlande', 'Islande', 'Italie',
+            'Lettonie', 'Liechtenstein', 'Lituanie', 'Luxembourg', 'Malte', 'Norvège', 'Pays-Bas', 'Pologne',
+            'Portugal', 'Roumanie', 'Royaume-Uni', 'Slovaquie', 'Slovénie', 'Suède', 'République Tchèque'
+        );
+
+        if($key!='commission' || !array_key_exists('carte', $line)) {
+            return null;
+        }
         $amount = self::getValue($line, 'amount', $relation['amount']);
-        if (in_array($line['pays carte'],$EEE_countries))
-            return -0.20 - round($amount*0.005, 2);
-        else
-            return -0.20 - round($amount*0.005, 2) - 0.76;
+        if (in_array($line['pays carte'], $EEE_countries)) {
+            return -0.20 - round($amount * 0.005, 2);
+        } else {
+            return -0.20 - round($amount * 0.005, 2) - 0.75;
+        }
     }
 
     static public function compute_payment_id($line, $key, $relation) {
-        if ($key != 'payment_id') return null;
+        if ($key != 'payment_id') {
+            return null;
+        }
         $reference = self::getValue($line, 'reference', $relation['reference']);
-        if (ereg('-([0-9]+)$', $reference, $matches))
+        if (preg_match('/-([0-9]+)$/', $reference, $matches)) {
             return $matches[1];
-        else
+        } else {
             return null;
+        }
     }
 }
 
-// vim:set et sw=4 sts=4 sws=4 foldmethod=marker enc=utf-8:
+// vim:set et sw=4 sts=4 sws=4 foldmethod=marker fenc=utf-8:
 ?>