Closed (fixed)
Project:
Taxonomy Views Integrator
Version:
8.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
14 Feb 2017 at 19:33 UTC
Updated:
25 Oct 2017 at 22:30 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
dylan donkersgoed commentedRan into this issue myself, here's a patch. There doesn't seem to be an actual corresponding config entity and I don't think the uuid is actually used for anything so it looks like just taking out the one line and updating the tests is enough.
Note that this won't fix existing config - you'll need to manually uncheck the tvi option on the vocabulary/term, save it, then load it and reconfigure it for each previously configured vocabulary/term.
Comment #3
dawehnerThank you for providing a patch!
Yeah I'm not sure we can do much for existing content, as the API for example makes it impossible to change a UUID of a config entity.
Comment #4
star-szrI closed #2883119: StorageComparer thinks tvi.taxonomy_vocabulary.NAME.yml is a renamed version of taxonomy.vocabulary.NAME.yml as a duplicate of this issue.
I verified that the patch removes all references to uuid from the module. Not sure it's worth writing a test here, so setting to RTBC.
Comment #5
kristiaanvandeneyndePlease also add credit for legolasbo for his patch in the other issue.
Comment #6
star-szrComment #7
sittard commentedThanks for this patch. I've tested this and can confirm it's working as expected. Please can one of the maintainers update the release to include this patch. It's a critical fix especially now with the Drupal 8 era of config/sync.
Comment #8
kevinquillen commentedComment #10
kevinquillen commentedComment #11
kevinquillen commentedComment #12
dawehnerThank you @kevinquillen!