Хочу поделится

[Vitaly_K]

Привет всем !

Не знаю обсуждалось ли такая ситуация раньше, но признаться был удивлен ею!

На днях случайно столкнулся с такой проблемой!

В системе 1 пользователь создал новый заказ (salesTable) и больше ничего с ним не делал и не обновлял форму с заказами.
2-й пользователь у себя обработал этот же заказ.
Через какое-то время 1-ый пользователь решил удалить у себя этот заказ.Нажал крестик удаления и как ни странно удалил его, хотя обработанный заказ нельзя удалить.
Решил протестировать эту ситуацию и действительно оказалось, что возможно таким образом удалить заказ.

Проблему решил так: при удалении заказа, сначала обновлять источник данных (salestable_ds)

Но все таки очень странно, почему так произошло. Получается заказ удаляется из формы,а не из базы данных???????

[George Nordic]

А что тут странного??
Заказ - это черновик. Главное - накладные. Заказ можно удалить. Накладные - только сторнировать. Заказ, в зависимости от настроек, или удаляется совсем, или переходит в статус "Удаленный".

С Уважением,
Георгий

[Vitaly_K]

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

[KMV]

И этот статус при удалении проверяется на DataSource формы?

[simply2double]

Цитата:

Сообщение от George Nordic

А что тут странного??
Заказ - это черновик
...

Скорее не черновик... а некая папочка, объединяющая разнородные документики по одной теме... в частности по операции с клиентом. Плюс набор лекал, по которым можно эти самые документики быстро создавать.

Существует мнение, что заказы после проведения накладных надо удалять. Такой подход нам даже рекомендовали консультатнты из CITP. Единственное членораздельное обоснование, которое я слышал, этого уменьшение числа записей таблиц SalesTable и SalesLine... и как следствие уменьшение размера БД. Плата за это - свалка из накладных... счетов... фактур.. и прочих документов. А уж если использовать при создании заказов/закупок механизмы Intercompany концы вообще найти невозможно...

[simply2double]

Цитата:

Сообщение от Vitaly_K

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

Значит такой у вас функционал ))))

PS Добавь в таблицу SalesTableDelete поле, к примеру DeletedUserId, в которое пиши UserId удалившего заказ...
Примерно так:

// таблица SalesTableDelete
void initFromSalesTable(SalesTable _salesTable)
{
    this.salesId        = _salesTable.salesId;
    this.salesName      = _salesTable.salesName;
    this.custAccount    = _salesTable.custAccount;
    this.invoiceAccount = _salesTable.invoiceAccount;
    this.salesType      = _salesTable.salesType;
    this.projId         = _salesTable.projId;
    //вот он красавчик-->
    this.DeletedUserId  = CurUserId ( );
    //<--

    this.salesTable     = [_salesTable];
}

потом показательно выпори удалившего... и поверь больше ни одного заказа удалено не будет ))))

[macklakov]

Цитата:

Сообщение от simply2double

PS Добавь в таблицу SalesTableDelete поле, к примеру DeletedUserId, в которое пиши UserId удалившего заказ...

2Vitaly_K: Не слушайте Вы его, есть Администрирование-Настройки-Журнал базы данных

[simply2double]

Цитата:

Сообщение от macklakov

2Vitaly_K: Не слушайте Вы его, есть Администрирование-Настройки-Журнал базы данных

Ну по большому счету, в данном случае достаточно просто в свойствах таблицы SalesTableDelete включить свойства CreatedBy, CreatedDate, что бы видеть кто и когда грохнул заказ. И журналирование здесь совершенно необязательно, ибо это и есть почти журнал. Но ведь ветка называется ПРОГРАММИРОВАНИЕ.. вот и хотелось попрограммировать ))))

[Vitaly_K]

Извиняюсь, что отсутствовал на форуме

Сообщение от KVM
И этот статус при удалении проверяется на DataSource формы?

Вот именно, что хоть функционал у нас переписан, но delete вызывается из DataSource формы думаю также как и у всех. просто заказ не удаляется с определенным статусом,а тут на тебе казус.
Решение я нашел просто - перед методом super() делаю salesTable_ds.reread().....Но ведь это не правильно, когда если не обновил бы форму, то возможно удалить заказ.



Сообщение от simply2double
PS Добавь в таблицу SalesTableDelete поле, к примеру DeletedUserId, в которое пиши UserId удалившего заказ...

Ну это мне совсем не надо, все логи ведутся в журнале БД

[KMV]

Можно тогда поподробнее как реализована проверка этого статуса перед удалением?

[simply2double]

Цитата:

Сообщение от Vitaly_K

Сообщение от simply2double
PS Добавь в таблицу SalesTableDelete поле, к примеру DeletedUserId, в которое пиши UserId удалившего заказ...

Ну это мне совсем не надо, все логи ведутся в журнале БД

Интересно каков размер этих самых логов.. если они все ведутся

[Vitaly_K]

Сообщение от KVM
Можно тогда поподробнее как реализована проверка этого статуса перед удалением?

void  delete()
{
    SalesTable            deleteSalesTable;
    SH_DialogReason       sh_DialogReason;
    SH_ReasonCode              sh_ReasonCode;
    ;
    salesTable_ds.reRead();// то что я вставил

    if (salesTable.checkUpdate())
    {
        deleteSalesTable.data(SalesTable);

   
        if (salesTableType)
        {
            salesTableType.sh_setReason(sh_ReasonCode);
            salesTableType.formMethodDataSourceDelete(element, salesTable_ds);
        }

        super();
        if(deleteSalesTable.RecId)
        {
            SalesTable.sh_transferTo( SH_ActionType::Delete, deleteSalesTable);
        }
    }
}

[mazzy]

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

например, введенный вами текст
[xpp]while select LedgerTable
where LedgerTable.Accountnum == '01.01'[/xpp]

будет отображен форумом

while select LedgerTable
    where LedgerTable.Accountnum == '01.01'

[Vitaly_K]

ок, учту

[wb]

Цитата:

Сообщение от Vitaly_K

Ну это мне совсем не надо, все логи ведутся в журнале БД

Позвольте поитересоваться,
а как Вы по логу в журнале БД определяете кем и что было изменено в заказе?

[KMV]

2 Vitaly_K: подтверждаю проблему и в стандартном функционале

1. Первый пользователь создает заказ
2. Второй пользователь меняет в нем статус (или любое другое поле)
3. Первый пользователь удаляет текущий заказ

Смотрим в отладчике , в методе validateDelete таблицы SalesTable все изменения сделанные на 2 шаге в таблице не видны.

Кэш?

[Vitaly_K]

Цитата:

Сообщение от KMV

2 Vitaly_K: подтверждаю проблему и в стандартном функционале

1. Первый пользователь создает заказ
2. Второй пользователь меняет в нем статус (или любое другое поле)
3. Первый пользователь удаляет текущий заказ

Смотрим в отладчике , в методе validateDelete таблицы SalesTable все изменения сделанные на 2 шаге в таблице не видны.

Кэш?

KMV спасибо за участие, а то уже про логи начали спрашивать и отклонились от темы

И я про это думаю, что Кэш.... Как с этим бороться понятно, но ведь в чистом виде баг получается.
Вот и хотелось озвучить проблему! Возможно кому-то пригодится