[DRAFT] Add Procedure ReinstallExtension to trigger Install Code from AL #1657
Conversation
github-actions
bot
added
AL: System Application
From Fork
|
Issue #1659 is not valid. Please make sure you link an issue that exists, is open and is approved. |
JesperSchulz
changed the title
| @@ -405,5 +405,52 @@ codeunit 2500 "Extension Installation Impl" | |||
|
|
|||
| exit(Result = PolicyToTestFor); | |||
| end; | |||
|
|
|||
| procedure ReinstallExtension(PackageId: Guid; lcid: Integer; IsUIEnabled: Boolean): Boolean | |||
There was a problem hiding this comment.
We do not want to introduce this kind of trigger/functionality in the app. Re-running the upgrade code is dangerous, it can lead to data corruption.
If the version has not changed there must not be upgrade.
If there is a huge need to do this, we have platform capabilities to trigger the upgrade even if the version has not changed. We can trigger the upgrade in place, we are doing this for the cloud migration.
Alternative is to increase the version of the app and to publish that one.
Another alternative is that you can run the data transfer now if the cloud migration is enabled, so if you are running this code during the cloud migration, it can be run via an action from the UI.
This is a Draft because i think this is a hack and could be better solved by adding a possibility to trigger upgrade code (with DataTransfer) from AL Code.
But currently this seems to be the only way to trigger code that uses DataTransfer from AL.
Summary
Adds a new Procedure "ReinstallExtension" to Extension Management. The Procedure uninstalls and then installs the extension again. It also reinstalls the extensions that are dependent on the reinstalled extension.
By Installing the Extension the Install Codeunints of the extension are triggered.
Inside Install Codeunits we can use DataTransfer to move Data between Tables/Fields.
This is usefull if you have features that can be enabled by the user, but these features require some data beeing moved to new tables/fields. If we copy the Data using repeat until this can be very slow and time consuming. Accordin to the docs using DataTransfer instead, can result in a "~200x performance improvement".
Work Item(s)
Fixes #1659