Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
views_ui.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
15 Nov 2012 at 10:16 UTC
Updated:
29 Jul 2014 at 21:31 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dawehnerThis is the only change i'm not sure why it's needed, even $form_state['values'] is filled on the rebuild request,
it doesn't work as expect, so we maybe need investigation.
Comment #2
dawehnerFinally figured it out.
Comment #3
damiankloip commentedHere are some quick tests to show this failing and passing with the Daniel's fix.
Comment #4
dawehnerwe never c&p in tests, we never do :)
No need for that, the UITestBase does this for us.
Comment #5
damiankloip commentedThe tests and the correct patch containing the fix...
Comment #6
damiankloip commentedAnd with the comments from #4! :)
Comment #7
damiankloip commentedWe don't need the $edit_url variable now either. Sorry for the issue noise.
Comment #8
tim.plunkettI've cleaned up the docs and coding style, but this indeed was a bug of my creation :(
The tests look good, just double checking.
Comment #9
catchLooks good, comes with nice test coverage. Committed/pushed to 8.x.