- Issue created by @penyaskito
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
I'm not 100% sure if the proposed solution is the right UX.
The user has a confirmation form where it's informed that this would happen. They could always go to the dashboard, remove the offending blocks, and try to uninstall again.
- 🇬🇧United Kingdom catch
I think this could use
ConfigEntityInterface::onDependencyRemoval()
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
Good call, Probably we should do just as in
\Drupal\layout_builder\Entity\LayoutBuilderEntityViewDisplay::onDependencyRemoval
and remove the block or section affected. - Merge request !60Issue #3512768: Uninstalling a module deletes dashboards that have a block provided by that module → (Merged) created by penyaskito
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
That MR is straight copypaste from LayoutBuilderEntityViewDisplay and works as expected. This needs tests tho.
- 🇪🇸Spain plopesc Valladolid
Code LGTM!
tested manually and the blocks from the uninstalled dependencies are removed, leaving the rest of the dashboard unaffected.
Marrking as RTBC.
- 🇪🇸Spain plopesc Valladolid
Out of the scope this issue but I wonder if would be worth to open a core issue where these methods would be moved to a trait for scenarios like this.
- Assigned to plopesc
- Status changed to RTBC
about 2 months ago 7:47am 14 August 2025 - 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
Not opening a follow-up now for core, but tagging this for being able to find it later.
-
penyaskito →
committed a091783e on 2.x
Issue #3512768 by penyaskito, plopesc, catch: Fix uninstalling a module...
-
penyaskito →
committed a091783e on 2.x
Automatically closed - issue fixed for 2 weeks with no activity.