Merge pull request 'Added rework_status value validation on importer' (#377) from ranjith-dev into master
Some checks failed
Scan for leaked secrets using Kingfisher / kingfisher-secrets-scan (push) Has been cancelled

Reviewed-on: #377
This commit was merged in pull request #377.
This commit is contained in:
2026-02-23 10:12:08 +00:00

View File

@@ -142,8 +142,12 @@ class ProcessOrderImporter extends Importer
if ($recQuan == null || $recQuan == '') { if ($recQuan == null || $recQuan == '') {
$recQuan = 0; $recQuan = 0;
} }
if ($reworkStatus == null || $reworkStatus = '') { if ($reworkStatus == null || $reworkStatus = '' || $reworkStatus == 0 || $reworkStatus = '0') {
$reworkStatus = 0; $reworkStatus = 0;
} elseif ($reworkStatus == 1 || $reworkStatus = '1') {
$reworkStatus = 1;
} else {
$warnMsg[] = 'Invalid rework status found';
} }
if (Str::length($plantCod) < 4 || ! is_numeric($plantCod) || ! preg_match('/^[1-9]\d{3,}$/', $plantCod)) { if (Str::length($plantCod) < 4 || ! is_numeric($plantCod) || ! preg_match('/^[1-9]\d{3,}$/', $plantCod)) {