Пересчет inventSum

[glibs]

Цитата:

Сообщение от Хочулия

...
А можно ли из InventSum удалить пустые строки
...

Можно, только работать потом не будет.

Цитата:

Сообщение от Хочулия

...
или чем аукнется тоже?
...

Как минимум отчеты по номенклатуре по остаткам и стоимости запасов на дату "поведет".

[Logger]

Цитата:

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

Можно, только работать потом не будет.

Как минимум отчеты по номенклатуре по остаткам и стоимости запасов на дату "поведет".

Спасибо.
А вы не могли бы подробнее раскрыть тему ?

Покопал классы
InventSumDate
InventSumDateDim
InventSumDatePhysical
InventSumDatePhysicalDim
InventSumDateValue
InventSumDateValueReport
InventSumDateValueReportDim

не нашел нигде зависимости от наличия закрытых Inventsum.
Исключение - метод
\Classes\InventSumDateValueReportDim\newParameters

server static InventSumDateValueReportDim newParameters(
    TransDate       _perDate,
    ItemId          _itemId,
    InventDim       _inventDimCriteria,
    InventDimParm   _inventDimParm

    )
{
    InventSumDateValueReportDim inventSumDateValueReportDim = new InventSumDateValueReportDim();
    InventDimParm           inventDimParmGroupBy;
    Query                   query;
    QueryRun                queryRun;
    InventDim               inventDim;
    ;

    inventSumDateValueReportDim.parmItemId(_itemId);
    inventSumDateValueReportDim.parmPerDate(_perDate);
    inventSumDateValueReportDim.parmInventDim(_inventDimCriteria);
    inventSumDateValueReportDim.parmInventDimParm(_inventDimParm);

    if (InventTable::find(_itemId).ItemDimCostPrice == NoYes::Yes)
    {
        inventDimParmGroupBy = _inventDimParm.data();
        inventDimParmGroupBy.setActiveItemDim(InventTable::find(_itemId).DimGroupId);
        inventSumDateValueReportDim.parmInventDimParm(inventDimParmGroupBy);

        query = InventSum::newQuery(null,_itemId,_inventDimCriteria,_inventDimParm,inventDimParmGroupBy);
        query.dataSourceTable(tablenum(InventSum)).findRange(fieldnum(InventSum,Closed)).value('');

        queryRun = new QueryRun(query);
        while (queryRun.next())
        {
            inventDim = queryRun.get(tablenum(InventDim));
            inventSumDateValueReportDim.parmInventDim(inventDim);
            inventSumDateValueReportDim.init();
        }

        inventSumDateValueReportDim.parmInventDimParm(_inventDimParm);
    }
    else
    {
    inventSumDateValueReportDim.init();
    }

    return inventSumDateValueReportDim;
}

Не подскажете зачем было делать цикл

        while (queryRun.next())
        {
            inventDim = queryRun.get(tablenum(InventDim));
            inventSumDateValueReportDim.parmInventDim(inventDim);
            inventSumDateValueReportDim.init();
        }

для номенклатурных аналитик ?

[Pustik]

Цитата:

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

Можно, только работать потом не будет.

Как минимум отчеты по номенклатуре по остаткам и стоимости запасов на дату "поведет".

а пересчет InventSum разве не решит проблему?

[Logger]

Цитата:

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

а пересчет InventSum разве не решит проблему?

Дык...
В этом собственно и проблема. Что Inventsum большой и держать closed записи не хочется.

Все-таки есть разница 65 мультов записей в табличке или 2,5

[AraraT®]

Цитата:

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

Дык...
В этом собственно и проблема. Что Inventsum большой и держать closed записи не хочется.

Все-таки есть разница 65 мультов записей в табличке или 2,5

Может быть Вам имеет смысл сделать партиционирование таблицы на уровне SQL и положить Закрытые записи на другой диск, а на быстром диске оставить только открытые?

[Logger]

Цитата:

Сообщение от AraraT®

Может быть Вам имеет смысл сделать партиционирование таблицы на уровне SQL и положить Закрытые записи на другой диск, а на быстром диске оставить только открытые?

