Closed (won't fix)
Project:
Drupal core
Version:
8.0.x-dev
Component:
forum.module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
2 Jul 2013 at 00:09 UTC
Updated:
11 Sep 2014 at 08:31 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
andypostlet's see the tests...
Comment #3
larowlanIs there an equivalent of field_delete_instance in d8? that used to have the 'cleanup' flag.
So we could call that, if something else used it - it would stick around, else it would be removed.
Comment #4
andypostre-roll
Comment #6
andypostContainers should be removed, also comment about disabled taxonomy module makes no sense
Comment #9
larowlanThe forum uninstall tests can be removed, as they are to check the field is gone.
The uninstall test I'm not sure on, its making sure that taxonomy can be removed once forum has been - from memory.
Comment #9.0
drummUpdated issue summary.
Comment #10
andyceo commented6: drupal8.forum-module.2032699-6.patch queued for re-testing.
Comment #12
temoor commentedRemoved field deletion and it's part of tests.
Comment #13
temoor commentedComment #15
xjmSo, I believe this issue runs counter to the expectations we've set up for config dependencies. What should happen is:
So, I think this issue is actually a wonfix, and what we should do instead is clean up all our data for #2224581: Delete forum data on uninstall.
Comment #16
xjmComment #17
andypost+1 to wontfix, not sure that "disable module" will be reverted for core so issue makes no sense without that
Comment #18
alexpottTotally agree to won't fix in light of #2224581: Delete forum data on uninstall