Ax2009 SP1 RU6. LedgerBalanceSum_CurrentMST.buildQuery(). Ошибка при пустой начальной дате

[Damn]

Не знаю в той ли я теме пишу.
Вопрос у меня про метод LedgerBalanceSum_CurrentMST.buildQuery()
В Ax2009 появился в этом методе в 57-й строке следующий код

        // <GEEU>
        if (fromDate || toDate)
        {
        // </GEEU>
            qr.value(SysQuery::range(fromDate,toDate));
        // <GEEU>
        }
        // </GEEU>

В результате стал некорректно работать "Анализ счёта", если в нём не заполнить начальную дату. При пустой начальной дате система считает начальное сальдо по счёту и делает это за весь период. В Ax 3.0 не было показанных выше модификаций GEEU и такой ошибки в анализе счёта не возникало.
Непонятно только что исправлять - LedgerBalanceSum_CurrentMST.buildQuery() или RLedgerSheetEngine_AccountReview.initAccounts() ?
Нужно ли где-то в системе чтобы сальдо считалось при пустых fromDate и toDate ?

[mazzy]

Цитата:

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

Нужно ли где-то в системе чтобы сальдо считалось при пустых fromDate и toDate ?

Да, нужно.
Я считаю это багой локализации.
По моему мнению надо исправлять LedgerBalanceSum_CurrentMST.buildQuery()

[AlexSD]

Цитата:

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

Да, нужно.
Я считаю это багой локализации.
По моему мнению надо исправлять LedgerBalanceSum_CurrentMST.buildQuery()

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

[mazzy]

Цитата:

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

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

при пустой начальной дате надо считать от начала времен
при пустой конечной дате надо считать до конца времен

при пустой начальной дате в отчете "Анализ счета" (и во всех других отчетах) начальное сальдо должно быть = 0.

[AlexSD]

Цитата:

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

при пустой начальной дате надо считать от начала времен
при пустой конечной дате надо считать до конца времен

при пустой начальной дате в отчете "Анализ счета" (и во всех других отчетах) начальное сальдо должно быть = 0.

Если приглядеться пристальнее, наш фикс лишь восстанавливает логику sys-овского функционала нарушенного при модификации метода SysQuery::range().

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

[mazzy]

как скажете.

я только повторю изначальную проблему:

Цитата:

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

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

[AlexSD]

Да. Таким образом поменялась логика sys-ких запросов. Еще с 4-ки. Если даты в запросе не заданы, то берется весь период.

[George Nordic]

Что "так было задумано" и "именно так работает на SYS слое" еще не означает, что "так ПРАВИЛЬНО".

Вот интересно. Ошибки-то в общем-то, и нет. Но работает - не так, как ожидает пользователь и, на мой взгляд, некорректно.

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

С Уважением,
Георгий

[AlexSD]

Могу возразить, что это так же не значит, что "так НЕПРАВИЛЬНО". Этот фикс был сделан для некой баги. Кому-то значит показалось, что без этого фикса работает неправильно.

Кому верить?

[George Nordic]

Вот тут-то и проблема.

1. Нет информации, что за бага и как она "стрельнула". Но охотно верю, что такое могло быть.

2. Согласен. Не "правильно", а, скорее, "ожидаемо".

перекрывать SYS - не дело, т.к. непонятно, что еще может произойти и надо тестировать.
вводить галочку - тоже надо сильно прорабатывать, согласовывать, думать, на какие даты можно ставить дэйтнуль, а на какие- не стоит... по всей системе делать или только здесь.

Ждем предложений.

Георгий

[mazzy]

Цитата:

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

Кому верить?

Покупателям-Пользователям.

[AlexSD]

Цитата:

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

Покупателям-Пользователям.

Этот фикс тоже был сделан для покупателей-пользователей... Только они не поленились завести багу.