Возможно и так. Подумывал над этим.
Но мне кажется что хирургия это более надежный метод. Но более стремный.

А вы явные противопоказания к удалению закрытых записей знаете ?

[Kasper]

Задача проверки INVENTSUM возникает довольно часто.
Постарался решить эту проблему в общем случае. Написал статью
http://yaroslavbat.blogspot.com/2014...apta-2009.html
Хотел здесь новую тему создать (отклики собрать, если кто попробует), но прав нет
Отзывы буду приветствовать

[raz]

Цитата:

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

Задача проверки INVENTSUM возникает довольно часто.
Постарался решить эту проблему в общем случае. Написал статью
http://yaroslavbat.blogspot.com/2014...apta-2009.html
Хотел здесь новую тему создать (отклики собрать, если кто попробует), но прав нет
Отзывы буду приветствовать

Для этого есть класс InventSumReCalcItem. И периодическая операция.

[Perc]

Цитата:

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

Задача проверки INVENTSUM возникает довольно часто.

В 2009 не проверял. А акс3 наша которая работала 6 лет запускал проверку. Все сошлось с интвентрансом до последней циферки.

Цитата:

Постарался решить эту проблему в общем случае. Написал статью
http://yaroslavbat.blogspot.com/2014...apta-2009.html
Хотел здесь новую тему создать (отклики собрать, если кто попробует), но прав нет
Отзывы буду приветствовать

Ужос) Приветствуется как исследование акс и sql, но пользоваться этим промышленно не стоит)

[Kasper]

Цитата:

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

Дык...
В этом собственно и проблема. Что Inventsum большой и держать closed записи не хочется.

Все-таки есть разница 65 мультов записей в табличке или 2,5

Уважаемый Logger, попробуйте запустить мой скрипт, но НЕ делая commit -- пока только посмотрим логи. Я не могу вам квалифицированно ответить на вопрос, нужно ли держать closed записи, но может после запуска скрипта часть из 65 мультов сама уйдёт ? Не по причине, что они closed, а потому что они необоснованные (не базируются на INVENTTRANS)
Напишите, ушло ли что ни-будь и сколько, а ещё лучше, прикрепите сюда логи, там посмотрим...

[Logger]

Цитата:

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

Уважаемый Logger, попробуйте запустить мой скрипт, но НЕ делая commit -- пока только посмотрим логи. Я не могу вам квалифицированно ответить на вопрос, нужно ли держать closed записи, но может после запуска скрипта часть из 65 мультов сама уйдёт ? Не по причине, что они closed, а потому что они необоснованные (не базируются на INVENTTRANS)
Напишите, ушло ли что ни-будь и сколько, а ещё лучше, прикрепите сюда логи, там посмотрим...

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

Еще заметил что в скрипте есть строки вида

using T1 b on a.DATAAREAID = b.DATAAREAID and a.ITEMID = b.ITEMID and a.INVENTDIMID = b.INVENTDIMID

Вы умудрились свою аксапту запустить в режиме CaseInsensitive ?
Или просто забыли добавить
SUSTR(NLS_LOWER(a.DATAAREAID),1,4)
и
SUSTR(NLS_LOWER(a.ITEMID),1,20)

?

[Kasper]

Стоп !

Все Аксапты, которые я видел за последний год, работали в режиме CASE INCENSITIVE. Я вот поленился мануал почитать по установке SQL для Аксы, но считал что SQL для Аксы должен ставиться всегда регистронезависимым. Исходя из этого я и не делал приведение к регистру.

У Вас не так ?

На боевом приложении делать не надо

[Kasper]

Цитата:

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

Спасибо, посмотрю.
Вы умудрились свою аксапту запустить в режиме CaseInsensitive ?
?

Нет, похоже всё верно.
Вот, смотрю Axapta 2009 Installation Guide. Там написано:
During installation, you must choose a collation method that meets the following requirements:
Includes a sort order that is case-insensitive, accent-sensitive, kana-sensitive, and width-insensitive.

Поэтому у меня всё верно. Разве у Вас Акса работает в регистрозависимом режиме ? Мы говорим об Аксе 2009 ?

[Logger]

