User role delete getting error

This does delete the DB delete query code (plus all other code related the DB stored config), but it does not replace the functionality by actually deleting the config for the now deleted role. It just gets the value and does nothing with it.

Updated patch to also remove the now defunct config entry.

There's no schema for the config on this module.

My change was so small that I'm going to mark this as RTBC too.

Schema is addressed in 🐛 Missing config schema & fix weight spelling RTBC .

@ELC could you please create a MR and rebase this on 10.x now that 🐛 Missing config schema & fix weight spelling RTBC is merged?

aaron.ferris → made their first commit to this issue’s fork.

aaron.ferris → changed the visibility of the branch 3226405-userrole-delete to hidden.

Patch didn't apply cleanly for me off 10.x, but ive raised a MR with what looks like the main purpose of the patch

1. Delete db query functions (they look redundant anyway)
2. Clear role related config in front_page_user_role_delete

MR contains everything that was in the patch. Thank you @aaron.ferris

All of the database related queries were redundant code now that settings for the module are kept in config instead. The code in hook_user_role_delete() is the only part that hadn't been replaced resulting in dangling config items.

Fixed in 10.x (part of another commit).

Anybody → closed merge request !6

Automatically closed - issue fixed for 2 weeks with no activity.