Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
views.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
24 May 2013 at 03:21 UTC
Updated:
29 Jul 2014 at 22:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
toddtomlinson commentedComment #2
toddtomlinson commentedPatch applied that renames build_group_form.
Comment #3
toddtomlinson commentedComment #4
toddtomlinson commentedFixed the patch. Had included previous changes. Only lists build_group_form now.
Comment #5
oenie commentedAdd protected access modifier in front of the function to adher to the new OOP standards.
Comment #6
watsonerror commentedgot it
Comment #7
watsonerror commentedfunction is now protected
Comment #8
oenie commentedYou have to make sure there is one patch that contains all changes needed.
Your latest patch is only the latest change.
Try again, including all changes from the original patches.
Comment #9
zschmid commentedUpdated patch
Comment #11
zschmid commentedThe issue here is that the buildGroupForm namespace is already used at around line 419 of FilterPluginBase.php .
renaming the build_group_form method to buildGroupForm throws all sorts of errors. any suggestions on what we should rename this to, to avoid the conflict?
Comment #12
shnark commentedi read the function and picked a more specific name.
Comment #13
dawehnerLooks wonderful. There are just these two instances.
Comment #14
webchickCommitted and pushed to 8.x. Thanks!
Moving to Views for the change notice.
Comment #15
xjmWe'll create a single change notice in #1856630: [Change notice] [META] Rename Views methods to core standards.