Возврат денег из кассы

[sedovalx]

@hodkoff2 опиши задачу тут

[sedovalx]

Входит в план минимум #95

[sedovalx]

Я полагаю, должна быть сущность "Возврат денег":

• сумма
• дата возврата
• аренда, по которой был возврат (?)

По идее все возвраты должны быть привязаны к арендам. Верно?

[sedovalx]

Клиент может вызывать операцию возврата, сохранив новый объект типа Refund. Данный тип еще не реализован на сервере. Описание типа будет приведено в данной задаче после реализации серверной поддержки возвратов.

[sedovalx]

Сущность "Возврат денег" (Refund) добавлена, вот описание для клиента:

DS.Model.extend({
  changeTime: attr("date", {
    defaultValue: () => new Date()
  }),
  amount: attr("number"),
  rent: DS.belongsTo("rent", {inverse: null, async: true}),
  comment: attr("string"),
  creationDate: attr("date"),
  editDate: attr("date"),
  creator: DS.belongsTo("user", {inverse: null, async: true}),
  editor: DS.belongsTo("user", {inverse: null, async: true}),
});

[sedovalx]

@Argelein далее дело за тобой. В https://github.com/sedovalx/taxi/issues/97#issuecomment-122430358 описано, когда выполняется операция по возврату. Чтобы непосредственно выполнить возврат на клиенте нужно:

• создать объект типа refund в store
• заполнить ему поля changeTime (дата возврата), amount (сумма возврата) и rent (по какой аренде возврат)
• сохранить в store

[sedovalx]

Продолжим доделки по возврату денег по аренде тут, т.к. все сделали эту операцию с UI-мастером, вызываемым из редактора аренды.

[sedovalx]

Что кажется уместным:

• :white_check_mark: При нажатии на кнопку "Возврат" отображать мастер возврата с предзаполенным полем суммы.
• :white_check_mark: Значение в поле суммы должно равняться балансу аренды, если он положителен, и нулю, если баланс отрицательный.
• :white_check_mark: Значение суммы можно редактировать
• :white_check_mark: Если сохранение выполняется с указанием суммы меньше нуля или больше суммы в кассе, то отображается ошибка валидации, и сохранения не происходит.
• Если сохранение возврата прошло успешно, то аренда автоматом переходит в состояние "Закрыта".

[sedovalx]

:white_check_mark: Кроме того, было бы неплохо отображать в мастере возврата итоговый баланс аренды. Для информации.

[sedovalx]

@hodkoff2 что думаешь?

[sedovalx]

@hodkoff2 посмотри, пойдет ли

[hodkovdd]

в продолжении замечания о том что после перевода аренды в состояние "Закрыта" аренду можно вернуть в Активное (описана здесь https://github.com/sedovalx/taxi/issues/110#issuecomment-131933780)

Я сейчас по кругу одну и ту же аренду гонял:

• переводил в состояние "Под расчет"
• нажимал кнопку "Возврат"
• "Сохранить"
• Состяние "Активна"
• проверял состяние счета "В Кассе" (действительно сумма уменьшалась верно)
• снова переводил в сотояние "под Расчет"
• снова кнопка "Возврат"
• ...

В общем, нужно

• терминировать аренду сразу по переводу в состояние "Закрыта"
• возврат денег делается не отдельной кнопкой, а ВМЕСТЕ с переводом аренды в состяние "Закрыта" - это и есть условие ее закрытия - полный расчет с водителем! (перевод в состояние "Закрыта" проходит через подтверждение типа "вы уверены? -ДА/НЕТ")
• обратного пути из состояния "Под Расчет" быть не должно! Если мы прекращаем отношения с нанятым водителем, то мы переводим его аренду в состояние "под Расчет".

Почему не рассчитаться сразу и зачем это непонятное промежуточное состояние? Да потому что нам нужен таймаут в 2 недели за которые могут придти куча штрафов на водителя, и если мы с ним сразу рассчитаемся, то эти штрафы придется из своего кармана оплачивать.

[sedovalx]

Кажется, я тебе не то проверять сказал. Имел в виду задачу съема денег из кассы #91. Посмотри ее, пожалуйста. А возврат еще не готов, хотя то, что ты обратил внимание на эти проблемы - хорошо.

[sedovalx]

Переформулирую доделки более понятно для меня:

• Если аренда переведена в состояние "Под расчет", то не должно быть возможности перевести ее любое состояние кроме "Закрыта". Вопрос @hodkoff2: а если по ошибке под расчет отправили?
• При переводе аренды в состояние "Закрыта" у пользователя должно запрашиваться подтверждение. Если он подтверждает закрытие аренды, то ему отображается форма возврата. Если там он нажимает Сохранить, то осуществляется возврат указанной суммы, а аренда необратимо переводится в закрытое состояние. При этом машина должна освободиться для новых аренд. Если на форме возврата пользователь нажимает Отмена/нажимает кнопку Назад в браузере/закрывает браузер, то возврата и закрытия аренды не происходит, аренда остается под расчетом.

[sedovalx]

Мысли вслух. При закрытии аренды нам нужно дожидаться осуществления факта возврата и только после этого, в случае успешного осуществления возврата, закрывать аренду. Если делать переход к новому url формы возврата, то непонятно, как дожидаться завершения. Возможно имеет смысл форму возврата показывать в виде popup-окошка в шаблоне и контроллере управления арендой.