Change endpoint

answear · Sep 17, 2024 · 64ddbb5 · 64ddbb5
1 parent 958b982
commit 64ddbb5
Show file tree

Hide file tree

Showing 23 changed files with 333 additions and 158 deletions.
diff --git a/.php-cs-fixer.php b/.php-cs-fixer.php
@@ -20,7 +20,7 @@
             'strict_param' => false,
             'array_syntax' => ['syntax' => 'short'],
             'concat_space' => ['spacing' => 'one'],
-            'phpdoc_align' => [],
+            'phpdoc_align' => ['align' => 'left'],
             'phpdoc_summary' => false,
             'void_return' => false,
             'phpdoc_var_without_name' => false,

diff --git a/src/Client/Client.php b/src/Client/Client.php
@@ -2,10 +2,13 @@
 
 namespace Answear\FanCourierBundle\Client;
 
+use Answear\FanCourierBundle\ConfigProvider;
 use Answear\FanCourierBundle\Exception\RequestException;
 use Answear\FanCourierBundle\Exception\ResponseException;
 use Answear\FanCourierBundle\Logger\FanCourierLogger;
+use Answear\FanCourierBundle\Request\LoginRequest;
 use Answear\FanCourierBundle\Request\RequestInterface;
+use Answear\FanCourierBundle\Response\LoginResponse;
 use GuzzleHttp\Client as GuzzleClient;
 use GuzzleHttp\ClientInterface;
 use GuzzleHttp\Exception\GuzzleException;
