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 15:44 UTC
Updated:
29 Jul 2014 at 22:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
zhgenti commentedComment #2
zhgenti commentedComment #4
zhgenti commentedComment #6
oenie commentedAll occurences replaced; correct the issue below.
Add public access modifier in front of the function to adhere to the new OOP standards.
Comment #7
zhgenti commentedIssue in #6 addressed
Comment #9
zhgenti commented#7: rename-views-method-get_offset-to-getoffset-2002968-6.patch queued for re-testing.
Comment #10
oenie commentedIs there a specific reason why you have put an access modifier in front of set_items_per_page ?
Because that is part of a different rename, and might give clashes with #2003282: Rename Views method set_items_per_page() to setItemsPerPage()
Comment #11
dawehnerSo clearly needs work :(
Comment #12
zhgenti commentedIsn't it what you suggested in comment #6? I probably misunderstood something :(
Comment #13
heddnfunction getOffset()should be public. Also, I noticed that when you re-rolled the patch in #6 you rolled and interdiff, not a diff from HEAD.Comment #14
zhgenti commentedWhat a shame ( I looked at the different function in the comment 6... Should be fine now, also rolled over HEAD
Comment #15
dawehnerPerfect.
Comment #16
webchickCommitted and pushed to 8.x. Thanks!
Moving to Views for the change notice.
Comment #17
xjmWe'll create a single change notice in #1856630: [Change notice] [META] Rename Views methods to core standards.
Comment #18
xjmOops.