DMC/boka3
12
0
Fork 0
Code Issues13 Pull Requests Projects Releases Wiki Activity

Fixed a bug where input validation wasn't being done when adding new products

Browse Source
This commit is contained in:
Erik Thuning 2019-02-27 17:15:38 +01:00
parent 9ec0efd81d
commit ce40d5ddfd
1 changed files with 14 additions and 10 deletions

24
include/view.php

@ -939,6 +939,19 @@ class Ajax extends Responder {
foreach(array('id', 'name', 'serial', 'invoice', 'tags') as $key) {
unset($info[$key]);
}
if(!$name) {
return new Failure('Artikeln måste ha ett namn.');
}
if(!$serial) {
return new Failure('Artikeln måste ha ett serienummer.');
}
try {
$temp = new Product($serial, 'serial');
return new Failure('Det angivna serienumret finns redan på en annan artikel.');
} catch(Exception $e) {}
if(!$invoice) {
return new Failure('Artikeln måste ha ett fakturanummer.');
}
$product = null;
if(!$id) {
try {
@ -960,25 +973,16 @@ class Ajax extends Responder {
if($product->get_discardtime()) {
return new Failure('Skrotade artiklar får inte modifieras.');
}
if(!$name) {
return new Failure('Artikeln måste ha ett namn.');
}
if($name != $product->get_name()) {
$product->set_name($name);
}
if(!$serial) {
return new Failure('Artikeln måste ha ett serienummer.');
}
if($serial != $product->get_serial()) {
try {
$product->set_serial($serial);
} catch(Exception $e) {
return new Failure('Det angivna serienumret finns redan på en annan artikel.');
return new Failure("Serienumret upptaget. Det här meddelandet ska aldrig visas.");
}
}
if(!$invoice) {
return new Failure('Artikeln måste ha ett fakturanummer.');
}
if($invoice != $product->get_invoice()) {
$product->set_invoice($invoice);
}