Support from Acquia helps fund testing for Drupal Acquia logo

Comments

baldwinlouie’s picture

Assigned: Unassigned » baldwinlouie

taking by baldwinlouie and sillygwilo

baldwinlouie’s picture

Status: Active » Needs review
FileSize
3.46 KB

attaching patch

baldwinlouie’s picture

rerolling with access modifier

Status: Needs review » Needs work

The last submitted patch, drupal-camelcase-replace-2002496.3.patch, failed testing.

oenie’s picture

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.phpundefined
@@ -239,7 +239,7 @@ function has_more_records() {
+  function exposedFormAlter(&$form, &$form_state) { }

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/SqlBase.phpundefined
@@ -335,7 +335,7 @@ function offset_exposed() {
+  function exposedFormAlter(&$form, &$form_state) {

Still missing some public access modifiers.

+++ b/core/modules/views/views.moduleundefined
@@ -1489,7 +1489,7 @@ function views_exposed_form($form, &$form_state) {
-function views_form_views_exposed_form_alter(&$form, &$form_state) {
+function views_form_views_exposedFormAlter(&$form, &$form_state) {

This is an incorrect replacement. You are changing the name of a hook, which subsequently will never be called anymore. Should stay as is. (although the hook-system is on it's way out if i'm not mistaken)

baldwinlouie’s picture

Status: Needs work » Needs review
FileSize
3.05 KB

Rerolled with correct modifiers.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

No calls left, and patch still applies.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed c481e2f and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.