Резервирование. Баг или Фича?

[raz]

В связи с разбором возникавших у нас ошибок резервирования, обнаружил:

класс InventDimFixedClass

void initInventDimParm(InventDimParm _inventDimParm)
{
    ;
    _inventDimParm.data(inventDimParm);
}

ИМХО должно быть так:

void initInventDimParm(InventDimParm _inventDimParm)
{
    ;
    // edd ErrFix -->
    //_inventDimParm.data(inventDimParm);
    inventDimParm.data(_inventDimParm);
    // edd ErrFix <--
}

кто нибудь сталкивался?

[mazzy]

Цитата:

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

В связи с разбором возникавших у нас ошибок резервирования, обнаружил:

Спасибо. Стопудовая бага.

Метод initInventDimParm вызывается только из двух других методов.

[raz]

Цитата:

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

Спасибо. Стопудовая бага.

Метод initInventDimParm вызывается только из двух других методов.

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

[Dron AKA andy]

Коллеги, не торопитесь...
Судя по всему (глубоко пока не лез), метод делает именно то, что от него требуется - инициализирует поля переданного в него по ссылке InventDimParm. Надо бы разобраться получше с этим.

[belugin]

Просто мысль: мне всегда нравился обычай назвать методы не по-английски.

Вместо initByXXX в аксапте принято initXXX именно поэтому такое разночение и возникло...

[Raven Melancholic]

Цитата:

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

ИМХО должно быть так:

А вы не заметили, что после этой модификации у вас:
а) перестали запоминаться настройки отображения аналитики в формах;
б) инвентаризации "забыли" по каким аналитическим разрезам они создавались;
в) перестали работать транспортировки паллет.
Судя по всему, проблемы с резервированием у вас возникают по какой-то другой причине.

[raz]

Согласен с Dron AKA andy и с belugin.

[Maxim Gorbunov]

Присоединяюсь к Raven. Не баг это, так и должно быть.

Добавлю, что еще перестанут работать:
г) настройки складской аналитики в печатных формах в расчетах с клиентами/поставщиками;
д) выбор аналитики для печати в отгрузочной накладной (picking list) при старте производственного заказа.

raz, расскажите поподробнее, когда у Вас сброс флагов происходил?

[raz]

Цитата:

Сообщение от Maxim Gorbunov

raz, расскажите поподробнее, когда у Вас сброс флагов происходил?

Как оказалось все было ОК.
У нас используется самописная scm, в закупках планируется цепочка поставки (создаются переносы и заказ), в строке закупки есть ссылка на маршрут, так вот при смене маршрута цепочка должна перепланироваться. Ну и в результате многочисленных хотелок пользователей что то сломали. Мне задачу поставили: "ошибка с резервированием". При анализе наткнулся на выше указанный код и судя по тесту кода подумал что в нем ошибка, т.к. именно в этом месте слетал флаг у склада (точнее все флаги аналитик), далее запрос шел без учета аналитик и резервировались левые проводки. Сейчас вроде разобрались.
Резервирование не трогали, ошибка в наших наработках.