Comments 16
пример использования неплохо бы включить в README.md
Хорошо, добавлю.
А по коду есть замечания?
А по коду есть замечания?
Особо не вникал, но выглядит весьма достойно.
Навскидку — в Client::createBil() Я бы параметризацию инкапсулировал. Ну и аксессоры (во многих местах) не помешали бы — но тут уже больше дело вкуса.
Грустно, что требуется php не ниже 5.3.
Навскидку — в Client::createBil() Я бы параметризацию инкапсулировал. Ну и аксессоры (во многих местах) не помешали бы — но тут уже больше дело вкуса.
Грустно, что требуется php не ниже 5.3.
По поводу зависимости версии php — наверно стоило сделать попроще, но уж очень мне нравятся новые возможности. У меня большая часть кода только под 5.4 работает (в основном, из-за короткой формы массивов). В нашем случае это не страшно, т.к. сервер свой, и поддерживать 5.2 нет смысла.
Даже если сервер свой — не всегда есть возможность/смысл обновлятся до свежих версий. Из проектов с которыми я сейчас работаю, большинство на 5.2, есть 5.1. Один, новый на 5.4. Пять три нет вообще нигде.
PHP поддерживает BC. И если у вас не совсем ужасный код на 5.2, то он почти наверняка заработает на 5.3. А вот держать на серверах php ниже 5.3 может быть просто опасно (я уж не говорю про скорость, например).
После НГ планируем пробовать обновлятся. Посмотрим как пойдет — подводных камней масса. Самый крупный проект развернут на >20 виртуалках (6 физ серверов и облако). Для установки новой версии пхп, со всеми либами, необходимо обновлять дистрибутив- т.е создавать новые виртуалки и поднимать там весь софт, конфиги и тп.
Отправил ссылку на пост программистам Кошелька.
Как-то так я вижу SOAP/XML API
А за либу, конечно, спасибо.
А за либу, конечно, спасибо.
Спасибо, будем использовать. Как раз вовремя.
Sign up to leave a comment.
Библиотека для работы с QIWI через SOAP