Epic Fail Остатки на дату InventSumDateValueReportDim

[Evgeniy2020]

Epic Fail Остатки на дату InventSumDateValueReportDim

Здравствуйте,
была простая задача: получить остатки на определенную дату.
в компании используется складская аналитика Склад. (только).
Задачка простая на 30.09.2010 получить остатки по номенклатуре вида:
Номенлкатура, Склад, Количество, Сумма

по идее все должно работать за счет стандартного класса InventSumDateValueReportDim

как бы не так
есть номенклатура 'ЦТ0085140'
по ней движение появилось 05.11.2010
до этой даты 05.11.2010 НЕТ ПРОВОДОК в InventTrans с такой номенклатурой по любой складской аналитке

05.11.2010 куплено 20 штук
20.11.2010 продано 1 штука
на текущий день 19 штук осталось


на 30.09.2010 по этой номенклатуре должно быть ровно 0
по всем складским аналитикам.

но увы.

 InventDimId _inventDimId;
    date        _date;
    itemId      _itemId;
    Qty qty, qty2;
    InventSumDateValueReportDim     Invent; // класс расчета на определенную дату
    InventDim                       inventDimLoc;
    InventDimParm                   inventDimParmLoc;
    ;
    _itemId = 'ЦТ0085140';
    _inventDimId = 'ЦАН00069498';
    _date = mkdate(30,9,2010);
    inventDimParmLoc.InventLocationIdFlag = NoYes::Yes;
    inventDimParmLoc.InventGTDIdFlag_RU   = NoYes::No;
    inventDimLoc = inventDim::find(_inventDimId);
    Invent = InventSumDateValueReportDim::newParameters(
        _date,
        _itemId,
        inventDimLoc,
        inventDimParmLoc);
    qty  = Invent.PostedQty() - Invent.deductedQty() + Invent.receivedQty();
    qty2 = Invent.PostedQty();
   info(StrFmt("%1  %2",qty ,qty2));

упорно возвращает 19


а вот такой код возвращает все верно 0

 inventTrans   inventTrans;
 
    select sum(Qty) from inventTrans
    group by inventDimId, itemId
    where
    inventTrans.ItemId == 'ЦТ0085140' &&
    inventTrans.DatePhysical <= mkdate(30,09,2010) &&
    inventTrans.StatusIssue <= StatusIssue::Deducted &&
    inventTrans.StatusReceipt<= StatusReceipt::Received &&
    inventTrans.inventDimId == 'ЦАН00069498';
    info(strfmt("%1", inventTrans.Qty));

Вопрос: неужели на стандартный класс нельзя положиться,
если движение по номенклатуре началось позже указанной даты?
выдает ерунду.

[petr]

Цитата:

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

_inventDimId = 'ЦАН00069498';
...
inventDimLoc = inventDim::find(_inventDimId);

Никогда такого больше не пишите, даже в джобах!!!

должно быть inventDimLoc.InventLocationId = "Склад по которому хотите узнать остатки"

inventDimLoc = inventDim::find(_inventDimId); - просто удалите эту строчку.

Заработало?

P.S. Извините, если резковато получилось.

[Vadik]

Ставьте KB2215421 или RU6

Цитата:

If you backdate the reports to a date before the item was created for the first in, first out (FIFO) items, physical value adjustments are incorrectly calculated. The physical value adjustments that are posted in the future period are not reversed in the reports. These amounts are reflected in the posted physical value and in the inventory value in the reports. Therefore, even though no item exists, the reports show data.

[Evgeniy2020]

to Petr:

исправил аналитику, но ситуацию не поменялась.
также 19 штук возвращается.

при этом это ax 3.0 sp 4
странно что такое раньше не открывалось.

[lev]

Цитата:

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

...

Цитата:

If you backdate the reports to a date before the item was created for the first in, first out (FIFO) items, physical value adjustments are incorrectly calculated. The physical value adjustments that are posted in the future period are not reversed in the reports. These amounts are reflected in the posted physical value and in the inventory value in the reports. Therefore, even though no item exists, the reports show data.

Расчет остатков на дату обсуждался на форуме уже много раз (достаточно посмотреть внизу на список похожих тем). Но что странно, НИ В ОДНОЙ из тем не нашел упоминание про указанную фичу.
Или плохо искал, или это очевидно для всех или ещё что...

[Vadik]

Цитата:

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

