From 0c912c012285512b2db6ce456debc741dc675141 Mon Sep 17 00:00:00 2001
From: bison <bison.paolo@gmail.com>
Date: Wed, 31 Aug 2016 10:39:26 +0200
Subject: [PATCH] Bug fix: update person data

---
 .../controllers/api/v1/person/Person.php      | 29 ++++++++++---------
 1 file changed, 16 insertions(+), 13 deletions(-)

diff --git a/application/controllers/api/v1/person/Person.php b/application/controllers/api/v1/person/Person.php
index 9699d9fb9..7ab6e0d2f 100644
--- a/application/controllers/api/v1/person/Person.php
+++ b/application/controllers/api/v1/person/Person.php
@@ -74,21 +74,24 @@ class Person extends APIv1_Controller
 		{
 			if(isset($person["person_id"]) && !(is_null($person["person_id"])) && ($person["person_id"] != ""))
 			{
-				$this->PersonModel->addOrder("svnr", "DESC");
-				$result =  $this->PersonModel->loadWhere(array(
-					"person_id != " => $person["person_id"],
-					"SUBSTRING(svnr FROM 1 FOR 10) = " => $person["svnr"])
-				);
-				if (is_object($result) && $result->error == EXIT_SUCCESS &&
-					is_array($result->retval) && count($result->retval) > 0)
+				if (isset($person["svnr"]) && $person["svnr"] != "")
 				{
-					if (count($result->retval) == 1)
+					$this->PersonModel->addOrder("svnr", "DESC");
+					$result =  $this->PersonModel->loadWhere(array(
+						"person_id != " => $person["person_id"],
+						"SUBSTRING(svnr FROM 1 FOR 10) = " => $person["svnr"])
+					);
+					if (is_object($result) && $result->error == EXIT_SUCCESS &&
+						is_array($result->retval) && count($result->retval) > 0)
 					{
-						$person["svnr"] = $person["svnr"] . "v1";
-					}
-					else
-					{
-						$person["svnr"] = $person["svnr"] . "v" . ($result->retval[0]->svnr{11} + 1);
+						if (count($result->retval) == 1)
+						{
+							$person["svnr"] = $person["svnr"] . "v1";
+						}
+						else
+						{
+							$person["svnr"] = $person["svnr"] . "v" . ($result->retval[0]->svnr{11} + 1);
+						}
 					}
 				}