diff --git a/ChangeLog b/ChangeLog index 6e4c1388024..5f2520ec3f3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,6 +3,7 @@ English Dolibarr ChangeLog ***** ChangeLog for 3.0 compared to 2.9 ***** For users: +- New: Can filter on payment type and bank account in payment lists. - New: When sending supplier orders by mail, a text is predefined. - New: Upgrade process works with Postgresql. - New: Task #10538: Add filter on expiration date of subscription for diff --git a/htdocs/compta/paiement/liste.php b/htdocs/compta/paiement/liste.php index 80cd3f142dd..2c62831c980 100644 --- a/htdocs/compta/paiement/liste.php +++ b/htdocs/compta/paiement/liste.php @@ -1,6 +1,6 @@ - * Copyright (C) 2004-2009 Laurent Destailleur + * Copyright (C) 2004-2010 Laurent Destailleur * Copyright (C) 2005-2009 Regis Houssin * * This program is free software; you can redistribute it and/or modify @@ -53,11 +53,13 @@ if (! $sortfield) $sortfield="p.rowid"; /* -* View -*/ + * View + */ llxHeader('',$langs->trans("ListPayment")); +$form=new Form($db); + $sql = "SELECT DISTINCT p.rowid, p.datep as dp, p.amount,"; $sql.= " p.statut, p.num_paiement,"; //$sql.= " c.libelle as paiement_type,"; @@ -73,47 +75,55 @@ $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."facture as f ON pf.fk_facture = f.rowid"; $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."societe as s ON f.fk_soc = s.rowid"; if (!$user->rights->societe->client->voir && !$socid) { - $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."societe_commerciaux as sc ON s.rowid = sc.fk_soc"; + $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."societe_commerciaux as sc ON s.rowid = sc.fk_soc"; } $sql.= " WHERE p.fk_paiement = c.id"; $sql.= " AND s.entity = ".$conf->entity; if (!$user->rights->societe->client->voir && !$socid) { - $sql.= " AND sc.fk_user = " .$user->id; + $sql.= " AND sc.fk_user = " .$user->id; } if ($socid) { - $sql.= " AND f.fk_soc = ".$socid; + $sql.= " AND f.fk_soc = ".$socid; } // Search criteria if ($_REQUEST["search_ref"]) { - $sql .=" AND p.rowid=".$_REQUEST["search_ref"]; + $sql .=" AND p.rowid=".$_REQUEST["search_ref"]; +} +if ($_REQUEST["search_account"]) +{ + $sql .=" AND b.fk_account=".$_REQUEST["search_account"]; +} +if ($_REQUEST["search_paymenttype"]) +{ + $sql .=" AND c.code='".$_REQUEST["search_paymenttype"]."'"; } if ($_REQUEST["search_amount"]) { - $sql .=" AND p.amount=".price2num($_REQUEST["search_amount"]); + $sql .=" AND p.amount=".price2num($_REQUEST["search_amount"]); } if ($_REQUEST["search_company"]) { - $sql .=" AND s.nom like '%".addslashes($_REQUEST["search_company"])."%'"; + $sql .=" AND s.nom LIKE '%".addslashes($_REQUEST["search_company"])."%'"; } if ($_GET["orphelins"]) // Option for debugging purpose only { - // Paiements lies a aucune facture (pour aide au diagnostic) - $sql = "SELECT p.rowid, p.datep as dp, p.amount,"; - $sql.= " p.statut, p.num_paiement,"; - //$sql.= " c.libelle as paiement_type"; - $sql.= " c.code as paiement_code,"; - $sql.= " s.rowid as socid, s.nom"; - $sql.= " FROM ".MAIN_DB_PREFIX."paiement as p,"; - $sql.= " ".MAIN_DB_PREFIX."c_paiement as c"; - $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."paiement_facture as pf ON p.rowid = pf.fk_paiement"; - $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."societe as s ON f.fk_soc = s.rowid"; - $sql.= " WHERE p.fk_paiement = c.id"; - $sql.= " AND s.entity = ".$conf->entity; - $sql.= " AND pf.rowid IS NULL"; + // Paiements lies a aucune facture (pour aide au diagnostic) + $sql = "SELECT p.rowid, p.datep as dp, p.amount,"; + $sql.= " p.statut, p.num_paiement,"; + //$sql.= " c.libelle as paiement_type"; + $sql.= " c.code as paiement_code,"; + $sql.= " s.rowid as socid, s.nom"; + $sql.= " FROM ".MAIN_DB_PREFIX."paiement as p,"; + $sql.= " ".MAIN_DB_PREFIX."c_paiement as c"; + $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."paiement_facture as pf ON p.rowid = pf.fk_paiement"; + $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."societe as s ON f.fk_soc = s.rowid"; + $sql.= " WHERE p.fk_paiement = c.id"; + $sql.= " AND s.entity = ".$conf->entity; + $sql.= " AND pf.rowid IS NULL"; } $sql.= $db->order($sortfield,$sortorder); $sql.= $db->plimit( $limit+1 ,$offset); @@ -123,111 +133,116 @@ $resql = $db->query($sql); if ($resql) { - $num = $db->num_rows($resql); - $i = 0; + $num = $db->num_rows($resql); + $i = 0; - $paramlist=''; - $paramlist.=($_REQUEST["orphelins"]?"&orphelins=1":""); - $paramlist.=($_REQUEST["search_ref"]?"&search_ref=".$_REQUEST["search_ref"]:""); - $paramlist.=($_REQUEST["search_company"]?"&search_company=".$_REQUEST["search_company"]:""); - $paramlist.=($_REQUEST["search_amount"]?"&search_amount=".$_REQUEST["search_amount"]:""); + $paramlist=''; + $paramlist.=($_REQUEST["orphelins"]?"&orphelins=1":""); + $paramlist.=($_REQUEST["search_ref"]?"&search_ref=".$_REQUEST["search_ref"]:""); + $paramlist.=($_REQUEST["search_company"]?"&search_company=".$_REQUEST["search_company"]:""); + $paramlist.=($_REQUEST["search_amount"]?"&search_amount=".$_REQUEST["search_amount"]:""); - print_barre_liste($langs->trans("ReceivedCustomersPayments"), $page, "liste.php",$paramlist,$sortfield,$sortorder,'',$num); + print_barre_liste($langs->trans("ReceivedCustomersPayments"), $page, "liste.php",$paramlist,$sortfield,$sortorder,'',$num); - print '
'; - print ''; - print ''; - print_liste_field_titre($langs->trans("RefPayment"),"liste.php","p.rowid","",$paramlist,"",$sortfield,$sortorder); - print_liste_field_titre($langs->trans("Date"),"liste.php","dp","",$paramlist,'align="center"',$sortfield,$sortorder); - print_liste_field_titre($langs->trans("ThirdParty"),"liste.php","s.nom","",$paramlist,"",$sortfield,$sortorder); - print_liste_field_titre($langs->trans("Type"),"liste.php","c.libelle","",$paramlist,"",$sortfield,$sortorder); - print_liste_field_titre($langs->trans("Account"),"liste.php","ba.label","",$paramlist,"",$sortfield,$sortorder); - print_liste_field_titre($langs->trans("Amount"),"liste.php","p.amount","",$paramlist,'align="right"',$sortfield,$sortorder); - //print_liste_field_titre($langs->trans("Invoices"),"","","",$paramlist,'align="left"',$sortfield,$sortorder); - if ($conf->global->BILL_ADD_PAYMENT_VALIDATION) - { - print_liste_field_titre($langs->trans("Status"),"liste.php","p.statut","",$paramlist,'align="right"',$sortfield,$sortorder); - } - print "\n"; + print ''; + print '
'; + print ''; + print_liste_field_titre($langs->trans("RefPayment"),"liste.php","p.rowid","",$paramlist,"",$sortfield,$sortorder); + print_liste_field_titre($langs->trans("Date"),"liste.php","dp","",$paramlist,'align="center"',$sortfield,$sortorder); + print_liste_field_titre($langs->trans("ThirdParty"),"liste.php","s.nom","",$paramlist,"",$sortfield,$sortorder); + print_liste_field_titre($langs->trans("Type"),"liste.php","c.libelle","",$paramlist,"",$sortfield,$sortorder); + print_liste_field_titre($langs->trans("Account"),"liste.php","ba.label","",$paramlist,"",$sortfield,$sortorder); + print_liste_field_titre($langs->trans("Amount"),"liste.php","p.amount","",$paramlist,'align="right"',$sortfield,$sortorder); + //print_liste_field_titre($langs->trans("Invoices"),"","","",$paramlist,'align="left"',$sortfield,$sortorder); + if ($conf->global->BILL_ADD_PAYMENT_VALIDATION) + { + print_liste_field_titre($langs->trans("Status"),"liste.php","p.statut","",$paramlist,'align="right"',$sortfield,$sortorder); + } + print "\n"; - // Lines for filters fields - print ''; - print ''; - print ''; - print ''; - print ''; - print ''; - if ($conf->global->BILL_ADD_PAYMENT_VALIDATION) - { - print ''; - } - print "\n"; + // Lines for filters fields + print ''; + print ''; + print ''; + print ''; + print ''; + print ''; + print ''; + if ($conf->global->BILL_ADD_PAYMENT_VALIDATION) + { + print ''; + } + print "\n"; - $var=true; - while ($i < min($num,$limit)) - { - $objp = $db->fetch_object($resql); - $var=!$var; - print ""; + $var=true; + while ($i < min($num,$limit)) + { + $objp = $db->fetch_object($resql); + $var=!$var; + print ""; - print ''; + print ''; - print ''; + print ''; - // Company - print ''; + // Company + print ''; - print ''; - print ''; - print ''; + print ''; + print ''; + print ''; - if ($conf->global->BILL_ADD_PAYMENT_VALIDATION) - { - print ''; - } + if ($conf->global->BILL_ADD_PAYMENT_VALIDATION) + { + print ''; + } - print ''; + print ''; - $i++; - } - print "
'; - print ''; - print ' '; - print ''; - print ' '; - print ''; - print ''; - print ''; - print '
'; + print ''; + print ' '; + print ''; + print ''; + $form->select_types_paiements($_REQUEST["search_paymenttype"],'search_paymenttype','',2,1,1); + print ''; + $form->select_comptes($_REQUEST["search_account"],'search_account',0,'',1); + print ''; + print ''; + print ''; + print ''; + print '
'; - $paymentstatic->id=$objp->rowid; - $paymentstatic->ref=$objp->rowid; - print $paymentstatic->getNomUrl(1); - print ''; + $paymentstatic->id=$objp->rowid; + $paymentstatic->ref=$objp->rowid; + print $paymentstatic->getNomUrl(1); + print ''.dol_print_date($db->jdate($objp->dp),'day').''.dol_print_date($db->jdate($objp->dp),'day').''; - if ($objp->socid) - { - $companystatic->id=$objp->socid; - $companystatic->nom=$objp->nom; - print $companystatic->getNomUrl(1,'',24); - } - else print ' '; - print ''; + if ($objp->socid) + { + $companystatic->id=$objp->socid; + $companystatic->nom=$objp->nom; + print $companystatic->getNomUrl(1,'',24); + } + else print ' '; + print ''.$langs->trans("PaymentTypeShort".$objp->paiement_code).' '.$objp->num_paiement.''; - if ($objp->bid) - { - $accountstatic->id=$objp->bid; - $accountstatic->label=$objp->label; - print $accountstatic->getNomUrl(1); - } - else print ' '; - print ''.price($objp->amount).''.$langs->trans("PaymentTypeShort".$objp->paiement_code).' '.$objp->num_paiement.''; + if ($objp->bid) + { + $accountstatic->id=$objp->bid; + $accountstatic->label=$objp->label; + print $accountstatic->getNomUrl(1); + } + else print ' '; + print ''.price($objp->amount).''; - if ($objp->statut == 0) print ''; - print $paymentstatic->LibStatut($objp->statut,5); - if ($objp->statut == 0) print ''; - print ''; + if ($objp->statut == 0) print ''; + print $paymentstatic->LibStatut($objp->statut,5); + if ($objp->statut == 0) print ''; + print '
\n"; - print "
\n"; + $i++; + } + print "\n"; + print "\n"; } else { - dol_print_error($db); + dol_print_error($db); } $db->close(); diff --git a/htdocs/core/class/html.form.class.php b/htdocs/core/class/html.form.class.php index 45601f3b6b0..a38f534dca2 100644 --- a/htdocs/core/class/html.form.class.php +++ b/htdocs/core/class/html.form.class.php @@ -1551,7 +1551,7 @@ class Form * \brief Retourne la liste des modes de paiements possibles * \param selected Id du mode de paiement pre-selectionne * \param htmlname Nom de la zone select - * \param filtertype Pour filtre + * \param filtertype To filter on field type in llx_c_paiement * \param format 0=id+libelle, 1=code+code, 2=code+libelle * \param empty 1=peut etre vide, 0 sinon * \param noadmininfo 0=Add admin info, 1=Disable admin info @@ -1576,6 +1576,9 @@ class Form // On passe si on a demande de filtrer sur des modes de paiments particuliers if (sizeof($filterarray) && ! in_array($arraytypes['type'],$filterarray)) continue; + // We discard empty line if showempty is on because an empty line has already been output. + if ($empty && empty($arraytypes['code'])) continue; + if ($format == 0) print '