rgutierrez/resq
1
0
Fork 0
Code Issues 126 Pull requests 49 Projects Releases 4 Packages Wiki Activity Actions

Add error_logging in case of duplicate SAP battery. #371

Browse source
This commit is contained in:
Korina Cordero 2020-04-01 04:30:08 +00:00
parent 506505bff5
commit 1829e9112e
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/Command/ImportSAPBatteryCommand.php
View file

@ -77,7 +77,7 @@ class ImportSAPBatteryCommand extends Command
// clean up fields // clean up fields
$clean_brand = trim($fields[0]); $clean_brand = trim($fields[0]);
$clean_sku = trim($fields[1]); $clean_sku = strtoupper(trim($fields[1]));
$clean_size = trim($fields[2]); $clean_size = trim($fields[2]);
$output->writeln("Parsing $clean_sku..."); $output->writeln("Parsing $clean_sku...");
@ -110,11 +110,17 @@ class ImportSAPBatteryCommand extends Command
// create battery entry // create battery entry
$battery = new SAPBattery(); $battery = new SAPBattery();
$battery->setID($clean_sku) $battery->setID($clean_sku)
->setSize($size_hash[$clean_size]) ->setSize($this->batt_size_hash[$clean_size])
->setBrand($brand_hash[$clean_brand]); ->setBrand($this->batt_brand_hash[$clean_brand]);
$em->persist($battery); $em->persist($battery);
} }
else
{
// TODO: different batteries but same SKU: WCHD23BL-CPN00-LX
// how to process?
error_log('Battery with that sku already exists.');
}
} }
$em->flush(); $em->flush();