appwilio / cdek-sdk Goto Github PK

В документации к калькулятору содержится описание дополнительного поля services, который представляет собой массив с элементами из кода услуги и опционального дополнительного параметра услуги. В SDK я этого не нашёл. Может быть плохо искал?

orders и ordersPrint объявлены как Order[], а errors объявлено как PrintReceiptsError[], то есть массив совершенно другого типа. Здесь что-то другое имелось ввиду?

Можно ли понизить требования к версии PHP до текущей стабильной версии? Например, в Debian stable всё ещё есть только PHP 7.0.

У вас в проекте есть только два места где используется void и нигде не используется iterable. Есть несколько мест где объявлены публичные и приватные константы, но это тоже можно исправить без конфликтов. Ещё есть места с опциональными значениями, но это тоже поправимо. (Обычно всё кроме iterable проблемы не представляет.)

Никакие из собственных зависимостей не требуют PHP 7.1+, с этим тоже нет проблем.

Пока что получается так:

$ git diff -w --stat
 composer.json                           |  2 +-
 src/Common/City.php                     |  2 +-
 src/Common/Order.php                    | 10 +++++-----
 src/Common/Pvz.php                      |  2 +-
 src/Requests/CalculationRequest.php     | 12 ++++++------
 src/Requests/DeleteRequest.php          |  2 +-
 src/Requests/DeliveryRequest.php        |  4 ++--
 src/Requests/InfoReportRequest.php      |  4 ++--
 src/Requests/PrintReceiptsRequest.php   |  4 ++--
 src/Requests/PvzListRequest.php         | 10 +++++-----
 src/Requests/StatusReportRequest.php    |  4 ++--
 src/Responses/DeliveryResponse.php      |  2 +-
 src/Responses/PrintReceiptsResponse.php |  2 +-
 src/Responses/Types/Message.php         |  2 +-
 src/Responses/Types/Result.php          | 16 ++++++++--------
 15 files changed, 39 insertions(+), 39 deletions(-)

@sanmai, как и обещал (#15) сделал хард форк cdek-sdk, и что это за форк! Там уже 260+ новых коммитов и вы просто гляньте на список реализованного или на покрытие тестами или на документацию.

Апплодирую стоя.

В связи с наличием beta версии common-integration у СДЭК - предлагаю открыть новую ветку разработки beta-common-integration. СДЭК предоставил мне доступ к ней. Как таковой документации там еще не существует.

Первая необходимая правка в новой ветке:

• В тег ордер необходимо добавить данную информацию для изменения отправителя на третье лицо

<Sender company="РУ_ЕКАТЕРИНБУРГ_Мега_BCS" name="ООО Адидас">
            <address flat=" " house="-" street="620131, Екатеринбург, Металлургов,87  "/>
            <phone>8-800-250-04-05</phone>
</Sender>

Также в новой версии отличаются ответы.

Готов сделать первые правки для интеграции с бета версией в новой ветке.

/**
 * @JMS\XmlAttribute
 * @JMS\SerializedName("RecipientName")
 * @JMS\Type("string")
 *
 * @var string
 */
protected $RecipientEmail;

Should be @JMS\SerializedName("RecipientEmail")

This causes to an incorrect data representation when printing the PDF returned by PrintReceiptsRequest.

Их просто пока не добавили или была какая-то причина, что их нет?

День добрый. Добавьте пожалуйста теги с номерами версий - это сильно ускоряет установки через composer.