Исправление алгоритма генерации secretKey для многомерных массивов

Signed-off-by: Vitaly Baev <[email protected]>

src/Client.php

@@ -3,7 +3,7 @@
 namespace Vitalybaev\YandexDelivery;
 
 /**
- *
+ * Библиотека для выполнения запросов к API Яндекс.Доставки.
  */
 class Client
 {
@@ -65,7 +65,7 @@ function __construct($clientId, $senderId, $apiVersion = '1.0', $methodKeys = []
     }
 
     /**
-     * @return
+     * @return string
      */
     public function getSenderId()
     {
@@ -84,7 +84,7 @@ public function setSenderId($senderId)
     }
 
     /**
-     * @return
+     * @return string
      */
     public function getClientId()
     {
@@ -108,6 +108,8 @@ public function setClientId($clientId)
      * @param  string $method
      * @param  string $parameters
      * @return array
+     * @throws Exception\InvalidJsonException
+     * @throws Exception\MethodKeysNotExistsException
      */
     public function call($method, $parameters)
     {
@@ -121,16 +123,8 @@ public function call($method, $parameters)
             'sender_id' => $this->senderId,
         ]);
 
-        ksort($httpParameters);
-
-        // Получаем secret_key
-        $secretKeyBase = '';
-        foreach ($httpParameters as $value) {
-            $secretKeyBase .= $value;
-        }
-        $secretKeyBase .= $this->methodKeys[$method];
-
-        $httpParameters['secret_key'] = md5($secretKeyBase);
+        $secretKey = md5($this->getPostValues($httpParameters) . $this->methodKeys[$method]);
+        $httpParameters['secret_key'] = $secretKey;
 
         // Выполняем запрос
         $httpResponse = $this->httpClient->request('POST', "https://delivery.yandex.ru/api/$this->apiVersion/$method", [
@@ -146,4 +140,15 @@ public function call($method, $parameters)
 
         return $json;
     }
+
+    private function getPostValues($data)
+    {
+        if (!is_array($data)) {
+            return $data;
+        }
+        ksort($data);
+        return join('', array_map(function($k) {
+            return $this->getPostValues($k);
+        }, $data));
+    }
 }