Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
views.module
Priority:
Major
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
9 Oct 2012 at 00:28 UTC
Updated:
29 Jul 2014 at 21:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
xjmWe'll also need to add config, as seen in #1786990-13: [Module]Bundle is registered too late in WebTestBase::setUp(). Weird that something doesn't already require it.
Comment #2
xjmThis is unblocked now!
Comment #3
xjmLet's see if I can do this. :)
Comment #4
tim.plunkettI already tested this in the patch test issue, but it's just this + #1810378: Remove unneeded dependency on config.module
Since this is part of the core move, I'm going to roll it into my sandbox rebuilding steps, by amending it into http://drupalcode.org/sandbox/tim.plunkett/1799554.git/commitdiff/1342dfe when the time comes.
This is just a placeholder issue to remind me :)
Comment #5
tim.plunkettOkay, new magical commit is http://drupalcode.org/sandbox/tim.plunkett/1799554.git/commitdiff/d6eade2
I've updated the factoid :)
Comment #6
xjmLOL yay.
Comment #7
tim.plunkettSo I think we're actually still blocked on the nastier bug.
#1783692: Test the bundle manager to show the fail on update.php
#1708692: Bundle services aren't available in the request that enables the module
Comment #8
tim.plunkettThis is blocked on #1708692: Bundle services aren't available in the request that enables the module
Comment #9
xjmComment #10
tim.plunkettTesting to see if the patch in #1708692-45: Bundle services aren't available in the request that enables the module works for this.
Comment #11
tim.plunkettThat went in!
Comment #12
aspilicious commentedOh yes!
Comment #13
webchickGreat. I consider this clean-up after the initial feature, and only didn't go in at that time because we were blocked on critical bugs. Those have been resolved, so now it makes sense to put this in.
Committed and pushed to 8.x. Thanks!!
Comment #14.0
(not verified) commentedUpdated issue summary.