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 03:36 UTC
Updated:
29 Jul 2014 at 22:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
davmorr commentedComment #2
davmorr commentedChanged cache_set_expire() to cacheSetExpire()
Comment #3
davmorr commentedComment #4
oenie commentedAdd public access modifier in front of the function to adher to the new OOP standards.
Comment #5
oenie commenteddouble post - safely ignore
Comment #6
marlatt commentedAdded public to method. Patch looks good.
Comment #7
oenie commentedI think you missed a few occurences of this method:
core/modules/views/lib/Drupal/views/Plugin/views/cache/CachePluginBase.php:128:
core/modules/views/lib/Drupal/views/Plugin/views/cache/CachePluginBase.php:133:
core/modules/views/lib/Drupal/views/Plugin/views/cache/Time.php:121:
In my previous comment i mentioned public as modifier, but since it seems to get invoked on $this, the access modifier can be protected instead of public.
Be sure to add it to the function declaration missed as well.
Comment #8
marlatt commentedRe-rolled patch with corrections from #7
Comment #10
marlatt commentedUnable to apply patch from #8 to create interdiff. Methods changed and method access set to protected.
Comment #12
aspilicious commented#10: rename_views_method-2002412-10.patch queued for re-testing.
Comment #13
aspilicious commentedComment #14
alexpottCommitted 8625dfc and pushed to 8.x. Thanks!