Но что странно, НИ В ОДНОЙ из тем не нашел упоминание про указанную фичу

А я люблю, как в старые добрые времена, холодными зимними вечерами с трубочкой и бокалом коньяка у камина почитать свежий fixlist или rss из knowledge base


А если серьезно - ну был косяк, ну исправили, чего ж тут обсуждать - надо проверить на себе и исправить

[Raven Melancholic]

Цитата:

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

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

Увы, у автора:

Цитата:

ax 3.0 sp 4

Можно выложить сюда все операции по этой номенклатуре и сопоставления (например, в файле Excel)?

[Evgeniy2020]

2 Raven Melancholic:

у меня простая ситуация в верхний джоб подставляю любую номенклатуру
по которой движение началось позже даты 30.09.2010

взял для примера следующую номенклатуру у которой
первая закупка в Inventrans 10.11.2010

и те же грабли
в моем случае ерунда выдается если взять любую номенклатуру
по которой движение начинается после указанной даты (в данном случае 30.09.,2010 т.е. даты на которую получаем остатки)

попробуйте у себя верхний джоб и номенклатуру возьмите у которой
первая проводка в InvenTrans > 30.09.2010

[lev]

Цитата:

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

2 Raven Melancholic:

у меня простая ситуация в верхний джоб подставляю любую номенклатуру
по которой движение началось позже даты 30.09.2010

взял для примера следующую номенклатуру у которой
первая закупка в Inventrans 10.11.2010

и те же грабли
в моем случае ерунда выдается если взять любую номенклатуру
по которой движение начинается после указанной даты (в данном случае 30.09.,2010 т.е. даты на которую получаем остатки)

попробуйте у себя верхний джоб и номенклатуру возьмите у которой
первая проводка в InvenTrans > 30.09.2010

я у себя воспроизвел вашу ситуацию, только использовал класс InventSumDatePhysicalDim... та же самая картина. эта проблема распространяется на всю иерархию классов InventSumDate* видимо.

З.Ы. Ax 3.0 SP3

[Raven Melancholic]

Цитата:

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

попробуйте у себя верхний джоб и номенклатуру возьмите у которой
первая проводка в InvenTrans > 30.09.2010

Взял. Не повторилось (DAX4 SP2)

[Evgeniy2020]

Тогда остается найти отличия в Иерархии с тем что есть у Raven Melancholic.
я возможно загляну в другие версии и попробую сравнить, а также с хотфиксом.
после этого станет ясно

[Logger]

Попробовал багу, не воспроизвелось.
По ходу дела пока разбирал алгоритм работы классов, наткнулся на фильтрацию по InventTrans.valueOpen:
\Classes\InventSumDatePhysicalDim\selectInventTransPicked
\Data Dictionary\Tables\InventTrans\Methods\qtyPickedDate

        select sum(Qty) from inventTrans
            where inventTrans.ItemId         == _itemId                    &&
                  inventTrans.StatusReceipt  == StatusReceipt::None        &&
                  inventTrans.StatusIssue    == StatusIssue::Picked        &&
                  inventTrans.ValueOpen      == InventTransOpen::Yes       &&
                  inventTrans.DateInvent     >  _perDate
        #inventDimExistsJoin(inventTrans.InventDimId,inventDim,_inventDimCriteria,_inventDimParm);

Интересно какой смысл фильтровать по InventTrans.valueOpen, особенно для отчетов выдающих остатки на дату. По идее, результат вычисления остатка на дату не должен зависеть от того закрыта проводка сейчас или нет.

[Cardagant]

Как вариант, повысить производительность работы с InventTrans, уменьшив, где возможно, количество обрабатываемых записей. Правда, с агрегацией по одному полю ускорение может быть незначительным.

[Logger]

По поводу ускорения я тоже подумал.
Но не ценой же корректности возвращаемого результата !

[Владимир Максимов]

Цитата:

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

По поводу ускорения я тоже подумал.
Но не ценой же корректности возвращаемого результата !

А в чем не корректность? Разве может быть закрыта проводка в статусе расхода "Скомплектовано"?

[Logger]

Да, вы правы. Я это пропустил. Анализировал как учитываются проводки у которых статус уже поменялся до Продано и забыл что здесь такие не участвуют.

Т.е. видимо в этом случае данное условие используют как хинт подсказывающий оптимизатору схватить высокоселективный в этих условиях индекс OpenItemIdx. А сама выборка не сужается.