eagleas / webmoney

Interface library for WebMoney Transfer payment system. Gem.
http://www.wmtransfer.com
44 stars 31 forks source link

Added check_owner X19 interface #6

Closed divineforest closed 12 years ago

eagleas commented 12 years ago

Хотелось бы:

  1. спеку
  2. возможность проверять все параметры, а не только wmid, фио. М.б. сделать прямо соответствие параметров тому как в интерфейсе? т.е. не :first_name, а :fname и т.д. Так с ними работать оптом будет проще. Сделаете?
eagleas commented 12 years ago

Есть такой код: https://github.com/eagleas/webmoney/blob/master/lib/webmoney.rb#L184 который подставляет в конце урлов Cert если в конфиге PEM серификат указан. Т.е. указывать прямо https://github.com/evrone/webmoney/commit/ac102f905dd963953eedd19171c272e3775bedcb#L1R24 лайтовый - не совсем корректно. Здесь конечно спорный сам способ как я это реализовал, можно думать как сделать более изящно.

divineforest commented 12 years ago
  1. Попробую
  2. ok
  3. Видел, сразу не получилось, поэтому сделал по-быстрому. Там просто у тебя есть проверка, что адрес содержить стандартный w3s префикс, а адрес у x19 другой.
eagleas commented 12 years ago

Наверное вот так будет наиболее гибко: https://gist.github.com/1600770 если параметром :x509 => true, то применяем общее преобразование: url.sub(/.asp$/, 'Cert.asp') Могу эту часть сделать сам.

divineforest commented 12 years ago

ok

eagleas commented 12 years ago

done, pushed :)

divineforest commented 12 years ago

Добавил спеки

eagleas commented 12 years ago

Остался один момент - вмержить мой бранч в твой, и прbвести в твоем коде название этого интерфейса к общему стандарту (== название как в API WM) :check_user вместо :check_owner

divineforest commented 12 years ago

Не работает в ruby 1.8.7. Похоже из-за того, что lambda? из ruby 1.9

eagleas commented 12 years ago

Попробуй заменить на respond_to?(:call)

divineforest commented 12 years ago

respond_to?(:call) помогло. Спушил фикс

eagleas commented 12 years ago

Постараюсь в ближайшее время. м.б. даже сегодня проверить и закомитить в мастер.

divineforest commented 12 years ago

Всё ок?

eagleas commented 12 years ago

Никак не доберусь до него

eagleas commented 12 years ago

Проверил, запушил. Спасибо за помощь!

divineforest commented 12 years ago

пожалуйста :)