Глюк в резервировании?

[maxsmirnov]

очевидно глюкавый код:

SalesLine.reserveQty()

PHP код:

InventQty   reserveQty()
{
    InventQty   qty;
    ;

    if (! this.reservation)
        qty   = 0;

    if (this.remainInventPhysical < 0)
        qty   = 0;

    qty   = -this.remainInventPhysical;

    return qty;
} 


наткнулся на него при раскапывании причины того, почему у меня при создании заказов по контракту все позиции автоматически резервируются, несмотря на то что в параметрах расчетов с клиентами стоит резервирование "вручную"

ps: ax3 cis sp3 hf1

[Wamr]

Точно такой же код в 2.5 (без SP)
Сомневаюсь, что это глюк, скорее всего, так и было задумано.
Это метод, который возвращает количество, которое теоретически может быть зарезервировано по данной строке заказа.
Надо смотреть как используется данное число дальше.

[mazzy]

Цитата:

Изначально опубликовано maxsmirnov
очевидно глюкавый код:
....
наткнулся на него при раскапывании причины того, почему у меня при создании заказов по контракту все позиции автоматически резервируются, несмотря на то что в параметрах расчетов с клиентами стоит резервирование "вручную"

ps: ax3 cis sp3 hf1

А почему ты считаешь, что код глюкавый?
параметры клиента вообще говоря не при чем.

позиция резервируется, когда в СТРОКЕ написано авторезервирование
в строку этот параметр передается из заголовка

но при копировании из контракта авторезервирвание в строки заказа должно передаваться из строк контракта.
В строки контракта они должны были педаться из заголовка контракта.

вопрос: что стоит в строках контракта? что стоит в заголовке контракта?
вопрос: кто-нибудь правит заголовки или строки контракта или заказа? а программно?

[maxsmirnov]

извиняюсь, не совсем ясно изложил свои соображения
тот-же код, но с комментариями:

PHP код:

InventQty   reserveQty()
{
    InventQty   qty;
    ;

    if (! this.reservation) //если стоит резервирование вручную
        qty   = 0; //тогда ноль, но это значение не возвращается никогда!

    if (this.remainInventPhysical < 0)
        qty   = 0;

    qty   = -this.remainInventPhysical; //потому что здесь оно переписывается

    return qty;
} 


[mazzy]

опа. действительно прикольно.

[maxsmirnov]

в строках и контракта, и заказа на отпуск стоит "вручную"
все делается не программно

меня смущает отсутствие return в двух местах.
т.е. не смущает, я, когда постил сообщение, скорее не задавал вопрос, а ставил общественность в известность о глюке.

теперь я уже не уверен
а именно - смущает то, что такой глюк прожил аж со времен 2.5 без СП

[mazzy]

Цитата:

Изначально опубликовано maxsmirnov
меня смущает отсутствие return в двух местах.

Да, я сразу и не понял.

Теперь смотри что происходит:
метод SalesLine.ReserveQty() вызывается только из одного места - из InventUpd_Estimated.updateReservation()
А там уже есть проверки.
Поэтому код в ReserveQty неправильный, но сейчас эта неправильность латентна и никак не проявляется.

[maxsmirnov]

InventUpd_Estimated.updateReservation() - на это там проверок нет.

т.е. глюк налицо - при создании заказа на отгрузку по контракту, в котором в строках и в шапке стоит резервирование "вручную", создается заказ со строками, в которых указано резервирование "вручную", тем не менее, происходит резервирование номенклатуры в кол-ве указанном remainInventPhysical строки.
(в параметрах модуля тоже стоит "вручную")

глюк лечится добавлением return-ов в SalesLine.ReserveQty()

[mazzy]

спасибо.

[Wamr]

Действительно, кривой код, спасибо.

Цитата:

Изначально опубликовано maxsmirnov
InventUpd_Estimated.updateReservation() - на это там проверок нет.

Проверка есть - movement.mustBeAutoReserved()

Цитата:

а именно - смущает то, что такой глюк прожил аж со времен 2.5 без СП

К сожалению, это не единичный случай

Цитата:

глюк лечится добавлением return-ов в SalesLine.ReserveQty()

Это понятно, но ведь теперь неплохо бы проверить все места в системе, откуда этот метод вызывается.

[maxsmirnov]

SalesLine.reserveQty() вызывается только из одного места (см. выше)
я тут еще вот какую прелесть нашел:
smmQuotationLine.reserveQty()

PHP код:

public InventQty  reserveQty()
{
    InventQty   qty;
    ;

    if (!this.reservation)
    {
        qty = 0;
    }

    if (this.remainInventPhysical < 0)
    {
        qty = 0;
    }

    qty     = -this.remainInventPhysical;

    return qty;
} 


было-бы одинаково с точностью до символа - я бы подумал что код скопирован из SalesLine...
загадки во тьме

[Wamr]

Цитата:

Изначально опубликовано maxsmirnov
SalesLine.reserveQty() вызывается только из одного места (см. выше)

И это место дорого стоит InventUpd_Estimated популярный классик

Цитата:

я тут еще вот какую прелесть нашел:
smmQuotationLine.reserveQty()

Вот видишь, традиция у них такая .. Best Practice

[miklenew]

Интересно, а данный код в Ax 4.0 или Ax 5.0 исправили.
Может кто-нибудь у кого стоит посмотрит.

[Sada]

В 4 исправлено

InventQty   reserveQty()
{

    if (! this.Reservation)
        return 0;

    if (this.RemainInventPhysical < 0)
        return 0;

    return -this.RemainInventPhysical;

}

[miklenew]

Спасибо.