Opened 16 months ago

Closed 8 months ago

Last modified 8 months ago

#1222 closed дефект (fixed)

БД: Отгрузка: функция указания даты срабатывает только на запись в Историю, а запись в таблице Заказы всегда делается с текущей датой

Reported by: san Owned by: Denis_N
Priority: major Component: БД изделий АДС
Keywords: Cc:

Description (last modified by san)

Change History (13)

comment:1 by san, 16 months ago

Description: modified (diff)
Summary: Отгрузка: функция указания даты срабатывает только на запись в Историю, а запись в таблице Заказы всегда делается с текущей датойБД: Отгрузка: функция указания даты срабатывает только на запись в Историю, а запись в таблице Заказы всегда делается с текущей датой

comment:2 by alx, 16 months ago

Несколько раз прочитал, но так и не понял, в чем заключается дефект. Запись делается с текущей датой. Таким образом, поле datetime в записи содержит дату создания заказа. Что не так?

Last edited 16 months ago by alx (previous) (diff)

comment:3 by san, 16 months ago

Недавно в интерфейс была добавлена функция "Отгрузка задним числом" - при отгрузке в истории изделий и заказе указывается не текущая дата, а заданная пользователем.
Баг в том что эта функция работала наполовину, в истории дата была та которую указали, а в заказах нет.

in reply to:  3 comment:4 by alx, 16 months ago

Replying to san:

Недавно в интерфейс была добавлена функция "Отгрузка задним числом" - при отгрузке в истории изделий и заказе указывается не текущая дата, а заданная пользователем.

Как странно... Не могу удержаться от вопроса - зачем?

comment:5 by san, 16 months ago

Например если заказ был срочно отгружен вчера, а в базе отмечают этот факт только сегодня.

in reply to:  5 comment:6 by alx, 16 months ago

Replying to san:

Например если заказ был срочно отгружен вчера, а в базе отмечают этот факт только сегодня.

Если этот комментарий адресован мне, то, прости, но он на мой вопрос не отвечает.

comment:7 by san, 16 months ago

зачем?

Для возможности указать в бд точную дату отгрузки

Last edited 16 months ago by san (previous) (diff)

in reply to:  7 comment:8 by alx, 16 months ago

Replying to san:

зачем?

Для возможности указать в бд точную дату отгрузки

А, так поле datetime предназначено для записи даты отгрузки заказа! А я думал, что оно для даты создания заказа... Теперь все понятно, спасибо за разъяснение...

Хотя нет, не все... Если там хранится дата отгрузки заказа, то что там записано у заказов, которые еще не отгружены? Ведь, насколько я вижу, это поле не может содержать NULL...

И еще непонятно, почему дату отгрузки надо вводить вручную - почему нельзя записывать туда текущую дату автоматически?

comment:9 by san, 16 months ago

datetime - создание заказа
deadline - дата планируемой готовности заказа, при отгрузке заменяется датой фактической отгрузки

И еще непонятно, почему дату отгрузки надо вводить вручную - почему нельзя записывать туда текущую дату автоматически?

На этот вопрос я телепатически ответил ранее :) иногда такая необходимость есть:

Например если заказ был срочно отгружен вчера, а в базе отмечают этот факт только сегодня.

По умолчанию(если не трогать чекбокс) при отгрузке устанавливается текущая дата

in reply to:  9 comment:10 by alx, 16 months ago

Replying to san:

datetime - создание заказа
deadline - дата планируемой готовности заказа, при отгрузке заменяется датой фактической отгрузки

А, спасибо за разъяснение. Я думал, что речь идет о datetime...

Тут я, конечно, мог бы удивиться, что поле, в котором хранится дата отгрузки заказа, называется deadline, но у нас там практически все очень странно называется... :) Я, например, до сегодняшнего дня думал, что в этом поле хранится максимально допустимая дата отгрузки заказа (именно это обычно называют deadline'ом)... :)

иногда такая необходимость есть:

Например если заказ был срочно отгружен вчера, а в базе отмечают этот факт только сегодня.

Мне все-таки непонятно, почему может возникнуть необходимость ждать следующего дня чтобы отметить отгрузку заказа вместо того чтобы отметить ее сразу после отгрузки. Ты только что устно говорил мне о замечании мной разных странностей, так вот это я нахожу странным. :) Впрочем, это уже, наверное, выходит за рамки темы тикета...

Last edited 16 months ago by alx (previous) (diff)

comment:11 by san, 8 months ago

Resolution: fixed
Status: newclosed

In 354/base:

При отгрузке "задним числом", установленная пользователем дата отгрузки записывается теперь и в Заказы.
Close #1222

in reply to:  11 comment:12 by alx, 8 months ago

Replying to san:

При отгрузке "задним числом", установленная пользователем дата отгрузки записывается теперь и в Заказы.

Извините что вмешиваюсь, но меня смущает наличие в коде установки "забитого гвоздями" часового пояса. Логика подсказывает, что такого никогда не должно требоваться. Нет ли здесь ошибки?

comment:13 by san, 8 months ago

Мне тоже это показалось неправильным, но мотороллер не мой, я просто перенёс кусок кода выше)
Думаю стоит создать отдельный тикет.

Note: See TracTickets for help on using tickets.