The daily Axapta WTF

[kashperuk]

Цитата:

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

А сколько ещё изменений было в этом ревью ? Там же есть зависимость между длинной изменения и качеством его ревью

Еще где-то 10 других изменений. Я их побоялся смотреть

[ax_mct]

Цитата:

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

Если бы дело было только в этом.
Это изменение отревьюило 2 человека.. То есть втроем, с учетом того что сам девелопер тоже правил, они не смогли нормально скобочки сосчитать, и их никак не удивила эта странная констркция со сравнением двух энумов..

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

AmountMST cost = this.itemPrice(ModuleInventPurchSales::Invent);
    AmountMST sales = this.itemPrice(ModuleInventPurchSales::Sales);
    ...

    //Avoid division by zero
    if (sales == 0)
    {
        mcrCatalogAreaAnalysisTmp.CogsPercent = 0;
    }
    else
    {
        mcrCatalogAreaAnalysisTmp.CogsPercent = cost / sales;
    }

[macklakov]

Цитата:

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

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

Да не, он пытался проверить тоже самое значение которое попадет в знаменатель. Но зачем-то решил его вычислить повторно и при этом перепутал местоположение скобки. Код должен был выглядеть вот так:
if (this.itemPrice(ModuleInventPurchSales::Sales) == 0)
Такое случается по запаре. Ничего особенного.
Но вот фикс шедеврален! Это же человек не просто колотил код на скорость. Это вдумчиво выловили баг, подумали над его природой и исправили. Кажется я начинаю понимать почему местный саппорт так неохотно репортит даже доказанные баги с указанием где что и как поломано в продуктовую команду.

[dech]

Есть такая полезная программка PVS-Studio.
Один из постов про нее на хабре очень в тему.
https://habrahabr.ru/company/pvs-studio/blog/329090/
Не мешало бы аналог для АХ забабахать

[dech]

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

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

[ax_mct]

Цитата:

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

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

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

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

[VORP]

Ну у человека была задача пофиксить ошибку компиляции, которой раньше не было
Может быть он и видел эти скобочки, но исправление скобочек приведёт что что-то будет по другому работать, а следовательно могут упасть автоматы, а потом найдут того, кто поменял и спросят - зачем поменял, а автоматы не поменял, а менять автоматы надо полдня прогонять их и разбираться почему они падают(не факт что из-за изменения, но это надо ещё доказать), и вообще тебе надо было сделать чтоб компилировалось просто

[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 на "Основные средства" и снова интерпретируется системой, как запрос на движения, относящиеся к международному, а не российскому, модулю ОС.

[Pustik]

Цитата:

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

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

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

[Logger]

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

[mnt_dx]

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

[AlGol]

Цитата:

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

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

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

[potential]

Цитата:

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

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

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

[macklakov]

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

[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