* QUAL use switch case instead of if elseif statements for actions
* Deleted unnecessary action comments
* Added default case
* Added contributor details
---------
Co-authored-by: William Mead <william.mead@manchenumerique.fr>
* NEW : When an user unset the batch management of products, transformation of each batch stock mouvement in global stock mouvement
* FIX : Alert on deserialize product
* FIX : Correction of the trad
* Fixing style errors.
* FIX : PR returns
* FIX : using llx_product_batch table
* FIX : langs
* Fixing style errors.
* FIX : $value must be an absolute value
* FIX : init $inventorycode before while loop
* Fixing style errors.
* FIX : "Yes (unique serial number required)" choice must not be available on product edit if its current status batch is "Yes (lot required)"
* Fixing style errors.
* FIX : Just display a message when we change from batch to serial
* Fixing style errors.
* Update card.php
* Update card.php
* Update product.class.php
* Update product.class.php
---------
Co-authored-by: Adrien Raze <adrien.raze@atm-consulting.fr>
Co-authored-by: stickler-ci <support@stickler-ci.com>
Co-authored-by: Laurent Destailleur <eldy@destailleur.fr>
* reception: reOpen: mirror $this->statut in $this->status
$this->statut is the deprecated variable, $this->status should also get
the correct value.
* reception: setDraft: mirror $this->statut in $this->status
$this->statut is the deprecated variable, $this->status should also get
the correct value.
* reception: add missing $weight field
Fix warning:
Undefined property: Reception::$weight
* societe: add country_id field
* reception: use getDolGlobalInt when suitable
Using empty() implies that the value actually exists on the stdClass at
$conf->global, but it's not always the case. getDolGlobalInt will handle
this smoothly by checking first, which solves warnings like those:
Undefined property: stdClass::$STOCK_CALCULATE_ON_RECEPTION
* ReceptionTest: add new test
The test checks the usual workflow of the Reception class, with, on the
one hand, the common CRUD operations:
- create
- fetch
- update
- delete
And on the other hand, the status handling for Reception:
- valid: STATUS_DRAFT -> STATUS_VALID
- setClosed: STATUS_VALID -> STATUS_CLOSED
- reOpen: STATUS_CLOSED -> STATUS_VALID
- setDraft: STATUS_VALID -> STATUS_DRAFT
The stocks lines are not tested yet, and the error cases, like any other
transition not described above, are not tested either. The permissions
for some of the operations are hardcoded for the test and there is no
failure check when the permission is not set yet.
* Define DoliDB::VERSIONMIN as mandatory constant in subclass
See https://stackoverflow.com/a/43134924/6378634
This also helps static tool checks
* fixup! Define DoliDB::VERSIONMIN as mandatory constant in subclass
