Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards

CommentFileSizeAuthor
#14 views_rename_op_empty-2003556-14.patch5.62 KBSpartyDan
#12 views_rename_op_empty-2003556-12.patch5.28 KBnathangervais
#10 views_rename_op_empty-2003556-10.patch5.36 KBnathangervais
#8 views_rename_op_empty-2003556-8.patch5.25 KBnathangervais
#2 drupal-camelcase-replace-2003556-7451770.patch5.54 KBbaldwinlouie

Comments

baldwinlouie’s picture

baldwinlouie commented
Assigned: Unassigned » baldwinlouie

taking this with sillygwailo

baldwinlouie’s picture

baldwinlouie commented
StatusFileSize
new drupal-camelcase-replace-2003556-7451770.patch5.54 KB

attaching patch

baldwinlouie’s picture

baldwinlouie commented
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

heddn
Primary language English
Location Illinois
commented
Status: Needs work » Needs review
baldwinlouie’s picture

baldwinlouie commented

#2: drupal-camelcase-replace-2003556-7451770.patch queued for re-testing.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Needs work

This probably should get protected.

nathangervais’s picture

nathangervais commented
Assigned: baldwinlouie » nathangervais
Status: Needs work » Needs review
StatusFileSize
new views_rename_op_empty-2003556-8.patch5.25 KB

Rerolled patch with protected modifiers, from latest head.

dawehner’s picture

dawehner
Primary language German
commented
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

nathangervais commented
Title: Rename Views method op_empty() to addEmpty() » Rename Views method op_empty() to opEmpty()
Status: Needs work » Needs review
StatusFileSize
new views_rename_op_empty-2003556-10.patch5.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

nathangervais commented
Status: Needs work » Needs review
StatusFileSize
new views_rename_op_empty-2003556-12.patch5.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

SpartyDan commented
Assigned: nathangervais » Unassigned
Status: Needs work » Needs review
StatusFileSize
new views_rename_op_empty-2003556-14.patch5.62 KB

re-roll

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Perfect!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
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.