search

baibaratsky / php-webmoney

WebMoney API PHP Library
Other
73 stars 32 forks source link

Light auth implementation #75

Closed roman-rybalko closed 9 years ago

roman-rybalko commented 9 years ago

Привет! Проверь пожалуйста мою реализацию аутентификации сертификатами (WebPro/Light). Я перенес пару методов и констант в базовый класс, 1 новый метод. Интерфейс не сломался.

baibaratsky commented 9 years ago

Все перенесенные константы, свойства и методы лучше вернуть на место (и новый метод туда же), а в CurlRequester использовать

use baibaratsky\WebMoney\Api\X\Request;

...

if ($request instanceof Request && $request->getAuthType() === Request::AUTH_LIGHT) {
roman-rybalko commented 9 years ago

Все понял. Сделаю новый pull request.

chuprik commented 9 years ago

Не нужно делать новый, можно сделать коммит, пуш и открыть этот пулл реквест. Все новые коммиты подтянутся.

roman-rybalko commented 9 years ago

Да ну, потом не разобрать среди этой тучи дифов что к чему.

chuprik commented 9 years ago

А вот тут ты не прав) Когда код меняется, diff скрывается, и так проще отсматривать изменения :)

roman-rybalko commented 9 years ago

Ладно, понял, в след. раз буду в том-же pull request делать. Я еще пишу X21 и X22.