Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
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!