valtech / aem-easy-content-upgrade

AEM Easy Content Upgrade simplifies content migrations in AEM projects
Other
61 stars 25 forks source link

Take script content into consideration for checking if script was already executed #213

Closed kwin closed 1 year ago

kwin commented 1 year ago

Currently in /var/aecu-installhook/<package-path-of-script> there is a simple date property maintained with name executed to decide if the script has been executed before. That one does not take into account modifications on the groovy script itself, i.e. even if the groovy script has been modified it will not re-execute it. It would be good to store a hash of the script which was executed and only skip execution if it was executed before with the same hash.

kwin commented 1 year ago

This isn't true, I outlined the exact conditions when a script is re-executed in https://github.com/valtech/aem-easy-content-upgrade/pull/214.