diff --git a/htdocs/core/class/canvas.class.php b/htdocs/core/class/canvas.class.php index dce217db542..9d637305dd7 100644 --- a/htdocs/core/class/canvas.class.php +++ b/htdocs/core/class/canvas.class.php @@ -77,7 +77,7 @@ class Canvas * and MVC properties: ->control (Controller), ->control->object (Model), ->template_dir (View) * * @param module Name of target module (thirdparty, contact, ...) - * @param card Type of card (ex: card, info, contactcard, ...) + * @param card Type of card (ex: 'card', 'info', 'contactcard', ...) or '' for a list page * @param canvas Name of canvas (ex: mycanvas, default, or mycanvas@myexternalmodule) */ function getCanvas($module, $card, $canvas) @@ -161,7 +161,7 @@ class Canvas } /** - * get object + * Get object * * @param param1 Param1 * @param param2 Param2 @@ -224,16 +224,16 @@ class Canvas /** * Return the template to display canvas (if it exists) * - * @param mode 'create', ''='view', 'edit' - * @return string Path to display canvas file if it exists, '' otherwise. + * @param string $mode 'create', ''='view', 'edit', 'list' + * @return string Path to display canvas file if it exists, '' otherwise. */ function displayCanvasExists($mode='view') { $newmode=$mode; if (empty($newmode)) $newmode='view'; if (empty($this->template_dir)) return 0; - //print $this->template_dir.$this->card.'_'.$newmode.'.tpl.php'; - if (file_exists($this->template_dir.$this->card.'_'.$newmode.'.tpl.php')) return 1; + //print $this->template_dir.($this->card?$this->card.'_':'').$newmode.'.tpl.php'; + if (file_exists($this->template_dir.($this->card?$this->card.'_':'').$newmode.'.tpl.php')) return 1; else return 0; } @@ -242,16 +242,16 @@ class Canvas * Variables used by templates may have been defined, loaded before * into the assign_values function. * - * @param mode 'create', 'view', 'edit' - * @param id Id of object to show + * @param string $mode 'create', 'view', 'edit' + * @param int $id Id of object to show */ function display_canvas($mode='view',$id=0) { global $db, $conf, $langs, $user, $canvas; global $id, $form, $formfile; - //print $this->template_dir.$this->card.'_'.$mode.'.tpl.php';exit; - include($this->template_dir.$this->card.'_'.$mode.'.tpl.php'); // Include native PHP template + //print $this->template_dir.($this->card?$this->card.'_':'').$mode.'.tpl.php';exit; + include($this->template_dir.($this->card?$this->card.'_':'').$mode.'.tpl.php'); // Include native PHP template } } diff --git a/htdocs/core/class/commonobject.class.php b/htdocs/core/class/commonobject.class.php index 3602a8401e3..0795a760fbd 100644 --- a/htdocs/core/class/commonobject.class.php +++ b/htdocs/core/class/commonobject.class.php @@ -52,7 +52,7 @@ abstract class CommonObject $sql.= " FROM ".MAIN_DB_PREFIX.$this->table_element; $sql.= " WHERE ref = '".$this->ref."'"; $sql.= " AND entity = ".$conf->entity; - dol_syslog("CommonObject::verifyNumRef sql=".$sql, LOG_DEBUG); + dol_syslog(get_class($this)."::verifyNumRef sql=".$sql, LOG_DEBUG); $resql = $this->db->query($sql); if ($resql) { @@ -62,7 +62,7 @@ abstract class CommonObject else { $this->error=$this->db->lasterror(); - dol_syslog("CommonObject::verifyNumRef ".$this->error, LOG_ERR); + dol_syslog(get_class($this)."::verifyNumRef ".$this->error, LOG_ERR); return -1; } } @@ -80,19 +80,19 @@ abstract class CommonObject { global $user,$conf,$langs; - dol_syslog("CommonObject::add_contact $fk_socpeople, $type_contact, $source"); + dol_syslog(get_class($this)."::add_contact $fk_socpeople, $type_contact, $source"); // Check parameters if ($fk_socpeople <= 0) { $this->error=$langs->trans("ErrorWrongValueForParameter","1"); - dol_syslog("CommonObject::add_contact ".$this->error,LOG_ERR); + dol_syslog(get_class($this)."::add_contact ".$this->error,LOG_ERR); return -1; } if (! $type_contact) { $this->error=$langs->trans("ErrorWrongValueForParameter","2"); - dol_syslog("CommonObject::add_contact ".$this->error,LOG_ERR); + dol_syslog(get_class($this)."::add_contact ".$this->error,LOG_ERR); return -2; } @@ -126,7 +126,7 @@ abstract class CommonObject $sql.= $this->db->idate($datecreate); $sql.= ", 4, '". $id_type_contact . "' "; $sql.= ")"; - dol_syslog("CommonObject::add_contact sql=".$sql); + dol_syslog(get_class($this)."::add_contact sql=".$sql); $resql=$this->db->query($sql); if ($resql) @@ -200,7 +200,7 @@ abstract class CommonObject $sql = "DELETE FROM ".MAIN_DB_PREFIX."element_contact"; $sql.= " WHERE rowid =".$rowid; - dol_syslog("CommonObject::delete_contact sql=".$sql); + dol_syslog(get_class($this)."::delete_contact sql=".$sql); if ($this->db->query($sql)) { if (! $notrigger) @@ -218,7 +218,7 @@ abstract class CommonObject else { $this->error=$this->db->lasterror(); - dol_syslog("CommonObject::delete_contact error=".$this->error, LOG_ERR); + dol_syslog(get_class($this)."::delete_contact error=".$this->error, LOG_ERR); return -1; } } @@ -243,7 +243,7 @@ abstract class CommonObject $sql.= " WHERE element_id =".$this->id; $sql.= " AND fk_c_type_contact IN (".$listId.")"; - dol_syslog("CommonObject::delete_linked_contact sql=".$sql); + dol_syslog(get_class($this)."::delete_linked_contact sql=".$sql); if ($this->db->query($sql)) { return 1; @@ -251,7 +251,7 @@ abstract class CommonObject else { $this->error=$this->db->lasterror(); - dol_syslog("CommonObject::delete_linked_contact error=".$this->error, LOG_ERR); + dol_syslog(get_class($this)."::delete_linked_contact error=".$this->error, LOG_ERR); return -1; } } @@ -288,7 +288,7 @@ abstract class CommonObject if ($statut >= 0) $sql.= " AND ec.statut = '".$statut."'"; $sql.=" ORDER BY t.name ASC"; - dol_syslog("CommonObject::liste_contact sql=".$sql); + dol_syslog(get_class($this)."::liste_contact sql=".$sql); $resql=$this->db->query($sql); if ($resql) { @@ -440,7 +440,7 @@ abstract class CommonObject $sql.= " AND tc.active = 1"; if ($status) $sql.= " AND ec.statut = ".$status; - dol_syslog("CommonObject::getIdContact sql=".$sql); + dol_syslog(get_class($this)."::getIdContact sql=".$sql); $resql=$this->db->query($sql); if ($resql) { @@ -453,7 +453,7 @@ abstract class CommonObject else { $this->error=$this->db->error(); - dol_syslog("CommonObject::getIdContact ".$this->error, LOG_ERR); + dol_syslog(get_class($this)."::getIdContact ".$this->error, LOG_ERR); return null; } @@ -606,7 +606,7 @@ abstract class CommonObject $sql.= $field." = '".$value."'"; $sql.= " WHERE rowid = ".$id; - dol_syslog("CommonObject::updateObjectField sql=".$sql, LOG_DEBUG); + dol_syslog(get_class($this)."::updateObjectField sql=".$sql, LOG_DEBUG); $resql = $this->db->query($sql); if ($resql) { @@ -632,7 +632,7 @@ abstract class CommonObject if (! $this->table_element) { - dol_print_error('',"CommonObject::load_previous_next_ref was called on objet with property table_element not defined", LOG_ERR); + dol_print_error('',get_class($this)."::load_previous_next_ref was called on objet with property table_element not defined", LOG_ERR); return -1; } @@ -720,7 +720,7 @@ abstract class CommonObject { if (! $this->table_element) { - dol_syslog("CommonObject::setProject was called on objet with property table_element not defined",LOG_ERR); + dol_syslog(get_class($this)."::setProject was called on objet with property table_element not defined",LOG_ERR); return -1; } @@ -729,7 +729,7 @@ abstract class CommonObject else $sql.= ' SET fk_projet = NULL'; $sql.= ' WHERE rowid = '.$this->id; - dol_syslog("CommonObject::setProject sql=".$sql); + dol_syslog(get_class($this)."::setProject sql=".$sql); if ($this->db->query($sql)) { $this->fk_project = $projectid; @@ -754,7 +754,7 @@ abstract class CommonObject { if (! $this->table_element) { - dol_syslog("CommonObject::setDocModel was called on objet with property table_element not defined",LOG_ERR); + dol_syslog(get_class($this)."::setDocModel was called on objet with property table_element not defined",LOG_ERR); return -1; } @@ -766,7 +766,7 @@ abstract class CommonObject // if ($this->element == 'facture') $sql.= " AND fk_statut < 2"; // if ($this->element == 'propal') $sql.= " AND fk_statut = 0"; - dol_syslog("CommonObject::setDocModel sql=".$sql); + dol_syslog(get_class($this)."::setDocModel sql=".$sql); $resql=$this->db->query($sql); if ($resql) { @@ -791,12 +791,12 @@ abstract class CommonObject { if (! $this->table_element_line) { - dol_syslog("CommonObject::line_order was called on objet with property table_element_line not defined",LOG_ERR); + dol_syslog(get_class($this)."::line_order was called on objet with property table_element_line not defined",LOG_ERR); return -1; } if (! $this->fk_element) { - dol_syslog("CommonObject::line_order was called on objet with property fk_element not defined",LOG_ERR); + dol_syslog(get_class($this)."::line_order was called on objet with property fk_element not defined",LOG_ERR); return -1; } @@ -1042,7 +1042,7 @@ abstract class CommonObject { if (! $this->table_element) { - dol_syslog("CommonObject::update_note was called on objet with property table_element not defined", LOG_ERR); + dol_syslog(get_class($this)."::update_note was called on objet with property table_element not defined", LOG_ERR); return -1; } @@ -1058,7 +1058,7 @@ abstract class CommonObject } $sql.= " WHERE rowid =". $this->id; - dol_syslog("CommonObject::update_note sql=".$sql, LOG_DEBUG); + dol_syslog(get_class($this)."::update_note sql=".$sql, LOG_DEBUG); if ($this->db->query($sql)) { $this->note = $note; @@ -1067,7 +1067,7 @@ abstract class CommonObject else { $this->error=$this->db->error(); - dol_syslog("CommonObject::update_note error=".$this->error, LOG_ERR); + dol_syslog(get_class($this)."::update_note error=".$this->error, LOG_ERR); return -1; } } @@ -1081,7 +1081,7 @@ abstract class CommonObject { if (! $this->table_element) { - dol_syslog("CommonObject::update_note_public was called on objet with property table_element not defined",LOG_ERR); + dol_syslog(get_class($this)."::update_note_public was called on objet with property table_element not defined",LOG_ERR); return -1; } @@ -1089,7 +1089,7 @@ abstract class CommonObject $sql.= " SET note_public = '".$this->db->escape($note_public)."'"; $sql.= " WHERE rowid =". $this->id; - dol_syslog("CommonObject::update_note_public sql=".$sql); + dol_syslog(get_class($this)."::update_note_public sql=".$sql); if ($this->db->query($sql)) { $this->note_public = $note_public; @@ -1130,7 +1130,7 @@ abstract class CommonObject $sql.= ' WHERE '.$this->fk_element.' = '.$this->id; if ($exclspec) $sql.= ' AND product_type <> 9'; - dol_syslog("CommonObject::update_price sql=".$sql); + dol_syslog(get_class($this)."::update_price sql=".$sql); $resql = $this->db->query($sql); if ($resql) { @@ -1194,7 +1194,7 @@ abstract class CommonObject $sql .= ' WHERE rowid = '.$this->id; //print "xx".$sql; - dol_syslog("CommonObject::update_price sql=".$sql); + dol_syslog(get_class($this)."::update_price sql=".$sql); $resql=$this->db->query($sql); if ($resql) { @@ -1203,14 +1203,14 @@ abstract class CommonObject else { $this->error=$this->db->error(); - dol_syslog("CommonObject::update_price error=".$this->error,LOG_ERR); + dol_syslog(get_class($this)."::update_price error=".$this->error,LOG_ERR); return -1; } } else { $this->error=$this->db->error(); - dol_syslog("CommonObject::update_price error=".$this->error,LOG_ERR); + dol_syslog(get_class($this)."::update_price error=".$this->error,LOG_ERR); return -1; } } @@ -1236,7 +1236,7 @@ abstract class CommonObject $sql.= ", '".$this->element."'"; $sql.= ")"; - dol_syslog("CommonObject::add_object_linked sql=".$sql); + dol_syslog(get_class($this)."::add_object_linked sql=".$sql); if ($this->db->query($sql)) { $this->db->commit(); @@ -1293,7 +1293,7 @@ abstract class CommonObject } //print $sql; - dol_syslog("CommonObject::fetchObjectLink sql=".$sql); + dol_syslog(get_class($this)."::fetchObjectLink sql=".$sql); $resql = $this->db->query($sql); if ($resql) { @@ -1382,73 +1382,25 @@ abstract class CommonObject $sql.= " SET fk_statut = ".$statut; $sql.= " WHERE rowid=".$elementId; - dol_syslog("CommonObject::setStatut sql=".$sql, LOG_DEBUG); + dol_syslog(get_class($this)."::setStatut sql=".$sql, LOG_DEBUG); $resql = $this->db->query($sql); if (!$resql) { $this->error=$this->db->lasterror(); - dol_syslog("CommonObject::setStatut ".$this->error, LOG_ERR); + dol_syslog(get_class($this)."::setStatut ".$this->error, LOG_ERR); return -1; } return 1; } - /** - * Fetch field list - */ - function getFieldList() - { - global $conf, $langs; - - $this->field_list = array(); - - $sql = "SELECT rowid, name, alias, title, align, sort, search, enabled, rang"; - $sql.= " FROM ".MAIN_DB_PREFIX."c_field_list"; - $sql.= " WHERE element = '".$this->fieldListName."'"; - $sql.= " AND entity = ".$conf->entity; - $sql.= " ORDER BY rang ASC"; - - $resql = $this->db->query($sql); - if ($resql) - { - $num = $this->db->num_rows($resql); - - $i = 0; - while ($i < $num) - { - $fieldlist = array(); - - $obj = $this->db->fetch_object($resql); - - $fieldlist["id"] = $obj->rowid; - $fieldlist["name"] = $obj->name; - $fieldlist["alias"] = $obj->alias; - $fieldlist["title"] = $langs->trans($obj->title); - $fieldlist["align"] = $obj->align; - $fieldlist["sort"] = $obj->sort; - $fieldlist["search"] = $obj->search; - $fieldlist["enabled"] = verifCond($obj->enabled); - $fieldlist["order"] = $obj->rang; - - array_push($this->field_list,$fieldlist); - - $i++; - } - $this->db->free($resql); - } - else - { - dol_print_error($db,$sql); - } - } /** * Load type of canvas of an object if it exists * - * @param id Record id - * @param ref Record ref - * @return int <0 if KO, 0 if nothing done, >0 if OK + * @param int $id Record id + * @param string $ref Record ref + * @return int <0 if KO, 0 if nothing done, >0 if OK */ function getCanvas($id=0,$ref='') { diff --git a/htdocs/product/canvas/product/actions_card_product.class.php b/htdocs/product/canvas/product/actions_card_product.class.php index 975ac337119..d0354155633 100755 --- a/htdocs/product/canvas/product/actions_card_product.class.php +++ b/htdocs/product/canvas/product/actions_card_product.class.php @@ -36,25 +36,35 @@ class ActionsCardProduct extends Product //! Template container var $tpl = array(); + // List of fiels for action=list + var $field_list =array(); + + /** - * \brief Constructeur de la classe - * \param DB Handler acces base de donnees - * \param id Id produit (0 par defaut) + * Constructor + * + * @param DoliDB $DB Handler acces base de donnees + * @param string $targmetmodule Name of directory of module where canvas is stored + * @param string $canvas Name of canvas + * @param string $card Name of tab (sub-canvas) */ - function ActionsCardProduct($DB=0, $id=0, $user=0) + function ActionsCardIndividual($DB,$targetmodule,$canvas,$card) { $this->db = $DB; - $this->id = $id ; - $this->user = $user; - $this->module = "produit"; - $this->canvas = "default"; - $this->name = "default"; + $this->targetmodule = $targetmodule; + $this->canvas = $canvas; + $this->card = $card; + + $this->name = "product"; $this->definition = "Product canvas (défaut)"; $this->fieldListName = "product_default"; $this->next_prev_filter = "canvas='default'"; } + /** + * Return the title of card + */ function getTitle() { global $langs; @@ -76,7 +86,7 @@ class ActionsCardProduct extends Product /** * Assign custom values for canvas (for example into this->tpl to be used by templates) * - * @param action Type of action + * @param action Type of action */ function assign_values($action='') { @@ -179,8 +189,65 @@ class ActionsCardProduct extends Product $this->tpl['fiche_end']=dol_get_fiche_end(); } + + if ($action == 'list') + { + $this->LoadListDatas($GLOBALS['limit'], $GLOBALS['offset'], $GLOBALS['sortfield'], $GLOBALS['sortorder']); + } + } + + /** + * Fetch field list + */ + function getFieldList() + { + global $conf, $langs; + + $this->field_list = array(); + + $sql = "SELECT rowid, name, alias, title, align, sort, search, enabled, rang"; + $sql.= " FROM ".MAIN_DB_PREFIX."c_field_list"; + $sql.= " WHERE element = '".$this->fieldListName."'"; + $sql.= " AND entity = ".$conf->entity; + $sql.= " ORDER BY rang ASC"; + + $resql = $this->db->query($sql); + if ($resql) + { + $num = $this->db->num_rows($resql); + + $i = 0; + while ($i < $num) + { + $fieldlist = array(); + + $obj = $this->db->fetch_object($resql); + + $fieldlist["id"] = $obj->rowid; + $fieldlist["name"] = $obj->name; + $fieldlist["alias"] = $obj->alias; + $fieldlist["title"] = $langs->trans($obj->title); + $fieldlist["align"] = $obj->align; + $fieldlist["sort"] = $obj->sort; + $fieldlist["search"] = $obj->search; + $fieldlist["enabled"] = verifCond($obj->enabled); + $fieldlist["order"] = $obj->rang; + + array_push($this->field_list,$fieldlist); + + $i++; + } + $this->db->free($resql); + } + else + { + dol_print_error($db,$sql); + } + } + + /** * Fetch datas list */ @@ -188,7 +255,9 @@ class ActionsCardProduct extends Product { global $conf, $langs; - $this->list_datas = array(); + $this->getFieldList(); + + $this->list_datas = array(); //$_GET["sall"] = 'LL'; // Clean parameters @@ -256,7 +325,7 @@ class ActionsCardProduct extends Product $sql.= $this->db->order($sortfield,$sortorder); $sql.= $this->db->plimit($limit + 1 ,$offset); //print $sql; - + $resql = $this->db->query($sql); if ($resql) { diff --git a/htdocs/product/canvas/service/actions_card_service.class.php b/htdocs/product/canvas/service/actions_card_service.class.php index 41adaee95ef..5b162b43891 100755 --- a/htdocs/product/canvas/service/actions_card_service.class.php +++ b/htdocs/product/canvas/service/actions_card_service.class.php @@ -36,28 +36,40 @@ class ActionsCardService extends Product //! Template container var $tpl = array(); + // List of fiels for action=list + var $field_list =array(); + + /** - * \brief Constructeur de la classe - * \param DB Handler acces base de donnees - * \param id Id service (0 par defaut) + * Constructor + * + * @param DoliDB $DB Handler acces base de donnees + * @param string $targmetmodule Name of directory of module where canvas is stored + * @param string $canvas Name of canvas + * @param string $card Name of tab (sub-canvas) */ - function ActionsCardService($DB=0, $id=0, $user=0) + function ActionsCardIndividual($DB,$targetmodule,$canvas,$card) { $this->db = $DB; - $this->id = $id ; - $this->user = $user; + $this->targetmodule = $targetmodule; + $this->canvas = $canvas; + $this->card = $card; + $this->module = "service"; - $this->canvas = "service"; $this->name = "service"; $this->definition = "Services canvas"; $this->fieldListName = "product_service"; - $this->next_prev_filter = "canvas='service'"; } + /** + * Return the title of card + */ function getTitle() { - return 'Services'; + global $langs; + + return $langs->trans("Products"); } /** @@ -72,8 +84,9 @@ class ActionsCardService extends Product } /** - * \brief Assigne les valeurs pour les templates - * \param object object + * Assign custom values for canvas (for example into this->tpl to be used by templates) + * + * @param action Type of action */ function assign_values($action='') { @@ -156,6 +169,62 @@ class ActionsCardService extends Product $this->tpl['fiche_end']=dol_get_fiche_end(); } + + if ($action == 'list') + { + $this->LoadListDatas($GLOBALS['limit'], $GLOBALS['offset'], $GLOBALS['sortfield'], $GLOBALS['sortorder']); + } + + } + + + /** + * Fetch field list + */ + private function getFieldList() + { + global $conf, $langs; + + $this->field_list = array(); + + $sql = "SELECT rowid, name, alias, title, align, sort, search, enabled, rang"; + $sql.= " FROM ".MAIN_DB_PREFIX."c_field_list"; + $sql.= " WHERE element = '".$this->fieldListName."'"; + $sql.= " AND entity = ".$conf->entity; + $sql.= " ORDER BY rang ASC"; + + $resql = $this->db->query($sql); + if ($resql) + { + $num = $this->db->num_rows($resql); + + $i = 0; + while ($i < $num) + { + $fieldlist = array(); + + $obj = $this->db->fetch_object($resql); + + $fieldlist["id"] = $obj->rowid; + $fieldlist["name"] = $obj->name; + $fieldlist["alias"] = $obj->alias; + $fieldlist["title"] = $langs->trans($obj->title); + $fieldlist["align"] = $obj->align; + $fieldlist["sort"] = $obj->sort; + $fieldlist["search"] = $obj->search; + $fieldlist["enabled"] = verifCond($obj->enabled); + $fieldlist["order"] = $obj->rang; + + array_push($this->field_list,$fieldlist); + + $i++; + } + $this->db->free($resql); + } + else + { + dol_print_error($db,$sql); + } } /** @@ -165,6 +234,8 @@ class ActionsCardService extends Product { global $conf; + $this->getFieldList(); + $sql = 'SELECT DISTINCT p.rowid, p.ref, p.label, p.barcode, p.price, p.price_ttc, p.price_base_type,'; $sql.= ' p.fk_product_type, p.tms as datem,'; $sql.= ' p.duration, p.tosell as statut, p.seuil_stock_alerte'; diff --git a/htdocs/product/liste.php b/htdocs/product/liste.php index e02cabf7660..de8347ea424 100644 --- a/htdocs/product/liste.php +++ b/htdocs/product/liste.php @@ -105,15 +105,10 @@ if ($conf->categorie->enabled && GETPOST('catid')) $htmlother=new FormOther($db); $html=new Form($db); -if (! empty($objcanvas->template_dir)) +if (is_object($objcanvas) && $objcanvas->displayCanvasExists($action)) { - $classname = 'ActionsCard'.ucfirst($canvas); - include_once(DOL_DOCUMENT_ROOT.'/product/canvas/'.$canvas.'/actions_card_'.$canvas.'.class.php'); - - $object = new $classname($db); - $object->getFieldList(); - $object->LoadListDatas($limit, $offset, $sortfield, $sortorder); - $title = $object->getTitle(); + $objcanvas->assign_values('list'); // This must contains code to load data (must call LoadListDatas($limit, $offset, $sortfield, $sortorder)) + $objcanvas->display_canvas('list'); // This is code to show template } else { @@ -134,346 +129,346 @@ else { $texte = $langs->trans("ProductsAndServices"); } -} -$sql = 'SELECT DISTINCT p.rowid, p.ref, p.label, p.barcode, p.price, p.price_ttc, p.price_base_type,'; -$sql.= ' p.fk_product_type, p.tms as datem,'; -$sql.= ' p.duration, p.tosell, p.tobuy, p.seuil_stock_alerte,'; -$sql.= ' MIN(pfp.unitprice) as minsellprice'; -$sql.= ' FROM ('.MAIN_DB_PREFIX.'product as p'; -// We'll need this table joined to the select in order to filter by categ -if ($search_categ) $sql.= ", ".MAIN_DB_PREFIX."categorie_product as cp"; -$sql.= ') '; -//if ($fourn_id > 0) // The DISTINCT is used to avoid duplicate from this link -//{ - $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."product_fournisseur as pf ON p.rowid = pf.fk_product"; - $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."product_fournisseur_price as pfp ON pf.rowid = pfp.fk_product_fournisseur"; -//} -$sql.= ' WHERE p.entity IN (0,'.(! empty($conf->entities['product']) ? $conf->entities['product'] : $conf->entity).')'; -if ($search_categ) $sql.= " AND p.rowid = cp.fk_product"; // Join for the needed table to filter by categ -if ($sall) -{ - $sql.= " AND (p.ref LIKE '%".$db->escape($sall)."%' OR p.label LIKE '%".$db->escape($sall)."%' OR p.description LIKE '%".$db->escape($sall)."%' OR p.note LIKE '%".$db->escape($sall)."%')"; -} -# if the type is not 1, we show all products (type = 0,2,3) -if (dol_strlen($type)) -{ - if ($type == 1) $sql.= " AND p.fk_product_type = '1'"; - else $sql.= " AND p.fk_product_type <> '1'"; -} -if ($sref) $sql.= " AND p.ref like '%".$sref."%'"; -if ($sbarcode) $sql.= " AND p.barcode like '%".$sbarcode."%'"; -if ($snom) $sql.= " AND p.label like '%".$db->escape($snom)."%'"; -if (isset($tosell) && dol_strlen($tosell) > 0) $sql.= " AND p.tosell = ".$db->escape($tosell); -if (isset($tobuy) && dol_strlen($tobuy) > 0) $sql.= " AND p.tobuy = ".$db->escape($tobuy); -if (dol_strlen($canvas) > 0) $sql.= " AND p.canvas = '".$db->escape($canvas)."'"; -if ($catid) $sql.= " AND cp.fk_categorie = ".$catid; -if ($search_categ) $sql.= " AND cp.fk_categorie = ".$search_categ; -if ($fourn_id > 0) $sql.= " AND pf.fk_soc = ".$fourn_id; -$sql.= " GROUP BY p.rowid, p.ref, p.label, p.barcode, p.price, p.price_ttc, p.price_base_type,"; -$sql.= " p.fk_product_type, p.tms,"; -$sql.= " p.duration, p.tosell, p.tobuy, p.seuil_stock_alerte"; -//if (GETPOST("toolowstock")) $sql.= " HAVING SUM(s.reel) < p.seuil_stock_alerte"; // Not used yet -$sql.= $db->order($sortfield,$sortorder); -$sql.= $db->plimit($limit + 1 ,$offset); + $sql = 'SELECT DISTINCT p.rowid, p.ref, p.label, p.barcode, p.price, p.price_ttc, p.price_base_type,'; + $sql.= ' p.fk_product_type, p.tms as datem,'; + $sql.= ' p.duration, p.tosell, p.tobuy, p.seuil_stock_alerte,'; + $sql.= ' MIN(pfp.unitprice) as minsellprice'; + $sql.= ' FROM ('.MAIN_DB_PREFIX.'product as p'; + // We'll need this table joined to the select in order to filter by categ + if ($search_categ) $sql.= ", ".MAIN_DB_PREFIX."categorie_product as cp"; + $sql.= ') '; + //if ($fourn_id > 0) // The DISTINCT is used to avoid duplicate from this link + //{ + $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."product_fournisseur as pf ON p.rowid = pf.fk_product"; + $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."product_fournisseur_price as pfp ON pf.rowid = pfp.fk_product_fournisseur"; + //} + $sql.= ' WHERE p.entity IN (0,'.(! empty($conf->entities['product']) ? $conf->entities['product'] : $conf->entity).')'; + if ($search_categ) $sql.= " AND p.rowid = cp.fk_product"; // Join for the needed table to filter by categ + if ($sall) + { + $sql.= " AND (p.ref LIKE '%".$db->escape($sall)."%' OR p.label LIKE '%".$db->escape($sall)."%' OR p.description LIKE '%".$db->escape($sall)."%' OR p.note LIKE '%".$db->escape($sall)."%')"; + } + # if the type is not 1, we show all products (type = 0,2,3) + if (dol_strlen($type)) + { + if ($type == 1) $sql.= " AND p.fk_product_type = '1'"; + else $sql.= " AND p.fk_product_type <> '1'"; + } + if ($sref) $sql.= " AND p.ref like '%".$sref."%'"; + if ($sbarcode) $sql.= " AND p.barcode like '%".$sbarcode."%'"; + if ($snom) $sql.= " AND p.label like '%".$db->escape($snom)."%'"; + if (isset($tosell) && dol_strlen($tosell) > 0) $sql.= " AND p.tosell = ".$db->escape($tosell); + if (isset($tobuy) && dol_strlen($tobuy) > 0) $sql.= " AND p.tobuy = ".$db->escape($tobuy); + if (dol_strlen($canvas) > 0) $sql.= " AND p.canvas = '".$db->escape($canvas)."'"; + if ($catid) $sql.= " AND cp.fk_categorie = ".$catid; + if ($search_categ) $sql.= " AND cp.fk_categorie = ".$search_categ; + if ($fourn_id > 0) $sql.= " AND pf.fk_soc = ".$fourn_id; + $sql.= " GROUP BY p.rowid, p.ref, p.label, p.barcode, p.price, p.price_ttc, p.price_base_type,"; + $sql.= " p.fk_product_type, p.tms,"; + $sql.= " p.duration, p.tosell, p.tobuy, p.seuil_stock_alerte"; + //if (GETPOST("toolowstock")) $sql.= " HAVING SUM(s.reel) < p.seuil_stock_alerte"; // Not used yet + $sql.= $db->order($sortfield,$sortorder); + $sql.= $db->plimit($limit + 1 ,$offset); -dol_syslog("sql=".$sql); -$resql = $db->query($sql) ; -if ($resql) -{ - $num = $db->num_rows($resql); + dol_syslog("sql=".$sql); + $resql = $db->query($sql) ; + if ($resql) + { + $num = $db->num_rows($resql); - $i = 0; + $i = 0; - if ($num == 1 && ($sall || $snom || $sref || $sbarcode) && $action != 'list') - { - $objp = $db->fetch_object($resql); - Header("Location: fiche.php?id=".$objp->rowid); - exit; - } + if ($num == 1 && ($sall || $snom || $sref || $sbarcode) && $action != 'list') + { + $objp = $db->fetch_object($resql); + Header("Location: fiche.php?id=".$objp->rowid); + exit; + } - $helpurl=''; - if (isset($_GET["type"]) && $_GET["type"] == 0) - { - $helpurl='EN:Module_Products|FR:Module_Produits|ES:Módulo_Productos'; - } - if (isset($_GET["type"]) && $_GET["type"] == 1) - { - $helpurl='EN:Module_Services_En|FR:Module_Services|ES:Módulo_Servicios'; - } + $helpurl=''; + if (isset($_GET["type"]) && $_GET["type"] == 0) + { + $helpurl='EN:Module_Products|FR:Module_Produits|ES:Módulo_Productos'; + } + if (isset($_GET["type"]) && $_GET["type"] == 1) + { + $helpurl='EN:Module_Services_En|FR:Module_Services|ES:Módulo_Servicios'; + } - llxHeader('',$title,$helpurl,''); + llxHeader('',$title,$helpurl,''); - // Displays product removal confirmation - if (GETPOST('delprod')) dol_htmloutput_mesg($langs->trans("ProductDeleted",GETPOST('delprod'))); + // Displays product removal confirmation + if (GETPOST('delprod')) dol_htmloutput_mesg($langs->trans("ProductDeleted",GETPOST('delprod'))); - $param="&sref=".$sref.($sbarcode?"&sbarcode=".$sbarcode:"")."&snom=".$snom."&sall=".$sall."&tosell=".$tosell."&tobuy=".$tobuy; - $param.=($fourn_id?"&fourn_id=".$fourn_id:""); - $param.=isset($type)?"&type=".$type:""; - print_barre_liste($texte, $page, "liste.php", $param, $sortfield, $sortorder,'',$num); + $param="&sref=".$sref.($sbarcode?"&sbarcode=".$sbarcode:"")."&snom=".$snom."&sall=".$sall."&tosell=".$tosell."&tobuy=".$tobuy; + $param.=($fourn_id?"&fourn_id=".$fourn_id:""); + $param.=isset($type)?"&type=".$type:""; + print_barre_liste($texte, $page, "liste.php", $param, $sortfield, $sortorder,'',$num); - if (isset($catid)) - { - print "
"; - $c = new Categorie ($db, $catid); - $ways = $c->print_all_ways(' > ','product/liste.php'); - print " > ".$ways[0]."
\n"; - print "

"; - } + if (isset($catid)) + { + print "
"; + $c = new Categorie ($db, $catid); + $ways = $c->print_all_ways(' > ','product/liste.php'); + print " > ".$ways[0]."
\n"; + print "

"; + } - if (!empty($_GET["canvas"]) && file_exists(DOL_DOCUMENT_ROOT.'/product/canvas/'.$_GET["canvas"].'/actions_card_'.$_GET["canvas"].'.class.php')) - { - $fieldlist = $object->field_list; - $datas = $object->list_datas; - $picto='title.png'; - if (empty($conf->browser->firefox)) $picto='title.gif'; - $title_picto = img_picto('',$picto); - $title_text = $title; + if (!empty($_GET["canvas"]) && file_exists(DOL_DOCUMENT_ROOT.'/product/canvas/'.$_GET["canvas"].'/actions_card_'.$_GET["canvas"].'.class.php')) + { + $fieldlist = $object->field_list; + $datas = $object->list_datas; + $picto='title.png'; + if (empty($conf->browser->firefox)) $picto='title.gif'; + $title_picto = img_picto('',$picto); + $title_text = $title; - // Default templates directory - $template_dir = DOL_DOCUMENT_ROOT . '/product/canvas/'.$_GET["canvas"].'/tpl/'; - // Check if a custom template is present - if (file_exists(DOL_DOCUMENT_ROOT . '/theme/'.$conf->theme.'/tpl/product/'.$_GET["canvas"].'/list.tpl.php')) - { - $template_dir = DOL_DOCUMENT_ROOT . '/theme/'.$conf->theme.'/tpl/product/'.$_GET["canvas"].'/'; - } + // Default templates directory + $template_dir = DOL_DOCUMENT_ROOT . '/product/canvas/'.$_GET["canvas"].'/tpl/'; + // Check if a custom template is present + if (file_exists(DOL_DOCUMENT_ROOT . '/theme/'.$conf->theme.'/tpl/product/'.$_GET["canvas"].'/list.tpl.php')) + { + $template_dir = DOL_DOCUMENT_ROOT . '/theme/'.$conf->theme.'/tpl/product/'.$_GET["canvas"].'/'; + } - include($template_dir.'list.tpl.php'); // Include native PHP templates - } - else - { - print '
'; - print ''; - print ''; - print ''; - print ''; - print ''; + include($template_dir.'list.tpl.php'); // Include native PHP templates + } + else + { + print ''; + print ''; + print ''; + print ''; + print ''; + print ''; - print ''; + print '
'; - // Filter on categories - $moreforfilter=''; - if ($conf->categorie->enabled) - { - $moreforfilter.=$langs->trans('Categories'). ': '; - $moreforfilter.=$htmlother->select_categories(0,$search_categ,'search_categ'); - $moreforfilter.='       '; - } - if ($moreforfilter) - { - print ''; - print ''; - } + // Filter on categories + $moreforfilter=''; + if ($conf->categorie->enabled) + { + $moreforfilter.=$langs->trans('Categories'). ': '; + $moreforfilter.=$htmlother->select_categories(0,$search_categ,'search_categ'); + $moreforfilter.='       '; + } + if ($moreforfilter) + { + print ''; + print ''; + } - // Lignes des titres - print ""; - print_liste_field_titre($langs->trans("Ref"), $_SERVER["PHP_SELF"], "p.ref",$param,"","",$sortfield,$sortorder); - print_liste_field_titre($langs->trans("Label"), $_SERVER["PHP_SELF"], "p.label",$param,"","",$sortfield,$sortorder); - if ($conf->barcode->enabled) print_liste_field_titre($langs->trans("BarCode"), $_SERVER["PHP_SELF"], "p.barcode",$param,"","",$sortfield,$sortorder); - print_liste_field_titre($langs->trans("DateModification"), $_SERVER["PHP_SELF"], "p.tms",$param,"",'align="center"',$sortfield,$sortorder); - if ($conf->service->enabled && $type != 0) print_liste_field_titre($langs->trans("Duration"), $_SERVER["PHP_SELF"], "p.duration",$param,"",'align="center"',$sortfield,$sortorder); - if (empty($conf->global->PRODUIT_MULTIPRICES)) print_liste_field_titre($langs->trans("SellingPrice"), $_SERVER["PHP_SELF"], "p.price",$param,"",'align="right"',$sortfield,$sortorder); - print ''; - if ($conf->stock->enabled && $user->rights->stock->lire && $type != 1) print ''; - print_liste_field_titre($langs->trans("Sell"), $_SERVER["PHP_SELF"], "p.tosell",$param,"",'align="right"',$sortfield,$sortorder); - print_liste_field_titre($langs->trans("Buy"), $_SERVER["PHP_SELF"], "p.tobuy",$param,"",'align="right"',$sortfield,$sortorder); - print "\n"; + // Lignes des titres + print ""; + print_liste_field_titre($langs->trans("Ref"), $_SERVER["PHP_SELF"], "p.ref",$param,"","",$sortfield,$sortorder); + print_liste_field_titre($langs->trans("Label"), $_SERVER["PHP_SELF"], "p.label",$param,"","",$sortfield,$sortorder); + if ($conf->barcode->enabled) print_liste_field_titre($langs->trans("BarCode"), $_SERVER["PHP_SELF"], "p.barcode",$param,"","",$sortfield,$sortorder); + print_liste_field_titre($langs->trans("DateModification"), $_SERVER["PHP_SELF"], "p.tms",$param,"",'align="center"',$sortfield,$sortorder); + if ($conf->service->enabled && $type != 0) print_liste_field_titre($langs->trans("Duration"), $_SERVER["PHP_SELF"], "p.duration",$param,"",'align="center"',$sortfield,$sortorder); + if (empty($conf->global->PRODUIT_MULTIPRICES)) print_liste_field_titre($langs->trans("SellingPrice"), $_SERVER["PHP_SELF"], "p.price",$param,"",'align="right"',$sortfield,$sortorder); + print ''; + if ($conf->stock->enabled && $user->rights->stock->lire && $type != 1) print ''; + print_liste_field_titre($langs->trans("Sell"), $_SERVER["PHP_SELF"], "p.tosell",$param,"",'align="right"',$sortfield,$sortorder); + print_liste_field_titre($langs->trans("Buy"), $_SERVER["PHP_SELF"], "p.tobuy",$param,"",'align="right"',$sortfield,$sortorder); + print "\n"; - // Lignes des champs de filtre - print ''; - print ''; - print ''; - if ($conf->barcode->enabled) - { - print ''; - } - print ''; - - // Duration - if ($conf->service->enabled && $type != 0) - { - print ''; - } - - // Sell price - if (empty($conf->global->PRODUIT_MULTIPRICES)) - { + // Lignes des champs de filtre + print ''; + print ''; + print ''; + if ($conf->barcode->enabled) + { + print ''; + } print ''; - } - // Minimum buying Price - print ''; - - // Stock - if ($conf->stock->enabled && $user->rights->stock->lire && $type != 1) - { - print ''; - } - - print ''; - - print ''; - print ''; - - - $product_static=new Product($db); - $product_fourn =new ProductFournisseur($db); - - $var=true; - while ($i < min($num,$limit)) - { - $objp = $db->fetch_object($resql); - - // Multilangs - if ($conf->global->MAIN_MULTILANGS) // si l'option est active - { - $sql = "SELECT label"; - $sql.= " FROM ".MAIN_DB_PREFIX."product_lang"; - $sql.= " WHERE fk_product=".$objp->rowid; - $sql.= " AND lang='". $langs->getDefaultLang() ."'"; - $sql.= " LIMIT 1"; - - $result = $db->query($sql); - if ($result) - { - $objtp = $db->fetch_object($result); - if ($objtp->label != '') $objp->label = $objtp->label; - } - } - - $var=!$var; - print ''; - - // Ref - print '\n"; - - // Label - print ''; - - // Barcode - if ($conf->barcode->enabled) - { - print ''; - } - - // Date - print '\n"; - - // Duration - if ($conf->service->enabled && $type != 0) - { - print ''; - } - - // Sell price - if (empty($conf->global->PRODUIT_MULTIPRICES)) - { - print ''; - } + } - // Better buy price - print ''; } + + // Minimum buying Price + print ''; + + // Stock + if ($conf->stock->enabled && $user->rights->stock->lire && $type != 1) + { + print ''; + } + + print ''; - // Show stock - if ($conf->stock->enabled && $user->rights->stock->lire && $type != 1) - { - if ($objp->fk_product_type != 1) - { - $product_static->id = $objp->rowid; - $product_static->load_stock(); - print ''; - } - else - { - print ''; - } - } + print ''; + print ''; - // Status (to buy) - print ''; - // Status (to sell) - print ''; + $product_static=new Product($db); + $product_fourn =new ProductFournisseur($db); - print "\n"; - $i++; - } + $var=true; + while ($i < min($num,$limit)) + { + $objp = $db->fetch_object($resql); - if ($num > $conf->liste_limit) - { - if ($sref || $snom || $sall || $sbarcode || $_POST["search"]) - { - print_barre_liste('', $page, "liste.php", "&sref=".$sref."&snom=".$snom."&sall=".$sall."&tosell=".$tosell."&tobuy=".$tobuy, $sortfield, $sortorder,'',$num); - } - else - { - print_barre_liste('', $page, "liste.php", "&sref=$sref&snom=$snom&fourn_id=$fourn_id".(isset($type)?"&type=$type":"")."&tosell=".$tosell."&tobuy=".$tobuy, $sortfield, $sortorder,'',$num); - } - } + // Multilangs + if ($conf->global->MAIN_MULTILANGS) // si l'option est active + { + $sql = "SELECT label"; + $sql.= " FROM ".MAIN_DB_PREFIX."product_lang"; + $sql.= " WHERE fk_product=".$objp->rowid; + $sql.= " AND lang='". $langs->getDefaultLang() ."'"; + $sql.= " LIMIT 1"; - $db->free($resql); + $result = $db->query($sql); + if ($result) + { + $objtp = $db->fetch_object($result); + if ($objtp->label != '') $objp->label = $objtp->label; + } + } - print "
'; - print $moreforfilter; - print '
'; + print $moreforfilter; + print '
'.$langs->trans("BuyingPriceMinShort").''.$langs->trans("PhysicalStock").'
'.$langs->trans("BuyingPriceMinShort").''.$langs->trans("PhysicalStock").'
'; - print ''; - print ''; - print ''; - print ''; - print ''; - print ''; - print ' '; - print ''; - print ' '; - print '
'; + print ''; + print ''; + print ''; + print ''; + print ''; + print ''; print ' '; print ''; - print ' '; - print ''; - print ' '; - print ''; - print ' '; - print ''; - print ''; - print ''; - print '
'; - $product_static->id = $objp->rowid; - $product_static->ref = $objp->ref; - $product_static->type = $objp->fk_product_type; - print $product_static->getNomUrl(1,'',24); - print "'.dol_trunc($objp->label,40).''.$objp->barcode.''.dol_print_date($db->jdate($objp->datem),'day')."'; - if (preg_match('/([0-9]+)y/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationYear"); - elseif (preg_match('/([0-9]+)m/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationMonth"); - elseif (preg_match('/([0-9]+)w/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationWeek"); - elseif (preg_match('/([0-9]+)d/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationDay"); - else print $objp->duration; - print ''; - if ($objp->price_base_type == 'TTC') print price($objp->price_ttc).' '.$langs->trans("TTC"); - else print price($objp->price).' '.$langs->trans("HT"); + // Duration + if ($conf->service->enabled && $type != 0) + { + print ''; + print ' '; print ''; - if ($objp->minsellprice != '') + // Sell price + if (empty($conf->global->PRODUIT_MULTIPRICES)) { - //print price($objp->minsellprice).' '.$langs->trans("HT"); - if ($product_fourn->find_min_price_product_fournisseur($objp->rowid) > 0) - { - if ($product_fourn->product_fourn_price_id > 0) - { - $htmltext=$product_fourn->display_price_product_fournisseur(); - if ($conf->fournisseur->enabled && $user->rights->fournisseur->lire) print $html->textwithpicto(price($product_fourn->fourn_unitprice).' '.$langs->trans("HT"),$htmltext); - else print price($product_fourn->fourn_unitprice).' '.$langs->trans("HT"); - } - } + print ''; + print ' '; + print ''; + print ' '; + print ''; + print ' '; + print ''; + print ' '; print ''; - if ($product_static->stock_reel < $objp->seuil_stock_alerte) print img_warning($langs->trans("StockTooLow")).' '; - print $product_static->stock_reel; - print ' '; + print ''; + print ''; + print '
'.$product_static->LibStatut($objp->tosell,5,0).''.$product_static->LibStatut($objp->tobuy,5,1).'
"; - print '
'; - } -} -else -{ - dol_print_error($db); + $var=!$var; + print ''; + + // Ref + print ''; + $product_static->id = $objp->rowid; + $product_static->ref = $objp->ref; + $product_static->type = $objp->fk_product_type; + print $product_static->getNomUrl(1,'',24); + print "\n"; + + // Label + print ''.dol_trunc($objp->label,40).''; + + // Barcode + if ($conf->barcode->enabled) + { + print ''.$objp->barcode.''; + } + + // Date + print ''.dol_print_date($db->jdate($objp->datem),'day')."\n"; + + // Duration + if ($conf->service->enabled && $type != 0) + { + print ''; + if (preg_match('/([0-9]+)y/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationYear"); + elseif (preg_match('/([0-9]+)m/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationMonth"); + elseif (preg_match('/([0-9]+)w/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationWeek"); + elseif (preg_match('/([0-9]+)d/i',$objp->duration,$regs)) print $regs[1].' '.$langs->trans("DurationDay"); + else print $objp->duration; + print ''; + } + + // Sell price + if (empty($conf->global->PRODUIT_MULTIPRICES)) + { + print ''; + if ($objp->price_base_type == 'TTC') print price($objp->price_ttc).' '.$langs->trans("TTC"); + else print price($objp->price).' '.$langs->trans("HT"); + print ''; + } + + // Better buy price + print ''; + if ($objp->minsellprice != '') + { + //print price($objp->minsellprice).' '.$langs->trans("HT"); + if ($product_fourn->find_min_price_product_fournisseur($objp->rowid) > 0) + { + if ($product_fourn->product_fourn_price_id > 0) + { + $htmltext=$product_fourn->display_price_product_fournisseur(); + if ($conf->fournisseur->enabled && $user->rights->fournisseur->lire) print $html->textwithpicto(price($product_fourn->fourn_unitprice).' '.$langs->trans("HT"),$htmltext); + else print price($product_fourn->fourn_unitprice).' '.$langs->trans("HT"); + } + } + } + print ''; + + // Show stock + if ($conf->stock->enabled && $user->rights->stock->lire && $type != 1) + { + if ($objp->fk_product_type != 1) + { + $product_static->id = $objp->rowid; + $product_static->load_stock(); + print ''; + if ($product_static->stock_reel < $objp->seuil_stock_alerte) print img_warning($langs->trans("StockTooLow")).' '; + print $product_static->stock_reel; + print ''; + } + else + { + print ' '; + } + } + + // Status (to buy) + print ''.$product_static->LibStatut($objp->tosell,5,0).''; + + // Status (to sell) + print ''.$product_static->LibStatut($objp->tobuy,5,1).''; + + print "\n"; + $i++; + } + + if ($num > $conf->liste_limit) + { + if ($sref || $snom || $sall || $sbarcode || $_POST["search"]) + { + print_barre_liste('', $page, "liste.php", "&sref=".$sref."&snom=".$snom."&sall=".$sall."&tosell=".$tosell."&tobuy=".$tobuy, $sortfield, $sortorder,'',$num); + } + else + { + print_barre_liste('', $page, "liste.php", "&sref=$sref&snom=$snom&fourn_id=$fourn_id".(isset($type)?"&type=$type":"")."&tosell=".$tosell."&tobuy=".$tobuy, $sortfield, $sortorder,'',$num); + } + } + + $db->free($resql); + + print ""; + print ''; + } + } + else + { + dol_print_error($db); + } } diff --git a/htdocs/societe/canvas/default/actions_card_default.class.php b/htdocs/societe/canvas/default/actions_card_default.class.php index 6d35fcb8330..839b88704ab 100644 --- a/htdocs/societe/canvas/default/actions_card_default.class.php +++ b/htdocs/societe/canvas/default/actions_card_default.class.php @@ -36,10 +36,11 @@ class ActionsCardDefault extends ActionsCardCommon /** * Constructor - * @param DB Handler acces base de donnees - * @param targmetmodule Name of directory of module where canvas is stored - * @param canvas Name of canvas - * @param card Name of tab (sub-canvas) + * + * @param DoliDB $DB Handler acces base de donnees + * @param string $targmetmodule Name of directory of module where canvas is stored + * @param string $canvas Name of canvas + * @param string $card Name of tab (sub-canvas) */ function ActionsCardDefault($DB,$targetmodule,$canvas,$card) { @@ -66,14 +67,6 @@ class ActionsCardDefault extends ActionsCardCommon } - /** - * Assigne les valeurs POST dans l'objet - */ - function assign_post() - { - parent::assign_post(); - } - /** * Execute actions * @param Id of object (may be empty for creation) diff --git a/htdocs/societe/canvas/individual/actions_card_individual.class.php b/htdocs/societe/canvas/individual/actions_card_individual.class.php index 5538c7d8744..f717eb7bd6a 100644 --- a/htdocs/societe/canvas/individual/actions_card_individual.class.php +++ b/htdocs/societe/canvas/individual/actions_card_individual.class.php @@ -34,11 +34,12 @@ class ActionsCardIndividual extends ActionsCardCommon var $card; /** - * Constructor - * @param DB Handler acces base de donnees - * @param targetmodule Name of directory of module where canvas is stored - * @param canvas Name of canvas - * @param card Name of tab (sub-canvas) + * Constructor + * + * @param DoliDB $DB Handler acces base de donnees + * @param string $targmetmodule Name of directory of module where canvas is stored + * @param string $canvas Name of canvas + * @param string $card Name of tab (sub-canvas) */ function ActionsCardIndividual($DB,$targetmodule,$canvas,$card) { @@ -66,14 +67,6 @@ class ActionsCardIndividual extends ActionsCardCommon } - /** - * Assigne les valeurs POST dans l'objet - */ - function assign_post() - { - parent::assign_post(); - } - /** * Execute actions * @param Id of object (may be empty for creation)