FIX : it was not possible to update extrafields of expedition lines with batch without editing batch value
This commit is contained in:
parent
d1bd12ab33
commit
538a6870f1
@ -581,6 +581,7 @@ if (empty($reshook)) {
|
|||||||
$num_prod = count($lines);
|
$num_prod = count($lines);
|
||||||
for ($i = 0; $i < $num_prod; $i++) {
|
for ($i = 0; $i < $num_prod; $i++) {
|
||||||
if ($lines[$i]->id == $line_id) { // we have found line to update
|
if ($lines[$i]->id == $line_id) { // we have found line to update
|
||||||
|
$update_done = false;
|
||||||
$line = new ExpeditionLigne($db);
|
$line = new ExpeditionLigne($db);
|
||||||
$line->fk_expedition = $object->id;
|
$line->fk_expedition = $object->id;
|
||||||
|
|
||||||
@ -621,6 +622,8 @@ if (empty($reshook)) {
|
|||||||
if ($line->update($user) < 0) {
|
if ($line->update($user) < 0) {
|
||||||
setEventMessages($line->error, $line->errors, 'errors');
|
setEventMessages($line->error, $line->errors, 'errors');
|
||||||
$error++;
|
$error++;
|
||||||
|
} else {
|
||||||
|
$update_done=true;
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
setEventMessages($lotStock->error, $lotStock->errors, 'errors');
|
setEventMessages($lotStock->error, $lotStock->errors, 'errors');
|
||||||
@ -663,6 +666,8 @@ if (empty($reshook)) {
|
|||||||
if ($line->update($user) < 0) {
|
if ($line->update($user) < 0) {
|
||||||
setEventMessages($line->error, $line->errors, 'errors');
|
setEventMessages($line->error, $line->errors, 'errors');
|
||||||
$error++;
|
$error++;
|
||||||
|
} else {
|
||||||
|
$update_done=true;
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
setEventMessages($line->error, $line->errors, 'errors');
|
setEventMessages($line->error, $line->errors, 'errors');
|
||||||
@ -680,6 +685,8 @@ if (empty($reshook)) {
|
|||||||
if ($object->create_line_batch($line, $line->array_options) < 0) {
|
if ($object->create_line_batch($line, $line->array_options) < 0) {
|
||||||
setEventMessages($object->error, $object->errors, 'errors');
|
setEventMessages($object->error, $object->errors, 'errors');
|
||||||
$error++;
|
$error++;
|
||||||
|
} else {
|
||||||
|
$update_done=true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
@ -717,6 +724,8 @@ if (empty($reshook)) {
|
|||||||
if ($line->update($user) < 0) {
|
if ($line->update($user) < 0) {
|
||||||
setEventMessages($line->error, $line->errors, 'errors');
|
setEventMessages($line->error, $line->errors, 'errors');
|
||||||
$error++;
|
$error++;
|
||||||
|
} else {
|
||||||
|
$update_done=true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
unset($_POST[$stockLocation]);
|
unset($_POST[$stockLocation]);
|
||||||
@ -731,6 +740,8 @@ if (empty($reshook)) {
|
|||||||
if ($line->update($user) < 0) {
|
if ($line->update($user) < 0) {
|
||||||
setEventMessages($line->error, $line->errors, 'errors');
|
setEventMessages($line->error, $line->errors, 'errors');
|
||||||
$error++;
|
$error++;
|
||||||
|
} else {
|
||||||
|
$update_done=true;
|
||||||
}
|
}
|
||||||
unset($_POST[$qty]);
|
unset($_POST[$qty]);
|
||||||
}
|
}
|
||||||
@ -743,10 +754,18 @@ if (empty($reshook)) {
|
|||||||
if ($line->update($user) < 0) {
|
if ($line->update($user) < 0) {
|
||||||
setEventMessages($line->error, $line->errors, 'errors');
|
setEventMessages($line->error, $line->errors, 'errors');
|
||||||
$error++;
|
$error++;
|
||||||
|
} else {
|
||||||
|
$update_done=true;
|
||||||
}
|
}
|
||||||
unset($_POST[$qty]);
|
unset($_POST[$qty]);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if(empty($update_done)) {
|
||||||
|
$line->id = $lines[$i]->id;
|
||||||
|
$line->insertExtraFields();
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user