- Issue created by @trackleft2
- Merge request !141Close #3501451 Allow module uninstallation and reinstallation. β (Merged) created by trackleft2
- πΊπΈUnited States Greg Boggs Portland Oregon
greg boggs β made their first commit to this issueβs fork.
-
greg boggs β
committed 4298f6ef on 2.x authored by
trackleft2 β
Issue #3501451 by trackleft2: [regression] Configuration no longer...
-
greg boggs β
committed 4298f6ef on 2.x authored by
trackleft2 β
- πΊπΈUnited States Greg Boggs Portland Oregon
Wow. Thank you. Epic code :)
- πΊπΈUnited States trackleft2 Tucson, AZ πΊπΈ
My code was broken in the update FYI
this line in easy_breadcrumb.install
use Drupal\Core\Update\UpdateException;
should beuse Drupal\Core\Utility\UpdateException;
- πΊπΈUnited States trackleft2 Tucson, AZ πΊπΈ
Additionally the PHPUnit test says the module is failing to uninstall, which I am unable to reproduce manually.
https://git.drupalcode.org/issue/easy_breadcrumb-3501451/-/jobs/4101038#L80
- Merge request !142Issue #3501451 PHPCS fixes and fix broken update function β (Merged) created by trackleft2
- πΊπΈUnited States trackleft2 Tucson, AZ πΊπΈ
TIL we don't need the dependency keys after all.
Seems Drupal now uninstalls module installed config automatically.
We can keep the test, but everything else can be removed.
New merge request here: https://git.drupalcode.org/project/easy_breadcrumb/-/merge_requests/142
-
greg boggs β
committed d14791ba on 2.x authored by
trackleft2 β
Issue #3501451: [regression] Configuration no longer removed upon...
-
greg boggs β
committed d14791ba on 2.x authored by
trackleft2 β
Automatically closed - issue fixed for 2 weeks with no activity.