Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
views.module
Priority:
Minor
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
2 Apr 2013 at 20:01 UTC
Updated:
29 Jul 2014 at 22:07 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
alexmoreno commentedI will work on it as part of the Drupal London sprint may 2013
Comment #2
alexmoreno commentedI think it must be something wrong with the last patch, if someone could guide me through where to continue?
Comment #3
alexmoreno commentedok, sorry, I found the problem. I've recreated the patch again following this instructions: http://drupalladder.org/lesson/b93deb54-264f-19a4-49eb-db74bc6c2712
Comment #4
alexmoreno commentedComment #5
alexmoreno commentedHi tim.plunkett,
I have a quick question. I don't understand these changes:
- $options =& $form_state['values']['options'];
+ $options = &$form_state['values']['options'];
I guess it is simply coding standards, isn't it?
Thanks a lot.
Comment #6
alexmoreno commentedthere we go, here is the full patch for views and views_ui
Comment #7
alexmoreno commentedsorry, I forgot to attach the patch.
Comment #9
alexmoreno commented#7: 1959338-reroll-views.patch queued for re-testing.
Comment #11
dawehnerComment #12
superspring commentedSo here is my attempt at a re-roll for this patch.
A quick grep shows quite a few other changes that this no longer reflects.
If this passes the tests I'll add some more of them to the patch.
Comment #14
superspring commentedAttempt two
Comment #16
superspring commentedAttempt three
Comment #18
damiankloip commentedNice work superspring. I think an old underscored method call crept back in somehow :)
Comment #19
dawehnerEach line looks really fine.
Comment #20
ceardach commentedRemoved "Needs reroll" because latest patch is RTBC
Comment #21
xano18: 1959338-18.patch queued for re-testing.
Comment #22
webchickYay, less cruft! :)
Committed and pushed to 8.x. Thanks!
Comment #23
alexmoreno commentedgreat to know that my name will be in D8 :-)