The daily Axapta WTF

[VORP]

Да нет, не копи паст это. Просто человек пишет компилятор, фреймворк для екстеншенов для енумов. Он не знает и/или скорее всего не хочет знать ничего про цены и прочие MCR - этим кодом владеет команда MCR(условно говоря). Сейчас он поменял что-то в компиляторе и у него в некоторых местах ошибки компилятора. Править чужой код он не может, но надо сделать чтоб компилилось. Вот он и заменил цифру на енум, хоть оно и бестолково. По хорошему должен был кинуть багу на MCR что код походу неправильный. Но опять же, не факт что команда MCR будет править код, ведь клиенты баги не репортят, а там мало ли что кому кажется.

[ax_mct]

Цитата:

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

Или через тернарный оператор:

mcrCatalogAreaAnalysisTmp.CogsPercent = sales ? cost / sales : 0;

Это уже не gentle. Фиксить лучше в том же стиле, а если смещения строк нет - вообще идеально.
Заменить переменную - это одно, тернарным это уже переписывать код, так можно заменить один глупый баг другим, что часто и происходит при переписке кода.

[ax_mct]

Цитата:

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

Да нет, не копи паст это. Просто человек пишет компилятор, фреймворк для екстеншенов для енумов. Он не знает и/или скорее всего не хочет знать ничего про цены и прочие MCR - этим кодом владеет команда MCR(условно говоря). Сейчас он поменял что-то в компиляторе и у него в некоторых местах ошибки компилятора. Править чужой код он не может, но надо сделать чтоб компилилось. Вот он и заменил цифру на енум, хоть оно и бестолково. По хорошему должен был кинуть багу на MCR что код походу неправильный. Но опять же, не факт что команда MCR будет править код, ведь клиенты баги не репортят, а там мало ли что кому кажется.

Скорее всего так и есть. Была ошибка компилятора - её и пофиксили.
Но мне знаете ли страшно жить с такими системными программистами.
Там ведь не вопрос бизнес-логики, а деление на ноль. Поверяем одно значение, делим на другое.
Проверяем на ноль второй результат вызова функции, а делим с первым результатом вызова этой же функции. Для нормального программиста С/С++ мимо такого пройти просто невозможно. Это просто режет глаз.

[dech]

Цитата:

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

Да нет, не копи паст это. Просто человек пишет компилятор, фреймворк для екстеншенов для енумов. Он не знает и/или скорее всего не хочет знать ничего про цены и прочие MCR - этим кодом владеет команда MCR(условно говоря). Сейчас он поменял что-то в компиляторе и у него в некоторых местах ошибки компилятора. Править чужой код он не может, но надо сделать чтоб компилилось. Вот он и заменил цифру на енум, хоть оно и бестолково. По хорошему должен был кинуть багу на MCR что код походу неправильный. Но опять же, не факт что команда MCR будет править код, ведь клиенты баги не репортят, а там мало ли что кому кажется.

Да... Чем крупнее компания, тем сложнее поддерживать стабильный код. Здесь ведь и до этого компилилось, это же рантайм еррор. Думаю, где-то просто тесты не сработали. И потом видимо выдали на доработку уже не тому, кто этим кодом занимался.
В большой компании должно быть разграничение кода между специалистами, чтобы отвечать за свой участок и знать его как свои 4 пальца. :-)

[macklakov]

Цитата:

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

Править чужой код он не может, но надо сделать чтоб компилилось. Вот он и заменил цифру на енум, хоть оно и бестолково. По хорошему должен был кинуть багу на MCR что код походу неправильный. Но опять же, не факт что команда MCR будет править код, ведь клиенты баги не репортят, а там мало ли что кому кажется.

В принципе, выглядит очень правдоподобно. Только похоже что писали они не компилятор как таковой, а best practice check. Ведь единственный результат изменения в том, что теперь в коде нет константы "0". Можно даже предположить что это робот сделал.

[trud]

Цитата:

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

Да, с инфологом действительно все очень печально

Зарегистрировал "идею", кому не лень проголосуйте
https://ideas.dynamics.com/ideas/dyn...ions/ID0002243
также чтобы добавили поддержку файлов в RunBase
https://ideas.dynamics.com/ideas/dyn...ions/ID0002242

Цитата:

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

Но опять же, не факт что команда MCR будет править код, ведь клиенты баги не репортят, а там мало ли что кому кажется.

Кстати удивительно точно подмечено, собственно на мой взгляд все текущие проблемы AX7 вызваны подобным подходом - изначально здравые идеи верхнего уровня(практически все "новинки" AX7 очень хороши) разбиваются о совершенно "формальную" реализацию с пропуском многих мелких деталей, что делает их во многом трудно используемым или вообще непригодным.
Интересно есть ли какие-нибудь методологии позволяющие как-то бороться с таким(типа если рабочий на конвейере увидел брак, то он должен остановить конвейер, а не просто забить, так как это не его дело).
Ведь не работают же так Apple и Google

[kashperuk]

Цитата:

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

Ведь не работают же так Apple и Google

Еще как

[kashperuk]

Еще нашел такое сегодня

WHSWorkExecuteDisplayInquiryItem.buildItemInquiry():

...
    if (availQty == 0 && pickedQty == 0 && physInventQty == 0)
    {
        // clear all totals
        availQty        = 0;
        pickedQty       = 0;
        physInventQty   = 0;
    }
