Fix phpcs

This commit is contained in:
Laurent Destailleur 2018-12-15 14:11:59 +01:00
parent f55db6dd84
commit 4ff28d4580
2 changed files with 4 additions and 14 deletions

View File

@ -269,8 +269,6 @@ class Reception extends CommonObject
{ {
$error++; $error++;
} }
} }
if (! $error && $this->id && $this->origin_id) if (! $error && $this->id && $this->origin_id)
@ -322,7 +320,6 @@ class Reception extends CommonObject
$this->db->rollback(); $this->db->rollback();
return -1*$error; return -1*$error;
} }
} }
else else
{ {
@ -522,7 +519,6 @@ class Reception extends CommonObject
$soc->fetch($this->socid); $soc->fetch($this->socid);
// Define new ref // Define new ref
if (! $error && (preg_match('/^[\(]?PROV/i', $this->ref) || empty($this->ref))) // empty should not happened, but when it occurs, the test save life if (! $error && (preg_match('/^[\(]?PROV/i', $this->ref) || empty($this->ref))) // empty should not happened, but when it occurs, the test save life
{ {
@ -742,8 +738,6 @@ class Reception extends CommonObject
$this->error=$langs->trans("ErrorWarehouseRequiredIntoReceptionLine"); $this->error=$langs->trans("ErrorWarehouseRequiredIntoReceptionLine");
return -1; return -1;
} }
} }
// extrafields // extrafields
@ -799,7 +793,6 @@ class Reception extends CommonObject
if (isset($this->modelpdf)) $this->modelpdf=trim($this->modelpdf); if (isset($this->modelpdf)) $this->modelpdf=trim($this->modelpdf);
// Check parameters // Check parameters
// Put here code to add control on parameters values // Put here code to add control on parameters values
@ -881,7 +874,6 @@ class Reception extends CommonObject
$this->error=''; $this->error='';
$this->db->begin(); $this->db->begin();
// Stock control // Stock control
if ($conf->stock->enabled && $conf->global->STOCK_CALCULATE_ON_RECEPTION && $this->statut > 0) if ($conf->stock->enabled && $conf->global->STOCK_CALCULATE_ON_RECEPTION && $this->statut > 0)
@ -913,8 +905,6 @@ class Reception extends CommonObject
$result=$mouvS->livraison($user, $obj->fk_product, $obj->fk_entrepot, $obj->qty, 0, $langs->trans("ReceptionDeletedInDolibarr", $this->ref),'', $obj->eatby, $obj->sellby, $obj->batch); // Price is set to 0, because we don't want to see WAP changed $result=$mouvS->livraison($user, $obj->fk_product, $obj->fk_entrepot, $obj->qty, 0, $langs->trans("ReceptionDeletedInDolibarr", $this->ref),'', $obj->eatby, $obj->sellby, $obj->batch); // Price is set to 0, because we don't want to see WAP changed
} }
} }
else else
@ -1327,7 +1317,7 @@ class Reception extends CommonObject
/** /**
* Fetch all deliveries method and return an array. Load array this->listmeths. * Fetch all deliveries method and return an array. Load array this->listmeths.
* *
* @param id $id only this carrier, all if none * @param int $id only this carrier, all if none
* @return void * @return void
*/ */
function list_delivery_methods($id='') function list_delivery_methods($id='')
@ -1393,7 +1383,7 @@ class Reception extends CommonObject
/** /**
* Activate delivery method. * Activate delivery method.
* *
* @param id $id id method to activate * @param int $id id method to activate
* *
* @return void * @return void
*/ */
@ -1411,7 +1401,7 @@ class Reception extends CommonObject
/** /**
* DesActivate delivery method. * DesActivate delivery method.
* *
* @param id $id id method to desactivate * @param int $id id method to desactivate
* *
* @return void * @return void
*/ */