Merge pull request #21 from stefandoorn/check-no-stdclass-charge-value

Check no stdclass charges MonetaryValue
gabrielbull · Apr 20, 2015 · ac1ef51 · ac1ef51
2 parents b6a2e01 + 75085ed
commit ac1ef51
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/Ups/Entity/Charges.php b/src/Ups/Entity/Charges.php
@@ -15,7 +15,7 @@ function __construct($response = null)
             if (isset($response->CurrencyCode)) {
                 $this->CurrencyCode = $response->CurrencyCode;
             }
-            if (isset($response->MonetaryValue)) {
+            if (isset($response->MonetaryValue) && !$response->MonetaryValue instanceof \stdClass) {
                 $this->MonetaryValue = (float)$response->MonetaryValue;
             }
             if (isset($response->Code)) {
@@ -29,4 +29,4 @@ function __construct($response = null)
             }
         }
     }
-} 
+}
diff --git a/src/Ups/Response.php b/src/Ups/Response.php
@@ -13,14 +13,14 @@ class Response implements ResponseInterface
     /**
      * @var SimpleXMLElement
      */
-    protected $reponse;
+    protected $response;
 
     /**
      * @return SimpleXMLElement
      */
     public function getResponse()
     {
-        return $this->reponse;
+        return $this->response;
     }
 
     /**
@@ -29,7 +29,7 @@ public function getResponse()
      */
     public function setResponse(SimpleXMLElement $response)
     {
-        $this->reponse = $response;
+        $this->response = $response;
         return $this;
     }
 
@@ -50,4 +50,4 @@ public function setText($text)
         $this->text = $text;
         return $this;
     }
-}
+}