The daily Axapta WTF

[EVGL]

Модуль управления цехом в русском исполнении изобилует изумительными перлами:


N00b registration

[S.Kuskov]

Это наверное больше для вот этой темы подходит Метка @xxx##### переведена на русский некорректно...

[Evgeniy2020]

Курьез.
Есть Axapta 3. Есть системная таблица AccessRightsLists.
До этого кто то добавил несколько (10-12) запретов (NoAccess) для группы прав 'Admin' на несколько SecurityKey с id = 1, 23, и т.д.

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

После чего решено было удалить эти несколько 10-12 записей для группы прав Admin из этой таблицы. Выделив эти записи в Обозревателе таблицы AccessRightsList и нажав Alt+F9, записи благополучно удалились.
при этом слетели все права для группы прав Admin.

При попытке зайти пользователем с правами 'Admin' на экране не отображалось главное меню и доступ ко всему был закрыт, включая АОТ. Вот такая оказалась опасная операция удаления записей из этой таблицы для группы прав.

Благодаря наличичую второй группы прав эквивалентных админским удалось восстановить полный доступ для группы Admin.

[DTD]

\Classes\CustAutoCreate\setCustTable

protected void setCustTable()
{
    NumberSeq       num;
    ;

    custTable.clear();
    custTable.initValue();
    custTable.data(CustTable::find(templateCustAccount));

    if (custAccount)
    {
        custTable.AccountNum = custAccount;
    }
    else
    {
        custTable.AccountNum = NumberSeq::newGetNum(CustParameters::numRefCustAccount()).num();
    }

    if (CustTable::exist(custTable.AccountNum))
    {
        if (num)
        {
            num.abort();
            checkFailed("@SYS59641");
        }
        checkFailed("@SYS59639", custTable.AccountNum);
        throw error("@SYS23020");
    }

    if (num)
    {
        num.used();
    }
}

[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 - виден взвешенный, системный подход

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

[Vadik]

Цитата:

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

в чем противоречие?

В том-то и прелесть, что нет противоречий - пишем в паре мест в в текст проводки или название журнал метку при расщеплении, на ту же метку проверяем в паре других - не расщепление ли. Это я и называю "системный, взвешенный подход". Варианты, когда примечания в журнал могут еще и руками вводиться или пишется одним пользователем, а читается другим с другим языком интерфейса и цепочка ломается, рассматривать смысла нет - вероятность-то ничтожная

[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 с нормальным комментарием. Если понимаешь что работать без этого куска нормально не будет, но по каким либо причинам надо привести в компилируемый вид (в жизни всякое бывает, иногда реально надо), то надо прервать исполнение если дойдет до недоделанного места.

[ice]

Цитата:

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

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

ага, и теперь аксапта просто наводнена этими debug::assert(). надеемся что дойдут руки и доделают

[belugin]

Цитата:

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

ага, и теперь аксапта просто наводнена этими debug::assert(). надеемся что дойдут руки и доделают

Если, вдруг, кто-нибудь на самом деле думает что debug::assert признак чего-то недоделанного, прочитайте, пожалуйста этот пост

[Zabr]

Цитата:

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

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

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