Приколы курсовых разниц по контрагентам - и полное недоверие к ним после этого

[BOAL]

Открыл для себя, что в АХ4 СП2 ХФ2
В курсовых раницах
\Расчеты с клиентами\Периодические операции\Курсовая разница
\Расчеты с поставщиками\Периодические операции\Курсовая разница

не работают фильтры по Просмотрено
вроде мелочь, открываем форму и что видим?

\Forms\CustExchRateAdjustment\Data Sources\CustExchRateAdjustment\Methods\executeQuery
и
\Forms\VendExchRateAdjustment\Data Sources\VendExchRateAdjustment\Methods\executeQuery

PHP код:

void executeQuery()
{
    switch (allNotReviewedReviewed.valueStr())
    {
        case 'All' :
            criteriaReviewed.value('');
            break;

        case 'Not reviewed' :
            criteriaReviewed.value('0');
            break;

        case 'Reviewed' :
            criteriaReviewed.value('1');
            break;
    }
    super();
} 


Все это в SYS слое.

Это код работает только в английском языке!
Кто это делал? Стажер? Что еще и как он сделал в алгоритмах? Пугает!
Сам расчет тестили и вроде пашет...
Но "Ложки нашлись, а осадок остался"(С)

должно быть

PHP код:

public void executeQuery()
{
    switch (allNotReviewedReviewed.selection())
    {
        case AllNotReviewedReviewed::All :
            criteriaReviewed.value('');
            break;

        case AllNotReviewedReviewed::NotReviewed :
            criteriaReviewed.value(queryvalue(NoYes::No));
            break;

        case AllNotReviewedReviewed::Reviewed :
            criteriaReviewed.value(queryvalue(NoYes::Yes));
            break;
    }
    super();
} 


Что забавно, бестрактис этот родной код пройдет Из-за применения '' вместо ""

[ZVV]

Да, занятненько... И в 2009 тоже самое, кстати...
Спасибо.

[ZVV]

Кстати, если уж до конца бороться за чистоту кода, то надо бы это

Цитата:

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

            criteriaReviewed.value('');

заменить на

            criteriaReviewed.value(SysQuery::valueUnlimited());

[oip]

ZVV, если уж совсем красиво, то мне лично режет глаза отсутствие чего-то типа

...
default : 
    throw error(strFmt("@SYS19306",funcName()));

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

А так, весело конечно. Подходящий экспонат для старой забытой темы.
The daily Axapta WTF