#44 update doc and do not prevent resource init on available properties

shoppingflux · Jun 28, 2018 · dd5a8ca · dd5a8ca
1 parent 09da74d
commit dd5a8ca
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/src/Api/Order/OrderItem.php b/src/Api/Order/OrderItem.php
@@ -6,8 +6,19 @@
  */
 class OrderItem
 {
+    /**
+     * @var string
+     */
     private $reference;
+
+    /**
+     * @var int
+     */
     private $quantity;
+
+    /**
+     * @var float
+     */
     private $unitPrice;
 
     /**

diff --git a/src/Resource/AbstractResource.php b/src/Resource/AbstractResource.php
@@ -57,13 +57,15 @@ public function toArray()
     /**
      * @param string $property   The property name
      * @param bool   $initialize Indicates if the resource must be fetched from server
-     *                           in order to access to this property (when no present in partial representation)
+     *                           in order to access to this property (when not present in partial representation)
+     *                           The property absence in the current representation is mandatory to pull data
+     *                           from remote API
      *
      * @return mixed|null
      */
     protected function getProperty($property, $initialize = false)
     {
-        if (true === $initialize) {
+        if (true === $initialize && ! $this->resource->hasProperty($property)) {
             $this->initialize();
         }