Сводная таблица по выбранной в КС Аренде

[hodkovdd]

@sedovalx @Argelein

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

Отсюда навигация на форму с детализацией по выбранной операции.

[sedovalx]

А не дубль ли это? См. https://github.com/sedovalx/taxi/issues/85

[sedovalx]

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

[sedovalx]

Фильтра на типам операций нет, но нужен ли?

[hodkovdd]

@sedovalx Попробовал. Фильтра на типам операций нет, но нужен ли? Пока нет, если нужен будет заказчик сам скажет, тут впереди паровоза не стоит. Замечания:

• время платежей всегда 0:00

---

• При переходе на детализацию операции нет поля "Внес" с указанием пользователя, создавшего операцию:

[sedovalx]

С клиента неверное время приходит. Все время полночь. @Argelein нужно поправить поле ввода даты для операции.

[hodkovdd]

@sedovalx @Argelein Сейчас вообще нет данных об операциях (ни платежей ни списаний):

[sedovalx]

@hodkoff2 Пока не заводи новых данных на стенде, я посмотрю запросы.

[sedovalx]

@hodkoff2 проблема в том, что при создании операции на сервер отправляется неверное время ее создания. Это из-за того компонента, что мы используем для выбора времени. Точнее для выбора даты, т.к. он сейчас позволяет выбирать именно дату, а не время. В результате имеем сбои в расчете истории операций. @Argelein уже отметил эту проблему.

[hodkovdd]

@sedovalx @Argelein Кто и когда ее решит?

[sedovalx]

@hodkoff2 она уже должна быть решена https://github.com/sedovalx/taxi/issues/100

[kirzas]

@sedovalx Здесь остается последнее замечание от Димы https://github.com/sedovalx/taxi/issues/94#issuecomment-120768838, пункт 2: При переходе на детализацию операции нет поля "Внес" с указанием пользователя, создавшего операцию

Я попробовал добавить creator, но там пусто. Полагаю ,что сервер при запросе q-rent-history не отдает этого свойства.

[sedovalx]

Все верно, сервер не отдает сейчас. Но поле на сервере заполняется. Так что нужно только немного поправить q-rent-history на сервере. Но самостоятельно вы вряд ли это сделаете.

[hodkovdd]

@sedovalx @Argelein

1. ... уже должна быть решена https://github.com/sedovalx/taxi/issues/100... Не решена, операций нет, переоткрыл.

[sedovalx]

@Argelein у тебя тоже истории операций не видно?

[kirzas]

у меня видно, сейчас еще раз проверил, скрин внизу

[kirzas]

@hodkoff2 Дим, проверь, что ты создал для аренды, которую смотришь списания и платежи.

[hodkovdd]

@Argelein @sedovalx Проверил сейчас, операции отображаются, Остался вопрос о том что же было, в себе уверен, перед тем как переоткрыть все 3 раза проверил, скриншоты доказывают, что аренда та. Если это баг то еще всплывет. Теперь не понятно зачем здесь кнопка Сохранить - сводка доступна только для чтения, тут нечего сохранять.

[kirzas]

после решения #97 тут все будет немного по-другому, сохранить либо не будет, либо будет зачем :) тут забей на нее.

[kirzas]

Итак :) остается только один момент:

---

Здесь остается последнее замечание от Димы #94 (comment), пункт 2: При переходе на детализацию операции нет поля "Внес" с указанием пользователя, создавшего операцию

---

Я попробовал добавить creator, но там пусто. Полагаю ,что сервер при запросе q-rent-history не отдает этого свойства.

---

Все верно, сервер не отдает сейчас. Но поле на сервере заполняется. Так что нужно только немного поправить q-rent-history на сервере. Но самостоятельно вы вряд ли это сделаете.

[hodkovdd]

Ждем Сашу тут?

[kirzas]

ага или можешь попробовать сам :)

[sedovalx]

Сегодня-завтра займусь этим.

[hodkovdd]

@sedovalx а что значит null null в поле "внес"? это псевдоним admin?

[sedovalx]

Это значит, то имя и фамилия для админа не заполнены.

[hodkovdd]

@sedovalx Там же сейчас есть кнопка Сохранить Я поменял все даты операций, часть пропала потому что я перенес их в будущее, часть переехала в прошлое. Непорядок В сводке не должно быть возможности менять что либо. К тому же можно менять сумму!)))

[sedovalx]

Тогда вовсе не останется возможности что-то подправить в операциях. А такая необходимость всегда возникает.

[hodkovdd]

Как ты себе это представляешь?

[sedovalx]

В смысле зачем подправлять? Ошиблись в записи суммы/не тот тип операции указали/явку забыли проставить и сохранили. Дату наверное не имеет смысла давать редактировать, хотя хз.

[hodkovdd]

Какое это будет доказательство наличие\отсутствия платежей и их количества, если их можно менять в любой момент? Если кто то ошибается в момент ввода, отмена операции быть должна, безусловно, через админа. Пример: 1.В момент платежа водитель передает деньги кассиру. 2.Кассир вносит данные, сохраняет их. Водитель убеждается что его платеж зафиксирован верно, спокойно уходит. ... n. Возникает конфликт (Водитель: Как не платил на прошлой неделе, еще как платил!) n+1. Кассир показывает сводку по аренде где нет платежа. n+2. Водитель парирует - Вы же б_я здесь все меняете постоянно на_й. не*т! Тупик...

[sedovalx]

Ну ведь так же и через админа можно поменять, а результат -> n+1 -> n+2. Разве нет? Если водитель хоть раз заметит, что что-то в записях поменялось (пусть даже ошибка исправилась), то результат тот же.

[sedovalx]

Можно конечно режим просмотра операции сделать по умолчанию, а если ты админ, то тебе в режиме редактирования открывается.

[hodkovdd]

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

[sedovalx]

Хорошо, тогда для всех кроме админа сделаем только просмотр. Эта задача относится к #96 больше, но точно не сюда. Завел новую #111