Я почему-то решил что у вас оракл.
Видимо из-за баннеров

Вот и спросил.
Скрипт просматривал только по диагонали.

Чем вас не устроил InventSumReCalcItem о котором raz написал ?

[Kasper]

Доработал пост в блоге, учёл все комментарии.
Годится ?

Цитата:

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

Чем вас не устроил InventSumReCalcItem о котором raz написал ?

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

Цитата:

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

А вы явные противопоказания к удалению закрытых записей знаете ?

Квалифицированный ответ можно получить только от вендора, но можно порассуждать.
Допустим есть чистая настроенная база. Пока ещё ни одной складской проводки.
В складской картотеке есть одна номенклатура -- "Ведро"
И вот вы купили одно ведро и сразу же его продали. Нажимаем кнопку "Запасы". Видим - Ведро, количество - 0.
А если закрытую строку из InventSum удалить, то в форме "Запасы" будет пусто. Так не должно быть.

Так ?

[Perc]

Цитата:

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

А если закрытую строку из InventSum удалить, то в форме "Запасы" будет пусто. Так не должно быть.

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

[S.Kuskov]

А разве закрытой строка становится не после сопоставления закрытием склада? По-моему алгоритм закрытия как-то использует информацию из InventSum? Или я ошибаюсь? И точно ли не понадобятся такие закрытые строки в InventSum в случае открытия (отмены закрытия) склада?

[Logger]

Цитата:

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

А если закрытую строку из InventSum удалить, то в форме "Запасы" будет пусто. Так не должно быть.

Так ?

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

Или доработать форму "Запасы", чтобы все равно показывала записи.

[Logger]

Цитата:

Сообщение от S.Kuskov

А разве закрытой строка становится не после сопоставления закрытием склада? По-моему алгоритм закрытия как-то использует информацию из InventSum? Или я ошибаюсь? И точно ли не понадобятся такие закрытые строки в InventSum в случае открытия (отмены закрытия) склада?

На Inventsum.Insert() и Inventsum.update()
есть такой код :

    this.closedQty = this.isAllQtyFieldsZero();
    this.closed    = this.isAllValueFieldsZero() && this.closedQty;

(еще есть аналогичный код в хранимке, которая обновляет остаток в InvnetSum)

А в методах тупо идет сравнение количественных и суммовых полей с нулем.
На закрытие склад ей пофигу. Просто как правило суммовые поля выравниваются после закрытия что приводит к их обнулению и взведению галки closed. Но такое может быть и без закрытия склада.

Закрытие склада, насколько я помню не использует InventSum.
Табличка InventSum упоминается в методах
\Classes\InventCostClosingCancel_WorkInvent\checkAndDeleteVirtualTransfer
\Classes\InventCostClosingCancel_WorkInvent\deleteVirtualTransfers

но не для выборки из базы или фильтрации, а как накопительная переменная в памяти.

Цитата:

Сообщение от S.Kuskov

И точно ли не понадобятся такие закрытые строки в InventSum в случае открытия (отмены закрытия) склада?

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

[Logger]

Я бы еще подправил метод
\Classes\InventSumDateEngine\selectInventOnHand
вот это место

    insert_recordset inventSumDateTrans (ItemId, PostedQty, ReceivedQty, DeductedQty, PickedQty, RegisteredQty, PostedValue,
                                         #InventDimFields, ParmId, TransType)
        select ItemId, sum(PostedQty), sum(Received), sum(Deducted), sum(Picked), sum(Registered), sum(PostedValue)
        from inventSum
        group by ItemId
        where inventSum.Closed == NoYes::Yes
        join #InventDimFields, parmId /*scalar*/, inventSumDateType /*scalar*/
            from inventDim
            group by #InventDimFields
            where inventDim.InventDimId == inventSum.InventDimId
        exists join inventTrans
            where inventTrans.ItemId      == inventSum.ItemId
               && inventTrans.InventDimId == inventSum.InventDimId
               && inventTrans.DateStatus  >= perDate;

здесь код явно закладывается на наличие закрытых записей. Легко исправляется.

Насколько я помню используется только в одном отчете по остаткам.