...

Чтоб наверняка

[Товарищ ♂uatr]

[Player1]

Извечная проблема локализаторов - кол-во символов в переведенном слове больше чем в переводимом. В компьютерных играх кто по-креативнее пытались переводить такие слова как-то шуточно, а не тупо урезая слово. В данном случае видимо для 4-х буквенного "EXIT" подошло бы "УЙТИ"

[oip]

Устанавливаем SSRS. Вроде все успешно завершается. Лезем в AX - нет ничего. Смотрим в лог установки - и оба-на, оказывается ошибка. WTF? С ошибкой-то разобрались, но писать "Setup was completed successfully", когда на самом деле "Reporting Services Extension was not configured" как-то странно.

Цитата:

2017-07-20 20:16:42Z Error executing command: USE [*******];
2017-07-20 20:16:42Z DECLARE @tableName NVARCHAR(64);
2017-07-20 20:16:42Z DECLARE @tableId int;
2017-07-20 20:16:42Z DECLARE @utilElementTypeTable AS INT;
2017-07-20 20:16:42Z SELECT @tableName = N'SRSServers';
2017-07-20 20:16:42Z SELECT @utilElementTypeTable = 44;
2017-07-20 20:16:42Z SELECT @tableId = [AxId] FROM [ModelElement] WHERE [ElementType] = @utilElementTypeTable AND [Name] = @tableName;
2017-07-20 20:16:42Z SELECT @tableId;
2017-07-20 20:16:42Z Invalid object name 'ModelElement'.
2017-07-20 20:16:42Z Error executing command: use [*******]; exec [sp_AddReportServer] @serverId = N'*******', @serverUrl = N'*******', @isDefault = 1, @axReportFolder = N'DynamicsAX', @description = N'', @reportManagerUrl = N'*******', @sqlServerInstanceName = N'MSSQLSERVER', @aosServerName = N'*******', @configurationId = N'*******', @isSharepointIntegrated = 0, @recId = 5637268379
2017-07-20 20:16:42Z Cannot insert the value NULL into column 'AOSID', table '*******.dbo.SRSSERVERS'; column does not allow nulls. INSERT fails.
2017-07-20 20:16:42Z Changed database context to '*******'.
2017-07-20 20:16:42Z The statement has been terminated.
2017-07-20 20:16:42Z ******************************************************************************************************

2017-07-20 20:16:42Z Reporting Services Extension was not configured. An unknown issue occurred. Please contact your administrator.

2017-07-20 20:16:42Z ******************************************************************************************************

[dech]

Тонкости программирования с MapIterator

    if(!mi.more())
        while(mi.more()) // документы создавались, но откатились
        {
            ...

[TasmanianDevil]

Есть мнение, что было бы крайне не лишним у Enum'ов контролировать не только уникальность кодовых обозначений и числовых значений элементов, но и меток и их содержания.
Для примера, enum InventTransType, элементы Asset и FixedAssets_RU привязаны к разным меткам, @SYS12578 и @SYS19633, имеющих идентичное содержание, а именно "Основные средства".

Пробуем фильтровать складские проводки по типу движения "Основные средства", надеясь увидеть расходы по вводу ОС в российском модуле ОС - получаем фиг.
Ставим вместо "Основные средства" в значение ограничения по типу движения величину 100 (числовое значение элемента FixedAssets_RU) - и, о чудо, запрос на форме вернул данные.
Двигаемся дальше, добиваем запрос прочими ограничениями - получаем фиг . Что же случилось ? При вызове диалога редактирования запроса значение ограничения по полю типа движения "магическим" образом поменялось со 100 на "Основные средства" и снова интерпретируется системой, как запрос на движения, относящиеся к международному, а не российскому, модулю ОС.

[Logger]

А еще запрещать пустые метки.
А то при перепаковке запросов значение фильтра может теряться.

[Pustik]

Цитата:

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

Есть мнение, что было бы крайне не лишним у Enum'ов контролировать не только уникальность кодовых обозначений и числовых значений элементов, но и меток и их содержания.

Аналогичные проблемы у кнопок типа CommandButton, когда пытаешься в свойстве кнопки Command установить какое-нибудь значение, например "Отмена".

[mnt_dx]

Я немного недоумеваю от интерфейса АХ7 местами.
В форме настройки номенклатурных групп, например, предлагается несколько горизонтальных скроллов. Помоги себе сам:

[AlGol]

Цитата:

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

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

Это похоже глобальная проблема веб интерфейсов от Микрософта: такая же бяка в веб-интерфейсе Проджекта.

[macklakov]

Очень веселит манера MS, на определенной стадии обсуждения бага, отвечать "by design". Просьбы показать этот самый дизайн или внести в него исправления воспринимаются как возмутительное нарушение всяких норм приличия, граничащее с клиническим идиотизмом.

[potential]

Цитата:

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

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

я жутко извиняюсь, но Вы наверное имели в виду "вертикальных"

[kashperuk]

Цитата:

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

Очень веселит манера MS, на определенной стадии обсуждения бага, отвечать "by design". Просьбы показать этот самый дизайн или внести в него исправления воспринимаются как возмутительное нарушение всяких норм приличия, граничащее с клиническим идиотизмом.

Надо просто говорить.
The way it works now does not make sense for this and this industry.
Please change this bug to a DCR. It needs to be fixed