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:37 UTC
Updated:
29 Jul 2014 at 22:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
heddnComment #2
sillygwailoTaking this with baldwinlouie.
Comment #3
sillygwailoPatch attached, no testing done, method only appears in one place.
Comment #4
dawehnerThis should be marked as protected
These are the only places the method is used.
Comment #6
sillygwailoRe-rolled as protected.
Comment #8
jeroentrename can_build_group to canBuildGroup.
Comment #9
jeroentmarking as needs review.
Comment #11
mariafromatoz commentedPatch re-rolled, tested locally, let's see if I am lucky
Comment #12
oenie commentedLooks good to me now !
Comment #13
alexpottNeeds a re-roll
Comment #14
jibranReroll
Conflict
Resloved
Comment #15
sillygwailoEmpty patch? Re-rolled.
Comment #17
mariafromatoz commentedRe-rolling the patch following suggestions
[I am learning about patching but I do not think this is going to be the one; I am happy to try again if someone tells me what needs changing please]
Comment #18
dawehnerShould be a protected method.
Comment #19
sillygwailoRe-rolled with protected.
Comment #20
sillygwailoComment #22
dawehnerYou made a typo here: Should be canBuildGroup instead
Comment #23
jeroentRenamed can_build_group to canBuildGroup. Added protected access modifier as mentioned by dawehner.
Comment #24
dawehnerSeems fine.
Comment #25
alexpottCommitted 7d31103 and pushed to 8.x. Thanks!