D365FO: когда обновление ломает доработку

[Stitch_MS]

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

В другой ветке я уже приводил пример с лукапом, когда мы создаем обработчик события onLookup на контроле, а потом Майкрософт решает перекрыть метод lookup на том же котроле, и наш обрабочик становится мертвым кодом.

Только что напоролся на нечто другое:
Если к таблице А добавить поле и релейшн на таблицу Б, причем в таблице А уже есть некоторое поле и релейшен, указывающие на ту же таблицу Б в другом контексте, то вызов QueryBuildDataSource.relations(true) где угодно может привести к неожиданным результатам.

[raz]

class LedgerJournalCheckPost
мы использовали CoC на LedgerJournalCheckPost.postTrans(), последнее обновление:

    [SysObsoleteAttribute('Method postTrans has been deprecated. Please use postTransV2 method instead.', false)]
    protected boolean postTrans(

теперь есть новый метод LedgerJournalCheckPost.postTransV2().

Вот вам бесшовный и неразрушающий.

[sukhanchik]

Та же самая ситуация с примером, который использует Entiti.
Была Entiti под названием "X". Ее использовали в коде для целей загрузки некоего справочника (потому что исходный формат загрузки сильно отличался от предполагаемого, который был в стандарте). После обновления была выпущена Entiti под название "XV2", а предыдущая была объявлена устаревшей. Ну и собственно все. Код формально компилируется, но фактически - не работает.

В конкретно моем случае это была замена с EcoResProductEntity на EcoResProductV2Entity и замена класса EcoResProductEntityToCrossTableDataAdaptor на EcoResProductV2EntityToCrossTableDataAdaptor. Соответственно - сломался импорт номенклатур при переходе с 7.3 на 8.0. Учитывая, что этих версий Entiti в частности при загрузке клиентов / поставщиков уже минимум 3 штуки я видел - то подозреваю, что их использование в коде - могло бы также привести к необходимости корректировки этого кода.

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

[belugin]

https://docs.microsoft.com/en-us/dyn...considerations

Цитата:

Some methods will be attributed as obsolete to signal that they will be fully deprecated in the future. Any compiler warning that is generated due to the calling or wrapping of an obsolete method should be investigated to ensure that the expected code path still exists. In some cases, Microsoft code will directly call the new method in place of the obsolete method. When this happens, the code built around the obsolete method will not execute when expected.

[raz]

Цитата:

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

https://docs.microsoft.com/en-us/dyn...considerations

Дело в том, что в предыдущей версии нет и намека на предстоящие изменения.

    /// true if the journal was posted successfully; otherwise, false.
    /// </returns>
    protected boolean postTrans(
        LedgerVoucher               _ledgerVoucher,

А если клиент купил доработку, внедрил и должен мочь "забыть" о ней, но фиг вам. С иглы не слезть. Никогда нельзя доверять MS. Они всегда ломали и будут ломать. Маркетинг не совадает с реальностью. Клиент будет платить все время пока использует продукты MS, чего MS и хочет добиться обманывая всех своими сказками.

[dech]

Стоит взять пример с разработчиков DirectX, которые знали, что если кто-то выпускал игры для 6-й или 7 версии, то последующие версии не должны сломать функционал старых игр. Итог превзошел все ожидания. Интерфейсы менялись крайне редко, а если менялись, то по сути создавались новые, более совершенные или более полезные, обрастая новыми методами, либо измененными старыми. Старый же код продолжал жить, радуя поклонников ретро-игр.

[vmoskalenko]

Цитата:

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

Стоит взять пример с разработчиков DirectX, которые знали, что если кто-то выпускал игры для 6-й или 7 версии, то последующие версии не должны сломать функционал старых игр. Итог превзошел все ожидания. Интерфейсы менялись крайне редко, а если менялись, то по сути создавались новые, более совершенные или более полезные, обрастая новыми методами, либо измененными старыми. Старый же код продолжал жить, радуя поклонников ретро-игр.

Вот по этому же пути и пошли Data Entity. Ну те которые XV2, XV3, и т.д.
Как бы и старый код не должен ломаться и новые фичи будут доступны для новых разработок.
Но что-то пошло не так...
Решили использовать старые имена для public end points (OData)...

[Stitch_MS]

Еще пример. Класс SubledgerJournalizerProjectExtension,

    protected ProjEmplTrans createProjHourTrans(Container _projectActualHeaderContainer, SourceDocumentLineItem _sourceDocumentLineItem, ...)
    {
        ProjEmplTrans   projEmplTrans;
        PSAComponentGroupAssignment psaComponentGroupAssignment;

        projEmplTrans.clear();
        
        #declareProjectActualHeaderContainerVariablesMacro
        #projectActualHeaderContainerMacro = _projectActualHeaderContainer;

        projEmplTrans.ProjId = actualProjectId;
        projEmplTrans.CategoryId = actualCategoryId;
        projEmplTrans.ActivityNumber = actualActivityNumber;
        projEmplTrans.LinePropertyId = actualLinePropertyId;

Два макроса в коде выглядят так:

    #localmacro.declareProjectActualHeaderContainerVariablesMacro
        RefRecId            actualSourceDocumentLine;
        DataAreaId          actualProjectDataAreaId;
        ProjId              actualProjectId;
        ProjCategoryId      actualCategoryId;
        smmActivityNumber   actualActivityNumber;
        ProjLinePropertyId  actualLinePropertyId;
        CurrencyCode        actualTransactionCostCurrency;
        CurrencyCode        actualTransactionSalesCurrency;
        CurrencyCode        accountingCurrency;
        ProjTaxGroup        actualTaxGroupId;
        ProjTaxItemGroup    actualTaxItemGroupId;
        DimensionDefault    actualDefaultDimension;
        Qty                 actualQuantity;
        AmountCur           actualTransactionCurrencyCostAmount;
        AmountMST           actualAccountingCurrencyCostAmount;
        AmountCur           actualTransactionCurrencySalesAmount;
        AmountMST           actualAccountingCurrencySalesAmount;
    #endmacro

    #localmacro.projectActualHeaderContainerMacro
        [
        actualSourceDocumentLine,
        actualProjectDataAreaId,
        actualProjectId,
        actualCategoryId,
        actualActivityNumber,
        actualLinePropertyId,
        actualTransactionCostCurrency,
        actualTransactionSalesCurrency,
        accountingCurrency,
        actualTaxGroupId,
        actualTaxItemGroupId,
        actualDefaultDimension,
        actualQuantity,
        actualTransactionCurrencyCostAmount,
        actualAccountingCurrencyCostAmount,
        actualTransactionCurrencySalesAmount,
        actualAccountingCurrencySalesAmount
        ]
    #endmacro

Т.е. в методе createProjHourTrans первый параметр _projectActualHeaderContainer -- это, по сути, 17 параметров. Если изменить макросы, то список параметров в createProjHourTrans вроде как не изменился, возможно не будет считаться за breaking change, и может по-тихому сломать расширение, изменяющее значение одного из элементов в контейнере.

И в этом классе такой метод не один.

[ax_mct]

Цитата:

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

Еще пример. Класс SubledgerJournalizerProjectExtension,

Т.е. в методе createProjHourTrans первый параметр _projectActualHeaderContainer -- это, по сути, 17 параметров. Если изменить макросы, то список параметров в createProjHourTrans вроде как не изменился, возможно не будет считаться за breaking change, и может по-тихому сломать расширение, изменяющее значение одного из элементов в контейнере.

И в этом классе такой метод не один.

Интересно кто все эти программисты. Где все эти архитекторы и ревью кода.
В любом языке и на любой платформе так не пишут. Это как кучку в лифте наложить.
Максимум два часа времени создать сериализующийся класс для передачи эти параметров если очень надо упаковкой между клиентом и сервером.
А если все на сервере то вообще дебилы.

[trud]

Цитата:

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

Интересно кто все эти программисты. Где все эти архитекторы и ревью кода.

Я тут узнал, что в индийской культуре не принятно возражать старшим. так что если код писался "уважаемыми" людьми, то ревью тут не поможет

[EVGL]

Цитата:

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

Я тут узнал, что в индийской культуре не принятно возражать старшим. так что если код писался "уважаемыми" людьми, то ревью тут не поможет

Это австриец Michael Gall писал. Или по крайней мере концепцию создавал. Очень приятный и талантливый парень, кстати. Ну и вполне заслуженно блестящую карьеру сделал.

[ax_mct]

Цитата:

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

Это австриец Michael Gall писал. Или по крайней мере концепцию создавал. Очень приятный и талантливый парень, кстати. Ну и вполне заслуженно блестящую карьеру сделал.

Ну с PHD в Computer science не поспоришь.
https://www.linkedin.com/in/migall/

Но передавать динамическую кучку таким образом как-то не по программистски совсем.
Даже если без расширений, а просто в одной команде это уже проблемное место.
Бывают компромиссы но здесь просто недостаток опыта в программировании вообще.

[fed]

Цитата:

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

Ну с PHD в Computer science не поспоришь.
https://www.linkedin.com/in/migall/

Но передавать динамическую кучку таким образом как-то не по программистски совсем.
Даже если без расширений, а просто в одной команде это уже проблемное место.
Бывают компромиссы но здесь просто недостаток опыта в программировании вообще.

<SARCASM> Как я уже много раз говорил - Allocations and Source Document Infrastructure - это такая специальная функциональность, в которой криво все начиная с исходных требований и кончая мелкими нюансами кодинга. Вероятно, авторы как раз стремились подтвердить теорию о том, что бредовые исходные требования не могут быть преодолены с помощью хитроумных технологий программирования, и в любом случае породят кривой код - какие паттерны проектирования не применяй.
</SARCASM>

[raz]

https://www.linkedin.com/pulse/musin...s-jacob-roder/

[ax_mct]

Цитата:

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

https://www.linkedin.com/pulse/musin...s-jacob-roder/

Не уверен что по теме так как вроде бы здесь все на примерах и эта статья шире
Небольшое summary

Цитата:

Providing there's no deferral, the production update is executed a week after the sandbox update.

The customer wasn't able to test, though, during the week - some key stakeholders were out of the office.

We responded to the email to inform Microsoft we needed to defer, and never heard back.

We waited up until the deadline, still didn't hear back, so we reached out to ASfP (Advanced Support for Partners).

They told us to create a severity A support ticket requesting the deferral, which we did. Support said they'd tell the team and they'd follow up with us.

The update happened anyways.

И вот это

Цитата:

To our customer, though, we look like idiots that can't figure out what we're doing.

[raz]

Очередная проблема при апдейте на PU26.
Есть пара стандартных классов DocuTemplateRender и ExportToExcelDataEntityHelper, они обеспечивают поддержку entities, когда можно открывать данные в excel, править и публиковать назад в d365fo. Нам пришлось их продублировать, т.к. стандарт не работает в пакетном режиме. Так вот в PU26 поменялись списки поддерживаемых .net модулей, вот что заменили.

До PU26

using Microsoft.Dynamics.ApplicationPlatform.Services.Instrumentation;
using Microsoft.Dynamics.ApplicationPlatform.Environment;

PU26

using Microsoft.Dynamics.ApplicationPlatform.@Client.Instrumentation;

[ax_mct]

Цитата:

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

Очередная проблема при апдейте на PU26.
Есть пара стандартных классов DocuTemplateRender и ExportToExcelDataEntityHelper, они обеспечивают поддержку entities, когда можно открывать данные в excel, править и публиковать назад в d365fo. Нам пришлось их продублировать, т.к. стандарт не работает в пакетном режиме. Так вот в PU26 поменялись списки поддерживаемых .net модулей, вот что заменили.

До PU26

using Microsoft.Dynamics.ApplicationPlatform.Services.Instrumentation;
using Microsoft.Dynamics.ApplicationPlatform.Environment;

PU26

using Microsoft.Dynamics.ApplicationPlatform.@Client.Instrumentation;

А как именно вы это заметили? Пришел список изменений или вы столкнулись с ошибками в рантайм? Или при компиляции тестового?

[raz]

При ребилде наших моделей после накатки PU26.

[belugin]

JFYI
https://docs.microsoft.com/en-us/dyn...ed/one-version

Цитата:

Backward compatibility does not include non-X++/metadata APIs. Microsoft reserves the right to update versions of any dependencies the product uses, as well as remove dependencies without early warning. Microsoft does not commit to maintain backwards compatibility of dependent software libraries unless expressly stated.

[MarinaAX]

Установка PU24 поверх v.7.2 PU20 привело к тому, что модель перестрала компилироваться из-за того, что использовался "."-оператор, а чтобы избежать ошибки, надо было использовать "::"-оператор разрешения области (scope resolution operator).

Было: object1 = class1.Method1(...)
Стало: object1 = class1::Method1(...)

Остался только один вопрос: почему?