Added findDeliveryPoint method to get only one point

Author: xaviborja

src/MondialRelay/ApiClient.php

@@ -21,20 +21,17 @@ public function findDeliveryPoints(array $request)
         try{
             $request = $this->decorateRequest($request);
             $result = $this->client->WSI3_PointRelais_Recherche($request);
-            $this->checkResponse($result);
+            $this->checkResponse('WSI3_PointRelais_Recherche',$result);
             $delivery_points = [];
             if (!property_exists($result->WSI3_PointRelais_RechercheResult->PointsRelais, 'PointRelais_Details')){
                 return $delivery_points;
             }
             $label_position = 1;
+            if (is_object($result->WSI3_PointRelais_RechercheResult->PointsRelais->PointRelais_Details)){
+                return $this->createPoint($result->WSI3_PointRelais_RechercheResult->PointsRelais->PointRelais_Details);
+            }
             foreach($result->WSI3_PointRelais_RechercheResult->PointsRelais->PointRelais_Details as $destination_point){
-                $delivery_points[] = new Point(
-                    $destination_point->Num,
-                    trim($destination_point->LgAdr1),
-                    str_replace(",",".",$destination_point->Latitude),
-                    str_replace(",",".",$destination_point->Longitude),
-                    $destination_point->CP
-                );
+                $delivery_points[] = $this->createPoint($destination_point);
                 $label_position++;
             }
             return $delivery_points;
@@ -44,6 +41,19 @@ public function findDeliveryPoints(array $request)
 
     }
 
+    public function findDeliveryPoint($id, $country)
+    {
+        try {
+            return $this->findDeliveryPoints(array(
+                'NumPointRelais' => $id,
+                'Pays' => $country
+            ));
+
+        } catch ( \SoapFault $e ) {
+            throw new \Exception();
+        }
+    }
+
     private function decorateRequest($request)
     {
         $key = $this->websiteId;
@@ -56,17 +66,27 @@ private function decorateRequest($request)
         return $request;
     }
 
-    private function checkResponse($result)
+    private function checkResponse($method, $result)
     {
-        if ($result->WSI3_PointRelais_RechercheResult->STAT != 0) {
+        $method = $method . "Result";
+        if ($result->{$method}->STAT != 0) {
             $request = $this->decorateRequest([
-                'STAT_ID' => $result->WSI3_PointRelais_RechercheResult->STAT,
+                'STAT_ID' => $result->{$method}->STAT,
                 'Langue' => 'ES',
             ]);
             $error_response = $this->client->WSI2_STAT_Label($request);
             throw new \InvalidArgumentException($error_response->WSI2_STAT_LabelResult);
         }
     }
 
+    private function createPoint($response) {
+        return new Point(
+            $response->Num,
+            trim($response->LgAdr1),
+            str_replace(",",".",$response->Latitude),
+            str_replace(",",".",$response->Longitude),
+            $response->CP
+        );
+    }
 
 }

tests/MondialRelay/ApiClientTest.php

@@ -67,4 +67,13 @@ public function itShouldReturnAnArrayOfPointsIfParametersMatch()
             $this->assertInstanceOf(Point::class, $point);
         }
     }
+
+    /**
+     * @test
+     */
+    public function itShouldReturnAValidPoint()
+    {
+        $point = $this->client->findDeliveryPoint('077712','ES');
+        $this->assertInstanceOf(Point::class, $point);
+    }
 }