- Issue created by @dieterholvoet
- π§πͺBelgium dieterholvoet Brussels
Never mind, the module isn't actually obsolete: it provides 'update/delete own' permissions that core doesn't provide. This means that we should create a new version that converts part of the permissions to the ones provided by core, while keeping the others in this module.
- π§πͺBelgium dieterholvoet Brussels
What we could do is log a message in the update hook in case no roles are using permissions provided by this module anymore, which means that the module can be removed.
- Merge request !7Add update hook to convert permissions to the new core permissions β (Open) created by dieterholvoet
- Status changed to Needs review
10 days ago 12:26pm 25 February 2025 - π§πͺBelgium dieterholvoet Brussels
I implemented the update hook. I tried to let the update hook uninstall the module, but apparently that's not supported (yet): π Support uninstalling the module providing the update hook in hook_update_N Needs work .
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
This looks good to me - should we bother with an update test?
- π§πͺBelgium dieterholvoet Brussels
Writing tests is not my strong suit so I won't probably spend the time, but feel free if you want to.