It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.
Related to the Views in Drupal Core initiative.
Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards
Working on as part of core code sprint.
Replace two instances of cache_flush() with cacheFlush().
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/cache/CachePluginBase.phpundefined @@ -179,7 +179,7 @@ function cache_get($type) { + function cacheFlush() {
Add public access modifier in front of the function to adher to the new OOP standards.
Added public access modifier to cacheFlush function. Thanks, oenie!
Looks fine by me now.
Committed/pushed to 8.x, thanks!
Automatically closed -- issue fixed for 2 weeks with no activity.
Comments
Comment #1
web.dev.cas CreditAttribution: web.dev.cas commentedWorking on as part of core code sprint.
Comment #2
web.dev.cas CreditAttribution: web.dev.cas commentedReplace two instances of cache_flush() with cacheFlush().
Comment #3
oenie CreditAttribution: oenie commentedAdd public access modifier in front of the function to adher to the new OOP standards.
Comment #4
web.dev.cas CreditAttribution: web.dev.cas commentedAdded public access modifier to cacheFlush function. Thanks, oenie!
Comment #5
oenie CreditAttribution: oenie commentedLooks fine by me now.
Comment #6
catchCommitted/pushed to 8.x, thanks!