Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
views tests that need to create views from config should just create them, not 'import' them.
over in #1808248: Add a separate module install/uninstall step to the config import process, i'm messing with the ConfigImporter, because, you know, it's the importer. it makes zero sense that changes i'm making there should blow up code over here. i can see that this 'just extend ConfigImporter' pattern prolly didn't start with views, but it must die.
Comment | File | Size | Author |
---|---|---|---|
#1 | 2095115-1-views-wtf-import.patch | 19.06 KB | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) CreditAttribution: Anonymous commentedaaaaand here's a patch.
Comment #2
dawehnerNice patch, as this really simplifies that piece of code.
Comment #3
jibranTagging.
Comment #4
tim.plunkettThis pattern *totally* started with Views, and yet I agree it must die :)
+1
Comment #5
webchickCommitted and pushed to 8.x. Thanks!