Support from Acquia helps fund testing for Drupal Acquia logo

Comments

baldwinlouie’s picture

Assigned: Unassigned » baldwinlouie

taking this with sillygwailo

baldwinlouie’s picture

attaching patch

baldwinlouie’s picture

Status: Active » Needs review

changing to needs review

Status: Needs review » Needs work

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

heddn’s picture

Status: Needs work » Needs review
baldwinlouie’s picture

dawehner’s picture

Status: Needs review » Needs work

This probably should get protected.

nathangervais’s picture

Assigned: baldwinlouie » nathangervais
Status: Needs work » Needs review
FileSize
5.25 KB

Rerolled patch with protected modifiers, from latest head.

dawehner’s picture

Status: Needs review » Needs work
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/filter/GroupByNumeric.phpundefined
@@ -44,7 +44,7 @@ function op_simple($field) {
-  function op_empty($field) {
+  protected function addEmpty($field) {

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/filter/InOperator.phpundefined
@@ -125,13 +125,13 @@ function operators() {
-          'method' => 'op_empty',
+          'method' => 'addEmpty',

Let's go with opEmpty.

nathangervais’s picture

Title: Rename Views method op_empty() to addEmpty() » Rename Views method op_empty() to opEmpty()
Status: Needs work » Needs review
FileSize
5.36 KB

Patch rerolled. Renamed to opEmpty, updating bug title.

Status: Needs review » Needs work

The last submitted patch, views_rename_op_empty-2003556-10.patch, failed testing.

nathangervais’s picture

Status: Needs work » Needs review
FileSize
5.28 KB

Rerolled Patch.

Status: Needs review » Needs work

The last submitted patch, views_rename_op_empty-2003556-12.patch, failed testing.

SpartyDan’s picture

Assigned: nathangervais » Unassigned
Status: Needs work » Needs review
FileSize
5.62 KB

re-roll

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Perfect!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 6be4fdb and pushed to 8.x. Thanks!

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