The daily Axapta WTF

[macklakov]

\Data Dictionary\Tables\DimensionAttribute\Methods\getMainAccountDimensionAttribute

                dimAttribute.Name = "@SYS343426";  // DO NOT LOCALIZE! - This is the signular label applied to the view
...
                dimAttribute.HashKey = str2Guid('{AA1602A7-1B0E-4A14-864C-B5E30E61FC9A}');    // Use well-known guid

IMHO прекрасно.

[Vadik]

Вот какая прелесть с AX 2012 CU6 приплыла. Давайте-ка я с двух попыток угадаю в какую страну какому партнеру моя любимая sustained engineering team это зааутсорсила ?

[kashperuk]

Цитата:

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

Вот какая прелесть с AX 2012 CU6 приплыла. Давайте-ка я с двух попыток угадаю в какую страну какому партнеру моя любимая sustained engineering team это зааутсорсила ?

А вот и неправильно
Делали это, похоже, в Фарго

Судя по соответствующему багу - для Сингапура

[Vadik]

Цитата:

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

А вот и неправильно

Ну, формально я своих двух вариантов пока не озвучивал Как, неужели в САМОМ Фарго ?
P.S. Вообще, как знать - если посмотреть по перекрестным ссылкам на метке @SYS67430 - виден взвешенный, системный подход (уходит, сокрушенно качая головой)

[gl00mie]

Цитата:

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

если посмотреть по перекрестным ссылкам на метке @SYS67430 - виден взвешенный, системный подход

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

[twilight]

Если создать query, а потом view с таким же именем, а потом случайно перетащить query в узел датасорсов view (вместо узла Metadata) и попробовать выбрать там какое либо поле, то получится view, которыей ссылается сам на себя. AOS падает и не позволяет потом открывать view и таблицы. Лечиться только восстановлением приложения из бэкапа или ручным редактированием model store.
Аналогичная ошибка была и в 2009 версии...

[TasmanianDevil]

2009, 5.0.1500.6491
Любимая всеми функция удаления строк журналов, класс LedgerJournalDeleteTransaction, девственный слой sys.
кусок main()

ledgerJournalDeleteTransaction.getLast();
ledgerJournalDeleteTransaction.updateQueryBuild(args.record());

сам updateQueryBuild()

void  updateQueryBuild(LedgerJournalTrans ledgerJournalTrans)
{
    ;
    queryRun.query().dataSourceTable(tablenum(LedgerJournalTrans)).findRange(fieldnum(LedgerJournalTrans,JournalNum)).value(strfmt('%1',ledgerJournalTrans.JournalNum));
    queryRun.query().dataSourceTable(tablenum(LedgerJournalTrans)).addRange(fieldnum(LedgerJournalTrans,PaymentStatus)).value(SysQuery::range(CustVendPaymStatus::Rejected, CustVendPaymStatus::Deleted) + ',' + queryValue(CustVendPaymStatus::None));
}

Range по PaymentStatus занимается тупым размножением самого себя при каждом вызове и в один прекрасный момент функция тупо перестает работать ...

[Cardagant]

Перенесено из темы Классные программисты MS Бразилии создали сей шедевр

Нашёл интересный пример кода.
DAX 2009 (в AX 3.0 то же самое)

Класс ProdRouteType, метод update.

Привожу код метода:

void update()
{
    ProdTable        prodTable;
    ProdRoute        prodRoute_Orig= prodRoute.orig();
    ;
    ttsbegin;

    this.setBackStatus();

    if (prodRoute.OprPriority)
        prodRoute.LinkType = SchedJobLinkType::None;

    prodRoute.doUpdate();

    // Shop Floor Control begin
    if (Global::isConfigurationkeyEnabled(configurationkeynum(Jmgjob)))
    {
        JmgJobTable::synchronizeProdRoute(prodRoute);
    }
    // Shop Floor Control end

    if (0)
    {
        if  (prodRoute.BackorderStatus != prodRoute_Orig.BackorderStatus)
        {
            prodTable= prodRoute.prodTable(true);
            prodTable.status().setBackStatus();
            prodTable.doUpdate();
        }
    }

    ttscommit;
}

Не совсем понимаю условие

if (0)

По сути оно подразумевает, что код не будет выполняться.
Что вы думаете на этот счёт? Зачем так могли сделать?

[db]

Цитата:

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

Перенесено из темы Классные программисты MS Бразилии создали сей шедевр


По сути оно подразумевает, что код не будет выполняться.
Что вы думаете на этот счёт? Зачем так могли сделать?

1. Существует такой "стиль" выключения блоков кода. Ну типа закомментировать. Про уместность лучше промолчать, но возможно в предыдущей жизни автор его где то использовал
2. Автор просто сделал код компилируемым в надежде дописать условие чуть позже и забыл

[user_ax]

Цитата:

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

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

А не легче написать

//TODO

и не забудешь точно

[Cardagant]

Цитата:

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

А не легче написать

//TODO

и не забудешь точно

Скорее всего история данного примера начинается из очень ранних версий, где задач, возможно, ещё не было.

[db]

Цитата:

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

А не легче написать

//TODO

и не забудешь точно

вот throw error() или debug::assert() дают некую надежду что тебе рано или поздно напомнят в принудительном порядке, а TODO, увы, может жить вечно Если кусок действительно не важен для правильного исполнения, то можно и TODO с нормальным комментарием. Если понимаешь что работать без этого куска нормально не будет, но по каким либо причинам надо привести в компилируемый вид (в жизни всякое бывает, иногда реально надо), то надо прервать исполнение если дойдет до недоделанного места.

[Zabr]

Цитата:

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

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

Некомпилирующийся код не станет компилироваться от того, что его поставят под условие "if (0)". Вероятно, Вы имели в виду не "компилируемым", а "выполняющим правильные действия".

[S.Kuskov]

Цитата:

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

Некомпилирующийся код не станет компилироваться от того, что его поставят под условие "if (0)".

Именно так. Но теоретически можно представить представить себе ситуацию когда не содержащий ошибок компиляции код таким образом скрывается от выполнения, но не скрывается от компиляции. Например, для того что бы работали перекрестные ссылки, или просто чтобы бы при очередной компиляции проверять не появились ли ошибки в этом коде, связанные с изменениями в других объектах.

[Stitch_MS]

Коллега нашел: AX 2012 CU5, \Classes\RunBaseBatch\canGoBatch

/// <summary>
///    Returns false
/// </summary>
/// <returns>
///    false
/// </returns>
/// <remarks>
///    This method must be in this class because it is called from the dialogRunbase class.
/// </remarks>
/*MAN
The method must return false when the class is not batchable.
*/
public boolean canGoBatch()
{
    return true;
}

[user_ax]

Цитата:

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

Коллега нашел: AX 2012 CU5, \Classes\RunBaseBatch\canGoBatch

Спасибо, кэп

[lvan]

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

[EVGL]

В последнее время много приходится работать с шаблонами Sure Step. Русская версия с руководством для руководителя проекта вызывает восторг, просто по полу катался:

Цитата:

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

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

[belugin]

А мне стиль напомнил http://m.youtube.com/watch?v=e-qDHM0...%3De-qDHM0slB0

[kashperuk]

 .....
        catch (Exception::DuplicateKeyException)
        {
            // Prevent an endless loop
            if (tryCount < 5)
            {
                retry;
            }
            else
            {
                throw Exception::DuplicateKeyExceptionNotRecovered;
            }
        }

Интересно, на что они надеются, вызывая здесь Retry?