@@ -15,21 +18,45 @@ class Client
 {
     private const CONNECTION_TIMEOUT = 10;
     private const TIMEOUT = 30;
-
+
+    private ?string $token = null;
+
     public function __construct(
         private readonly RequestTransformerInterface $requestTransformer,
         private readonly FanCourierLogger $logger,
+        private readonly ConfigProvider $configProvider,
         private ?ClientInterface $client = null,
     ) {
         $this->client ??= new GuzzleClient(['timeout' => self::TIMEOUT, 'connect_timeout' => self::CONNECTION_TIMEOUT]);
     }
 
+    public function login(): void
+    {
+        $loginRequest = new LoginRequest(
+            $this->configProvider->username,
+            $this->configProvider->password,
+        );
+
+        $response = $this->request($loginRequest);
+
+        $loginResponse = LoginResponse::fromArray(
+            \json_decode($response->getBody()->getContents(), true),
+        );
+
+        $this->token = $loginResponse->token;
+    }
+
     public function request(RequestInterface $request): ResponseInterface
     {
         $this->logger->setRequestId(uniqid('FANCOURIER-', more_entropy: true));
 
         try {
             $psrRequest = $this->requestTransformer->transform($request);
+
+            if ($this->token) {
+                $psrRequest = $psrRequest->withHeader('Authorization', 'Bearer ' . $this->token);
+            }
+
             $this->logger->logRequest($request->getEndpoint(), $psrRequest);
 
             $response = $this->client->send($psrRequest);

diff --git a/src/Client/RequestTransformer.php b/src/Client/RequestTransformer.php
@@ -21,17 +21,11 @@ public function transform(RequestInterface $request): PsrRequestInterface
     {
         $url = $this->configProvider->apiUrl . $request->getEndpoint();
 
-        $formParams = [
-            'username' => $this->configProvider->username,
-            'client_id' => $this->configProvider->clientId,
-            'user_pass' => $this->configProvider->password,
-        ];
-
         return new HttpRequest(
             $request->getMethod(),
             new Uri($url),
-            ['Content-Type' => 'application/x-www-form-urlencoded'],
-            http_build_query(array_merge($formParams, $request->getOptions()))
+            ['Content-Type' => 'application/json'],
+            http_build_query($request->getOptions()),
         );
     }
 }
diff --git a/src/ConfigProvider.php b/src/ConfigProvider.php
@@ -7,7 +7,6 @@
 class ConfigProvider
 {
     public function __construct(
-        public readonly string $clientId,
         public readonly string $username,
         public readonly string $password,
         public readonly string $apiUrl,

diff --git a/src/DTO/AddressDTO.php b/src/DTO/AddressDTO.php
@@ -0,0 +1,43 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Answear\FanCourierBundle\DTO;
+
+use Webmozart\Assert\Assert;
+
+class AddressDTO
+{
+    public function __construct(
+        public readonly string $locality,
+        public readonly string $county,
+        public readonly string $street,
+        public readonly string $streetNo,
+        public readonly string $zipCode,
+        public readonly string $floor,
+        public readonly string $reference,
+    ) {
+    }
+
+    /**
+     * @param array{locality: string, county: string, street: string, streetNo: string, zipCode: string, floor: string, reference: string} $addressArray
+     */
+    public static function fromArray(array $addressArray): self
+    {
+        Assert::stringNotEmpty($addressArray['locality']);
+        Assert::stringNotEmpty($addressArray['county']);
+        Assert::stringNotEmpty($addressArray['street']);
+        Assert::stringNotEmpty($addressArray['streetNo']);
+        Assert::stringNotEmpty($addressArray['zipCode']);
+
+        return new self(
+            $addressArray['locality'],
+            $addressArray['county'],
+            $addressArray['street'],
+            $addressArray['streetNo'],
+            $addressArray['zipCode'],
+            $addressArray['floor'],
+            $addressArray['reference'],
+        );
+    }
+}
diff --git a/src/DTO/DrawerCollection.php b/src/DTO/DrawerCollection.php
@@ -12,12 +12,12 @@ class DrawerCollection
      * @param DrawerDTO[] $drawers
      */
     public function __construct(
-        public readonly array $drawers = []
+        public readonly array $drawers = [],
     ) {
     }
 
     /**
-     * @param array<array{number: int, type: string}> $drawerArray
+     * @param array<array{type: string, number: int}> $drawerArray
      */
     public static function fromArray(array $drawerArray): self
     {

diff --git a/src/DTO/DrawerDTO.php b/src/DTO/DrawerDTO.php
@@ -8,7 +8,7 @@ class DrawerDTO
 {
     public function __construct(
         public readonly int $number,
-        public readonly string $type
+        public readonly string $type,
     ) {
     }
 }
diff --git a/src/DTO/PickupPointDTO.php b/src/DTO/PickupPointDTO.php
@@ -10,52 +10,42 @@ class PickupPointDTO
 {
     public function __construct(
         public readonly string $id,
+        public readonly string $code,
         public readonly string $name,
         public readonly string $routingLocation,
         public readonly string $description,
-        public readonly string $county,
-        public readonly string $locality,
-        public readonly string $address,
-        public readonly string $zipCode,
-        public readonly string $locationReference,
-        public readonly float $latitude,
-        public readonly float $longitude,
+        public readonly AddressDTO $address,
+        public readonly string $latitude,
+        public readonly string $longitude,
         public readonly ScheduleCollection $schedule,
-        public readonly DrawerCollection $drawer
+        public readonly DrawerCollection $drawer,
     ) {
     }
 
     public static function fromArray(array $pickupPoint): self
     {
-        Assert::stringNotEmpty($pickupPoint['Id']);
-        Assert::stringNotEmpty($pickupPoint['Name']);
-        Assert::stringNotEmpty($pickupPoint['RoutingLocation']);
-        Assert::stringNotEmpty($pickupPoint['Description']);
-        Assert::stringNotEmpty($pickupPoint['County']);
-        Assert::stringNotEmpty($pickupPoint['Locality']);
-        Assert::stringNotEmpty($pickupPoint['Address']);
-        Assert::stringNotEmpty($pickupPoint['ZipCode']);
-        Assert::stringNotEmpty($pickupPoint['LocationReference']);
-        Assert::float($pickupPoint['Latitude']);
-        Assert::float($pickupPoint['Longitude']);
-        Assert::range($pickupPoint['Latitude'], -90, 90);
-        Assert::range($pickupPoint['Longitude'], -180, 180);
-        Assert::count($pickupPoint['Schedule'], 7);
+        Assert::stringNotEmpty($pickupPoint['id']);
+        Assert::stringNotEmpty($pickupPoint['name']);
+        Assert::stringNotEmpty($pickupPoint['routingLocation']);
+        Assert::stringNotEmpty($pickupPoint['description']);
+        Assert::notEmpty($pickupPoint['address']);
+        Assert::stringNotEmpty($pickupPoint['latitude']);
+        Assert::stringNotEmpty($pickupPoint['longitude']);
+        Assert::range((float) $pickupPoint['latitude'], -90, 90);
+        Assert::range((float) $pickupPoint['longitude'], -180, 180);
+        Assert::count($pickupPoint['schedule'], 7);
 
         return new self(
-            $pickupPoint['Id'],
-            $pickupPoint['Name'],
-            $pickupPoint['RoutingLocation'],
-            $pickupPoint['Description'],
-            $pickupPoint['County'],
-            $pickupPoint['Locality'],
-            $pickupPoint['Address'],
-            $pickupPoint['ZipCode'],
-            $pickupPoint['LocationReference'],
-            $pickupPoint['Latitude'],
-            $pickupPoint['Longitude'],
-            ScheduleCollection::fromArray($pickupPoint['Schedule']),
-            DrawerCollection::fromArray($pickupPoint['Drawer'])
+            $pickupPoint['id'],
+            $pickupPoint['code'],
+            $pickupPoint['name'],
+            $pickupPoint['routingLocation'],
+            $pickupPoint['description'],
+            AddressDTO::fromArray($pickupPoint['address']),
+            $pickupPoint['latitude'],
+            $pickupPoint['longitude'],
+            ScheduleCollection::fromArray($pickupPoint['schedule']),
+            DrawerCollection::fromArray($pickupPoint['drawer']),
         );
     }
 }
diff --git a/src/DTO/ScheduleCollection.php b/src/DTO/ScheduleCollection.php
@@ -12,20 +12,20 @@ class ScheduleCollection
      * @param ScheduleDTO[] $schedules
      */
     public function __construct(
-        public readonly array $schedules = []
+        public readonly array $schedules = [],
     ) {
     }
 
     /**
-     * @param array<array{startHour: string, stopHour: string}> $scheduleArray
+     * @param array<array{firstHour: string, secondHour: string}> $scheduleArray
      */
     public static function fromArray(array $scheduleArray): self
     {
         $schedules = [];
         foreach ($scheduleArray as $schedule) {
-            Assert::stringNotEmpty($schedule['startHour']);
-            Assert::stringNotEmpty($schedule['stopHour']);
-            $schedules[] = new ScheduleDTO($schedule['startHour'], $schedule['stopHour']);
+            Assert::stringNotEmpty($schedule['firstHour']);
+            Assert::stringNotEmpty($schedule['secondHour']);
+            $schedules[] = new ScheduleDTO($schedule['firstHour'], $schedule['secondHour']);
         }
 
         return new self($schedules);

diff --git a/src/DTO/ScheduleDTO.php b/src/DTO/ScheduleDTO.php
@@ -7,8 +7,8 @@
 class ScheduleDTO
 {
     public function __construct(
-        public readonly string $startHour,
-        public readonly string $stopHour
+        public readonly string $firstHour,
+        public readonly string $secondHour,
     ) {
     }
 }
diff --git a/src/DependencyInjection/AnswearFanCourierExtension.php b/src/DependencyInjection/AnswearFanCourierExtension.php
@@ -38,7 +38,6 @@ public function load(array $configs, ContainerBuilder $container): void
 
         $definition = $container->getDefinition(ConfigProvider::class);
         $definition->setArguments([
-            $this->config['clientId'],
             $this->config['username'],
             $this->config['password'],
             $this->config['apiUrl'],

diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -15,8 +15,7 @@ public function getConfigTreeBuilder(): TreeBuilder
 
         $treeBuilder->getRootNode()
             ->children()
-            ->scalarNode('clientId')->isRequired()->cannotBeEmpty()->end()
-            ?->scalarNode('username')->isRequired()->cannotBeEmpty()->end()
+            ->scalarNode('username')->isRequired()->cannotBeEmpty()->end()
             ?->scalarNode('password')->isRequired()->cannotBeEmpty()->end()
             ?->scalarNode('apiUrl')->isRequired()->cannotBeEmpty()->end()
             ?->scalarNode('logger')->defaultNull()->end()

diff --git a/src/Request/GetPickupPointsRequest.php b/src/Request/GetPickupPointsRequest.php
@@ -6,7 +6,7 @@
 
 class GetPickupPointsRequest implements RequestInterface
 {
-    private const ENDPOINT = '/pickup-points.php';
+    private const ENDPOINT = '/reports/pickup-points';
     private const HTTP_METHOD = 'POST';
 
     public function getEndpoint(): string
@@ -23,4 +23,9 @@ public function getOptions(): array
     {
         return ['type' => 'fanbox'];
     }
+
+    public function getBody(): array
+    {
+        return [];
+    }
 }
diff --git a/src/Request/LoginRequest.php b/src/Request/LoginRequest.php
@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Answear\FanCourierBundle\Request;
+
+class LoginRequest implements RequestInterface
+{
+    private const ENDPOINT = '/login';
+    private const HTTP_METHOD = 'POST';
+
+    public function __construct(
+        private readonly string $username,
+        private readonly string $password,
+    ) {
+    }
+
+    public function getEndpoint(): string
+    {
+        return self::ENDPOINT;
+    }
+
+    public function getMethod(): string
+    {
+        return self::HTTP_METHOD;
+    }
+
+    public function getOptions(): array
+    {
+        return [];
+    }
+
+    public function getBody(): array
+    {
+        return [
+            'username' => $this->username,
+            'password' => $this->password,
+        ];
+    }
+}
diff --git a/src/Request/RequestInterface.php b/src/Request/RequestInterface.php
@@ -11,4 +11,6 @@ public function getEndpoint(): string;
     public function getMethod(): string;
 
     public function getOptions(): array;
+
+    public function getBody(): array;
 }