Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
views.module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
24 May 2013 at 15:15 UTC
Updated:
29 Jul 2014 at 22:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
hansenm commentedI'm working on this at the DrupalCon Portland sprint.
Comment #2
hansenm commentedComment #4
heddnTests failing. Please investigate and re-roll a patch.
Comment #5
daven commented#2: views-rename-compile_fields-2002428-2.patch queued for re-testing.
Comment #6
daven commentedAccess modifier needs to be added to the function line as mentioned in the meta issue discussion:
http://drupal.org/node/1856630#comment-7451652
Should be "public function ..."
Comment #7
heddnActually, protected seems more appropriate ($this->instantiateView) is the only call. Let's see if it keeps the testbot happy.
Comment #8
heddnLet's try the right patch.
Comment #9
dcam commented#8 looks good to me. All instances of the instantiate_view() method have been renamed. instantiateView() has been given protected visibility.
I think that #2 was mistakenly posted to the wrong issue. It should have gone into #2002428: Rename Views method compile_fields() to compileFields().
Comment #10
alexpottCommitted 7790f21 and pushed to 8.x. Thanks!