Hi,
Context : a new D8 instance 8.2.5 with module_filter enabled, Pathauto who requires Token (but Token is disabled).
I search 'Path' in the search input then check Pathauto and click install.
Then I'm not redirected to the ConfirmFom who normally ask to confirm the installation of dependencies modules, instead the system module form reload and neither Pathauto or it's dependence are installed.
The redirect from the submitForm() from modules/system/src/Form/ModulesListForm is overridden in the submit callback module_filter_system_modules_redirect_submit.
The redirect from the submitForm() is call when there are dependencies to the module we want to enable in order to redirect to the system.modules_list_confirm.
Maybe a tempstore (user.private_tempstore) should be used instead but it implies to delete the value at some point as the search form element is used on module list, uninstall and perm form.
Anyway imo the redirect in module_filter_system_modules_redirect_submit should be removed atm, to be able to enable module with dependencies is prior to the filters input default value.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | module_filter-redirects_submit_breaks_confirm-2843318-3.patch | 1.44 KB | greenskin |
Comments
Comment #2
romsP commentedComment #3
greenskin commentedHere's a fix that allows the filter value to persist through the confirm submission process.
Comment #4
greenskin commentedMoving this along.
Comment #5
joelpittetYou can't disable a module in D8, do you mean uninstalled?
Comment #7
greenskin commented