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:27 UTC
Updated:
29 Jul 2014 at 22:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sillygwailobaldwinlouie and I are taking this one.
Comment #2
sillygwailoHere's a patch.
Comment #3
sillygwailoComment #5
Anonymous (not verified) commentedGoing to re-roll.
Comment #6
Anonymous (not verified) commentedI rolled a new patch and added the public keyword to the function, per the feedback on this issue: http://drupal.org/node/2002460
Comment #8
oenie commentedTo avoid confusion over the previous patches, i'm rerolling patch #2 with corrections on the acces modifiers for the changed functions.
Comment #9
oenie commentedComment #10
heddnNeeds a re-roll.
$ git apply 2002918-rename-get_items-3.patch
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php:1110
error: core/modules/views/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php: patch does not apply
Comment #11
sillygwailoRe-rolled against 8.x. Might need another re-roll when the advanced_render() change is committed.
Comment #12
sillygwailoPatch attached.
Comment #14
sillygwailo#12: 2002918-rename-get_items.patch queued for re-testing.
Comment #15
aspilicious commented:D thnx!
Comment #16
somepal commentedreviewed and checked all the calls. gtg with #12 2002918-rename-get_items.patch
Comment #17
alexpottNeeds a reroll
Comment #18
jibranRe-roll
Conflict
Resolved
Comment #19
aspilicious commentedprotected function getItems($values) {
Comment #20
oenie commentedComment #21
sillygwailoRe-rolled with protected in the PrerenderList.php.
Comment #23
sillygwailoRe-roll.
Comment #24
dawehnerPerfect
Comment #25
alexpottCommitted ac6bbe4 and pushed to 8.x. Thanks!
Comment #26
tim.plunkettI started porting FullCalendar to D8, and this bit me hard.
I remember adding this method to D7 so contrib could use it.
It very much needs to be public.
Comment #27
aspilicious commentedComment #28
alexpottCommitted a24345a and pushed to 8.x. Thanks!