baibaratsky / php-webmoney

WebMoney API PHP Library
Other
73 stars 32 forks source link

WMRequestSigner: WMID #15

Closed baibaratsky closed 11 years ago

baibaratsky commented 11 years ago

Что-то мне не очень нравится, что у нас WMID нужно сначала в конструктор WMRequestSigner передавать, а потом еще в запрос. Может, как-то отрефакторим? Только мне бы не хотелось убирать WMID из конструктора WMRequestSigner, чтобы его можно было использовать автономно, без привязки к остальной библиотеке.

chuprik commented 11 years ago

Билдер запросов в WebMoney, который будет принимать wmid и создавать WMApiRequest, а также WMRequestSigner (хотя он должен быть 1 в системе, я считаю)? В итоге будет непрозрачная передача wmid.

Либо можно сделать геттер для wmid в WMRequestSigner, и в sign заполнять signerWmid.

baibaratsky commented 11 years ago

Есть ситуации, когда WMRequestSigner не нужен, а signerWmid нужно заполнить.

chuprik commented 11 years ago

Как вариант, можно конечно при вызове sign передавать параметры для подписывания запроса, но тогда мы подписывание запроса будем вызывать сами перед WMApiRequestPerformer->perform().