Странный код в InventSum.update() и deadlocks

[Shannon]

[Kernel version — AX 4.0.2503.1109, application version — 4.0.2503.828, MS SQL]

В изрядно измененном приложении возникли блокировки номенклатуры, я поотлаживал, и наткнулся в таблице InventSum метод с таким кодом:

           
           ostPrice = postedQty ? postedValue / postedQty : 0;

           select firstonly forupdate inventTable
                      where inventTable.ItemId == itemId;

            if (inventTable && costPrice)
            {
                inventTable.CostPrice = costPrice;
                inventTable.update();
           }

}

Этот метод вызывается из InventSum.update(). Это только мне кажется, что это не очень правильно — вызывать из такого часто вызываемого метода, как InventSum.update(), какие-то апдейты? Может это быть причиной блокировки?

[mazzy]

Цитата:

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

Это только мне кажется, что это не очень правильно

Это не только "не очень правильно". Это варварство какое-то. И абсолютно неправильно.

Цитата:

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

           
                inventTable.CostPrice = costPrice;

1. В стандартном приложении ax4 у таблицы InventTable нет поля CostPrice
2. В стандартном приложении ax4 методы таблицы InventSum не записывают значение CostPricePcs, а только читают данные (причем только из InventSum и из таблицы моделей)

Скорее всего, у вас дико переписанная себестоимость.
И у вас еще будет масса поводов для удивления.

[oip]

Это ваш код, а не стандартный. В стандартне нет никакого inventTable.CostPrice и ничего левого в инвентСам.апдейт не апдейтится. Нет, это неправильно.

Upd. Опередили.

[Shannon]

Одна поправка — не «наш», а «их». Мне, похоже, посчастливилось разгребать «необратимые улучшения», внесенные людьми, для которых нет ничего святого.

[Shannon]

Цитата:

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

Это не только "не очень правильно". Это варварство какое-то. И абсолютно неправильно.

Спасибо. К счастью, у клиента есть программисты, которые понимают по-русски, и которые знают, кто такой Маззи. Теперь мне проще будет объяснить, что я не «просто придираюсь»