Merge pull request #904 from bretto36/handle-null-values

Handle null values

src/XeroPHP/Remote/Model.php

@@ -78,7 +78,7 @@ public function __construct(Application $application = null)
         $this->_data = [];
         $this->_associated_objects = [];
     }
-    
+
     public static function make(Application $application = null)
     {
         return new static($application);
@@ -524,7 +524,9 @@ public function __set($property, $value)
 
     protected function propertyUpdated($property, $value)
     {
-        if (! isset($this->_data[$property]) || $this->_data[$property] !== $value) {
+        $currentValue = isset($this->_data[$property]) ? $this->_data[$property] : null;
+
+        if ($currentValue !== $value) {
             //If this object can update itself, set its own dirty flag, otherwise, set its parent's.
             if (count(array_intersect($this::getSupportedMethods(), [Request::METHOD_PUT, Request::METHOD_POST])) > 0) {
                 //Object can update itself

tests/Application/ApplicationTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace XeroPHP\tests\Application;
+namespace XeroPHP\Tests\Application;
 
 use XeroPHP\Application;
 

tests/HelpersTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace XeroPHP\tests;
+namespace XeroPHP\Tests;
 
 use XeroPHP\Helpers;
 

tests/Remote/ModelTest.php

@@ -126,6 +126,12 @@ public function testPracticeManagerClientListIsReturned()
           <Position /> 
         </Contact>
       </Contacts>
+      <Groups>
+        <Group>
+          <ID>783949</ID>
+          <Name>1 Geotechnical Limited</Name>
+        </Group>
+      </Groups>
     </Client>
     <Client>
       <ID>697</ID>

tests/Webhook/EventTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace XeroPHP\tests;
+namespace XeroPHP\Tests\Webhook;
 
 use XeroPHP\Webhook;
 use XeroPHP\Application;

tests/WebhookTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace XeroPHP\tests;
+namespace XeroPHP\Tests;
 
 use XeroPHP\Application;
 use XeroPHP\Application\PrivateApplication;