diff --git a/htdocs/barcode/printsheet.php b/htdocs/barcode/printsheet.php index ba3c994fcf7ce..3415162c2604e 100644 --- a/htdocs/barcode/printsheet.php +++ b/htdocs/barcode/printsheet.php @@ -157,7 +157,7 @@ // Get encoder (barcode_type_coder) from barcode type id (barcode_type) $stdobject = new GenericObject($db); $stdobject->barcode_type = $fk_barcode_type; - $result = $stdobject->fetch_barcode(); + $result = $stdobject->fetchBarCode(); if ($result <= 0) { $error++; setEventMessages('Failed to get bar code type information '.$stdobject->error, $stdobject->errors, 'errors'); @@ -171,8 +171,8 @@ if (!$error && $stdobject !== null) { $code = $forbarcode; - $generator = $stdobject->barcode_type_coder; // coder (loaded by fetch_barcode). Engine. - $encoding = strtoupper($stdobject->barcode_type_code); // code (loaded by fetch_barcode). Example 'ean', 'isbn', ... + $generator = $stdobject->barcode_type_coder; // coder (loaded by fetchBarCode). Engine. + $encoding = strtoupper($stdobject->barcode_type_code); // code (loaded by fetchBarCode). Example 'ean', 'isbn', ... $diroutput = $conf->barcode->dir_temp; dol_mkdir($diroutput); diff --git a/htdocs/compta/facture/class/facture-rec.class.php b/htdocs/compta/facture/class/facture-rec.class.php index e0c28539c6b09..ea3d6f0ea68d5 100644 --- a/htdocs/compta/facture/class/facture-rec.class.php +++ b/htdocs/compta/facture/class/facture-rec.class.php @@ -203,8 +203,10 @@ class FactureRec extends CommonInvoice */ public $fk_societe_rib; + const PAYMENTCODETOEDITSOCIETERIB = "PRE"; + /** * 'type' if the field format ('integer', 'integer:ObjectClass:PathToClass[:AddCreateButtonOrNot[:Filter]]', 'varchar(x)', 'double(24,8)', 'real', 'price', 'text', 'html', 'date', 'datetime', 'timestamp', 'duration', 'mail', 'phone', 'url', 'password') * Note: Filter can be a string like "(t.ref:like:'SO-%') or (t.date_creation:<:'20160101') or (t.nature:is:NULL)" @@ -567,7 +569,7 @@ public function update(User $user, $notrigger = 0) $sql .= " localtax1 = ".((float) $this->total_localtax1).","; $sql .= " localtax2 = ".((float) $this->total_localtax2).","; $sql .= " total_ht = ".((float) $this->total_ht).","; - $sql .= " total_ttc = ".((float) $this->total_ttc); + $sql .= " total_ttc = ".((float) $this->total_ttc).","; $sql .= " fk_societe_rib = ".((int) $this->fk_societe_rib); // TODO Add missing fields $sql .= " WHERE rowid = ".((int) $this->id); diff --git a/htdocs/core/class/commonobject.class.php b/htdocs/core/class/commonobject.class.php index 620dbb57c6103..04d68e85489fd 100644 --- a/htdocs/core/class/commonobject.class.php +++ b/htdocs/core/class/commonobject.class.php @@ -425,25 +425,25 @@ abstract class CommonObject /** * @var int Barcode type - * @see fetch_barcode() + * @see fetchBarCode() */ public $barcode_type; /** * @var string Code of the barcode type - * @see fetch_barcode(), barcode_type + * @see fetchBarCode(), barcode_type */ public $barcode_type_code; /** * @var string Label of the barcode type - * @see fetch_barcode(), barcode_type + * @see fetchBarCode(), barcode_type */ public $barcode_type_label; /** * @var string - * @see fetch_barcode(), barcode_type + * @see fetchBarCode(), barcode_type */ public $barcode_type_coder; @@ -1631,7 +1631,7 @@ public function swapContactStatus($rowid) * @param int<0,1> $option 0=Return array id->label, 1=Return array code->label, 2=Return complete array * @param int<0,1> $activeonly 0=all status of contact, 1=only the active * @param string $code Type of contact (Example: 'CUSTOMER', 'SERVICE') - * @return array|array|array|null Array list of type of contacts (id->label if option=0, code->label if option=1), or null if error + * @return array|array|array|null Array list of type of contacts (id->label if option=0, code->label if option=1), or null if error */ public function liste_type_contact($source = 'internal', $order = 'position', $option = 0, $activeonly = 0, $code = '') { @@ -1702,7 +1702,7 @@ public function liste_type_contact($source = 'internal', $order = 'position', $o */ public function listeTypeContacts($source = 'internal', $option = 0, $activeonly = 0, $code = '', $element = '', $excludeelement = '') { - global $langs, $conf; + global $langs; $langs->loadLangs(array('bills', 'contracts', 'interventions', 'orders', 'projects', 'propal', 'ticket', 'agenda')); @@ -1784,8 +1784,6 @@ public function listeTypeContacts($source = 'internal', $option = 0, $activeonly */ public function getIdContact($source, $code, $status = 0) { - global $conf; - $result = array(); $i = 0; // Particular case for shipping @@ -1952,13 +1950,24 @@ public function fetchOneLike($ref) * if it is not defined, ->element must be defined to know default barcode type. * * @return int<-1,1> Return integer <0 if KO, 0 if can't guess type of barcode (ISBN, EAN13...), >0 if OK (all barcode properties loaded) + * @deprecated Use fetchBarCode() */ public function fetch_barcode() { // phpcs:enable - global $conf; + return fetchBarCode(); + } - dol_syslog(get_class($this).'::fetch_barcode this->element='.$this->element.' this->barcode_type='.$this->barcode_type); + /** + * Load data for barcode into properties ->barcode_type* + * Properties ->barcode_type that is id of barcode. Type is used to find other properties, but + * if it is not defined, ->element must be defined to know default barcode type. + * + * @return int<-1,1> Return integer <0 if KO, 0 if can't guess type of barcode (ISBN, EAN13...), >0 if OK (all barcode properties loaded) + */ + public function fetchBarCode() + { + dol_syslog(get_class($this).'::fetchBarCode this->element='.$this->element.' this->barcode_type='.$this->barcode_type); $idtype = $this->barcode_type; if (empty($idtype) && $idtype != '0') { // If type of barcode no set, we try to guess. If set to '0' it means we forced to have type remain not defined @@ -1967,7 +1976,7 @@ public function fetch_barcode() } elseif ($this->element == 'societe') { $idtype = getDolGlobalString('GENBARCODE_BARCODETYPE_THIRDPARTY'); } else { - dol_syslog('Call fetch_barcode with barcode_type not defined and cannot be guessed', LOG_WARNING); + dol_syslog('Call fetchBarCode with barcode_type not defined and cannot be guessed', LOG_WARNING); } } @@ -1976,10 +1985,11 @@ public function fetch_barcode() $sql = "SELECT rowid, code, libelle as label, coder"; $sql .= " FROM ".$this->db->prefix()."c_barcode_type"; $sql .= " WHERE rowid = ".((int) $idtype); - dol_syslog(get_class($this).'::fetch_barcode', LOG_DEBUG); + $resql = $this->db->query($sql); if ($resql) { $obj = $this->db->fetch_object($resql); + $this->barcode_type = $obj->rowid; $this->barcode_type_code = $obj->code; $this->barcode_type_label = $obj->label; diff --git a/htdocs/core/class/html.form.class.php b/htdocs/core/class/html.form.class.php index be5c3028b4e86..ab6f9fdf462c2 100644 --- a/htdocs/core/class/html.form.class.php +++ b/htdocs/core/class/html.form.class.php @@ -10290,8 +10290,8 @@ public function showbarcode(&$object, $width = 100, $morecss = '') // Complete object if not complete if (empty($object->barcode_type_code) || empty($object->barcode_type_coder)) { // @phan-suppress-next-line PhanPluginUnknownObjectMethodCall - $result = $object->fetch_barcode(); - //Check if fetch_barcode() failed + $result = $object->fetchBarCode(); + //Check if fetchBarCode() failed if ($result < 1) { return ''; } diff --git a/htdocs/product/card.php b/htdocs/product/card.php index b7e8cd562dfe4..6e9ebf57447ed 100644 --- a/htdocs/product/card.php +++ b/htdocs/product/card.php @@ -590,7 +590,7 @@ $stdobject = new GenericObject($db); $stdobject->element = 'product'; $stdobject->barcode_type = GETPOSTINT('fk_barcode_type'); - $result = $stdobject->fetch_barcode(); + $result = $stdobject->fetchBarCode(); if ($result < 0) { $error++; $mesg = 'Failed to get bar code type information '; @@ -835,7 +835,7 @@ $stdobject = new GenericObject($db); $stdobject->element = 'product'; $stdobject->barcode_type = GETPOSTINT('fk_barcode_type'); - $result = $stdobject->fetch_barcode(); + $result = $stdobject->fetchBarCode(); if ($result < 0) { $error++; $mesg = 'Failed to get bar code type information '; @@ -2555,7 +2555,7 @@ print $formbarcode->formBarcodeType($_SERVER['PHP_SELF'].'?id='.$object->id, $object->barcode_type, 'fk_barcode_type'); $fk_barcode_type = $object->barcode_type; } else { - $object->fetch_barcode(); + $object->fetchBarCode(); $fk_barcode_type = $object->barcode_type; print $object->barcode_type_label ? $object->barcode_type_label : ($object->barcode ? '
'.$langs->trans("SetDefaultBarcodeType").'
' : ''); } diff --git a/htdocs/product/price_suppliers.php b/htdocs/product/price_suppliers.php index fa1849b0192c6..d89ebed0e6e2a 100644 --- a/htdocs/product/price_suppliers.php +++ b/htdocs/product/price_suppliers.php @@ -1260,7 +1260,7 @@ function edit_price_from_multicurrency() { if (!empty($arrayfields['pfp.fk_barcode_type']['checked'])) { print ''; $productfourn->barcode_type = !empty($productfourn->supplier_fk_barcode_type) ? $productfourn->supplier_fk_barcode_type : 0; - $productfourn->fetch_barcode(); + $productfourn->fetchBarCode(); print $productfourn->barcode_type_label ? $productfourn->barcode_type_label : ($productfourn->supplier_barcode ? '
'.$langs->trans("SetDefaultBarcodeType").'
' : ''); print ''; }