stfalcon/PaymentBundle — pull request #1 by fightmaster (2012-03-22 18:35:04+00:00)
Title: Refactoring

Привет, Степан. Позволю писать на русском.

Я посмотрел, у тебя этот бандл немного отличается от того, что fwdays. Но вроде бы работает.

Код мне не нравится в целом. он не адаптирован для использования как бандла. Но, наверное, как и у меня не было времени.

Я немного отрефакторил:
* Подправил code style по symfony standarts
* Убрал приведение к float. Все равно на сервер интеркассы ты передаешь строку, им главное чтобы разделитель был точка. Твой код - НЕСТАБИЛЕН. Я пару месяцев назад на работе столкнулся с таким багом на своей машине. В линуксах, да и не только в них, php смотрит настройки машины (и вроде даже не настройки php), в зависимости от настроек ставит разделителем или "." или ",". У меня как раз это приведение ставило ЗАПЯТУЮ.
* Вынес настройки роутинга из аннотаций. Вообще, я ярый противник аннотаций в коде, который "называется" бандлом.

В принципе все, ничего страшного вроде не сделал.

Думаю чуть попозже вынесу настройки базы из аннотаций и необходимо добавить translation.