Added Update Method to Application when GUID property doesn't exist i.e TaxRate

src/XeroPHP/Application.php

@@ -197,7 +197,7 @@ public function loadByGUID($model, $guid)
         /** @var Remote\Model $class */
         $class = $this->validateModelClass($model);
 
-        if(!$guid){
+        if (! $guid) {
             throw new Remote\Exception\NotFoundException;
         }
 
@@ -238,7 +238,7 @@ public function loadByGUIDs($model, $guids)
         /** @var $class Remote\Model */
         $class = $this->validateModelClass($model);
 
-        if(empty($guids)){
+        if (empty($guids)) {
             return [];
         }
 
@@ -288,7 +288,7 @@ public function save(Remote\Model $object, $replace_data = false)
         //(special saving endpoints)
         $this->savePropertiesDirectly($object);
 
-        if (!$object->isDirty()) {
+        if (! $object->isDirty()) {
             return null;
         }
 
@@ -305,7 +305,51 @@ public function save(Remote\Model $object, $replace_data = false)
             $object->setApplication($this);
         }
 
-        if (!$object::supportsMethod($method)) {
+        if (! $object::supportsMethod($method)) {
+            throw new Exception(sprintf('%s doesn\'t support [%s] via the API', get_class($object), $method));
+        }
+
+        //Put in an array with the first level containing only the 'root node'.
+        $data = [$object::getRootNodeName() => $object->toStringArray(true)];
+        $url = new URL($this, $uri, $object::getAPIStem());
+        $request = new Request($this, $url, $method);
+
+        $request->setBody(Helpers::arrayToXML($data))->send();
+        $response = $request->getResponse();
+
+        if (false !== $element = current($response->getElements())) {
+            $object->fromStringArray($element, $replace_data);
+        }
+        //Mark the object as clean since no exception was thrown
+        $object->setClean();
+
+        return $response;
+    }
+
+    /**
+     * @param Remote\Model $object
+     * @param bool $replace_data
+     *
+     * @throws Exception
+     *
+     * @return Remote\Response|null
+     */
+    public function update(Remote\Model $object, $replace_data = false)
+    {
+        //Saves any properties that don't want to be included in the normal loop
+        //(special saving endpoints)
+        $this->savePropertiesDirectly($object);
+
+        if (! $object->isDirty()) {
+            return null;
+        }
+
+        $object->validate();
+
+        $method = $object::supportsMethod(Request::METHOD_POST) ? Request::METHOD_POST : Request::METHOD_PUT;
+        $uri = sprintf('%s/%s', $object::getResourceURI(), $object->getGUID());
+
+        if (! $object::supportsMethod($method)) {
             throw new Exception(sprintf('%s doesn\'t support [%s] via the API', get_class($object), $method));
         }
 
@@ -441,7 +485,7 @@ private function savePropertiesDirectly(Remote\Model $object)
      */
     public function delete(Remote\Model $object)
     {
-        if (!$object::supportsMethod(Request::METHOD_DELETE)) {
+        if (! $object::supportsMethod(Request::METHOD_DELETE)) {
             throw new Exception(
                 sprintf(
                     '%s doesn\'t support [DELETE] via the API',

src/XeroPHP/Models/Accounting/PurchaseOrder.php

@@ -306,7 +306,7 @@ public function getLineItems()
     public function addLineItem(LineItem $value)
     {
         $this->propertyUpdated('LineItems', $value);
-        if (! isset($this->_data['LineItems'])) {
+        if (!isset($this->_data['LineItems'])) {
             $this->_data['LineItems'] = new Remote\Collection();
         }
         $this->_data['LineItems'][] = $value;

src/XeroPHP/Models/Accounting/TaxRate.php

@@ -227,7 +227,7 @@ public function getTaxComponents()
     public function addTaxComponent(TaxComponent $value)
     {
         $this->propertyUpdated('TaxComponents', $value);
-        if (! isset($this->_data['TaxComponents'])) {
+        if (!isset($this->_data['TaxComponents'])) {
             $this->_data['TaxComponents'] = new Remote\Collection();
         }
         $this->_data['TaxComponents'][] = $value;

src/XeroPHP/Models/Accounting/TrackingCategory.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace XeroPHP\Models\Accounting;
 
 use XeroPHP\Remote;
@@ -207,7 +208,7 @@ public function getTrackingOptionID()
     {
         return $this->_data['TrackingOptionID'];
     }
- 
+
     /**
      * @param string $value
      *

src/XeroPHP/Models/Accounting/TrackingCategory/TrackingOption.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace XeroPHP\Models\Accounting\TrackingCategory;
 
 use XeroPHP\Remote;

src/XeroPHP/Remote/Model.php

@@ -212,7 +212,7 @@ public function fromStringArray($input_array, $replace_data = false)
                 continue;
             }
 
-            if ($isArray && ! is_array($input_array[$property])) {
+            if ($isArray && !is_array($input_array[$property])) {
                 $this->_data[$property] = null;
 
                 continue;
@@ -349,7 +349,7 @@ public static function castFromString($type, $value, $php_type)
             case self::PROPERTY_TYPE_BOOLEAN:
                 return in_array(strtolower($value), ['true', '1', 'yes'], true);
 
-            /** @noinspection PhpMissingBreakStatementInspection */
+                /** @noinspection PhpMissingBreakStatementInspection */
             case self::PROPERTY_TYPE_TIMESTAMP:
                 $timezone = new \DateTimeZone('UTC');
 
@@ -428,6 +428,24 @@ public function validate($check_children = true)
         return true;
     }
 
+    /**
+     * Shorthand update an object if it is instantiated with app context.
+     *
+     * @throws Exception
+     *
+     * @return Response|null
+     */
+    public function update()
+    {
+        if ($this->_application === null) {
+            throw new Exception(
+                '->update() is only available on objects that have an injected application context.'
+            );
+        }
+
+        return $this->_application->update($this);
+    }
+
     /**
      * Shorthand save an object if it is instantiated with app context.
      *
@@ -494,6 +512,10 @@ public function __isset($property)
      */
     public function __get($property)
     {
+        if (empty($property)) {
+            return false;
+        }
+
         $getter = sprintf('get%s', $property);
 
         if (method_exists($this, $getter)) {
@@ -513,6 +535,10 @@ public function __get($property)
      */
     public function __set($property, $value)
     {
+        if (empty($property)) {
+            return false;
+        }
+
         $setter = sprintf('set%s', $property);
 
         if (method_exists($this, $setter)) {
@@ -521,7 +547,7 @@ public function __set($property, $value)
 
         trigger_error(sprintf("Undefined property %s::$%s.\n", __CLASS__, $property));
     }
-
+    
     protected function propertyUpdated($property, $value)
     {
         $currentValue = isset($this->_data[$property]) ? $this->_data[$property] : null;