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:21 UTC
Updated:
29 Jul 2014 at 22:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
seantwalshWorking on this as part of the Portland2013 getting involved with core sprint.
Comment #2
seantwalshReplaced 11 instances of get_argument with getArgument.
Comment #3
oenie commentedAdd public access modifier in front of the functions to adher to the new OOP standards.
Comment #4
seantwalshWorking on adding access modifier.
Comment #5
seantwalshAdded public access modifier. Thanks oenie!
Comment #6
oenie commentedPatch looks ok to me now! If it passed the testbot review, it's RTBC.
Comment #7
oenie commentedUnfortunately we need a reroll, the patch doesn't apply anymore.
Comment #8
seantwalshRerolled!
Comment #9
dawehnerWrong indentation.
If we adapt it then properly: This should be @inheritdoc
Comment #10
dawehnerComment #11
seantwalshSorry about that, fixed indentation and updated with {@inheritdoc}.
Comment #12
dawehnerNice!
Comment #13
alexpottNeeds a reroll
Comment #14
jibranReroll
Comment #15
dawehnerGreat!
Comment #16
alexpottCommitted f1ba409 and pushed to 8.x. Thanks!