sedovalx / taxi

6 stars 3 forks source link

Путаница с балансами #110

Open sedovalx opened 9 years ago

sedovalx commented 9 years ago

В кассовой форме в столбце "Итого" отображается одна сумма, а в редакторе аренды в сводке другая. Они считаются по-разному by design, но это здорово путает. @hodkoff2 не будет ли правильнее считать эти суммы одинаково?

sedovalx commented 9 years ago

@hodkoff2 нужен твой ответ

hodkovdd commented 9 years ago

@sedovalx посмотрел, путаница есть. однако проблема не в названии. Во-первых мы отклонились от дизайна и появилась страница "Управление арендой" - гибрид двух форм с разной логикой:

В "Кассовой" форме поле итого соответствует логике "Сводки", но не "Форме расчета", отсюда появившийся конфликт. Кроме этого по дизайну и логике после перевода аренды в состояние "рассчитана" она должна:

sedovalx commented 9 years ago

Уточню. В редакторе аренды итого вычисляется как сумма балансов по счетам аренды, штрафов и ремонтов плюс залог. В кассовой форме итого вычисляется как сумма только отрицательных балансов по тем же счетам, без залога.

Но в целом, кажется, понял. Вот только разносить информацию по аренде в разные формы не хотелось бы. Сейчас все в одном месте - в управлении арендой. Там и история, и операции над статусами. Т.е. любое действие над арендой можно там сделать, любую информацию получить. Если разнесем, то скорее всего получим, что при просмотре формы расчета захотят посмотреть историю операций. Или при просмотре текущей сводки захотят узнать, а что будет, если прямо сейчас расчет с водителем произвести. Как ты думаешь?

С другой стороны ты прав - есть две разные задачи - текущая сводка и финальный расчет. То, что в них итого так по-разному рассчитывается, меня настораживает, но это ладно. Быть может в управлении арендой сделать два поля?

Какие еще задачи этим не будут покрыты?

sedovalx commented 9 years ago

Всё-всё, остановись - убедил! Разделил эти две формы. @hodkoff2 посмотри пожалуйста. Если есть замечания по ним, то на каждую форму с замечанием заводи отдельную задачу и замечания описывай там. Эту же задачу пока не закрываю, чтобы дообсуждать путаницу по балансам.

Расчеты балансов остались прежними: