- Issue created by @malcomio
- Merge request !20Issue #3491001: Use update hook to change permissions, not hook_post_update_NAME → (Merged) created by malcomio
- 🇺🇸United States smustgrave
Not sure I'm seeing how this is is breaking things?
- 🇬🇧United Kingdom malcomio
If an update hook in another module tries to make changes to permissions as part of the same deployment where search_api_exclude_entity is being upgraded, there's no way to ensure that the other hook runs after search_api_exclude_entity has updated the permissions.
That means that the permission change in the other module fails, because the outdated search_api_exclude_entity permissions are still assigned to some roles.
By using an update hook in search_api_exclude_entity, the custom module can declare a dependency on it.
-
smustgrave →
committed d52aca1d on 3.0.x authored by
malcomio →
Issue #3491001: Use update hook to change permissions, not...
-
smustgrave →
committed d52aca1d on 3.0.x authored by
malcomio →
- 🇺🇸United States smustgrave
Odd but sure, updated the hook based on our the 3.0.x version.
- 🇮🇳India suryabhi
issue still exist with different error
Exception thrown while performing a schema update. Cannot rename 'taxonomy_term__field_display_name' to 'field_deleted_data_9ae423e048': table 'field_deleted_data_9ae423e048' already exists.
- Merge request !21Resolve #3491001 "Permissions problems when upgrading from 2.x to 3.x" → (Closed) created by suryabhi
- 🇬🇧United Kingdom malcomio
@suryabhi I don't think that those exceptions relate to this module, and I don't see why we would need the change you're suggesting in merge request 21.
I think the appropriate status for this issue is probably "Fixed"
- Status changed to Fixed
1 day ago 6:49am 4 February 2025