Files: 
CommentFileSizeAuthor
#11 views-rename_op_simple-2003522-11.patch4.17 KBJeroenT
PASSED: [[SimpleTest]]: [MySQL] 55,968 pass(es).
[ View ]
#8 drupal_rename_op_simple-2003522-8.patch4.15 KBnathangervais
PASSED: [[SimpleTest]]: [MySQL] 57,740 pass(es).
[ View ]
#5 views-rename_op_simple-2003522-5.patch4.23 KBJeroenT
PASSED: [[SimpleTest]]: [MySQL] 56,991 pass(es).
[ View ]
#2 rename_op_simple.patch2.46 KBtrevjs
FAILED: [[SimpleTest]]: [MySQL] Repository checkout: failed to checkout from [git://git.drupal.org/project/drupal.git].
[ View ]

Comments

Assigned:Unassigned» trevjs

Assigned:trevjs» Unassigned
Status:Active» Needs review
StatusFileSize
new2.46 KB
FAILED: [[SimpleTest]]: [MySQL] Repository checkout: failed to checkout from [git://git.drupal.org/project/drupal.git].
[ View ]

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/filter/Date.phpundefined
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/filter/Date.phpundefined
@@ -178,7 +178,7 @@ function op_between($field) {
@@ -178,7 +178,7 @@ function op_between($field) {
+  function addSimpleQuery($field) {

functions needs to be public.

Status:Needs review» Needs work

The last submitted patch, rename_op_simple.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new4.23 KB
PASSED: [[SimpleTest]]: [MySQL] 56,991 pass(es).
[ View ]

Rename op_simple() to addSimpleQuery().

Status:Needs review» Reviewed & tested by the community

Done

Status:Reviewed & tested by the community» Needs work

In #2003582: Rename Views method op_equal() to opEqual() it was decided to not rename a function from op_whatever to addWhatever. So to be consistent this patch should just change the name to opSimple (we should also drop the Query as it is not a Query - it appears to add a comparison operator)

Status:Needs work» Needs review
StatusFileSize
new4.15 KB
PASSED: [[SimpleTest]]: [MySQL] 57,740 pass(es).
[ View ]

Rerolled patch to rename op_simple to opSimple

Title:Rename Views method op_simple() to addSimpleQuery()Rename Views method op_simple() to opSimple()

Fixing title

Status:Needs review» Needs work

Talkes with dawehner about this, should be protected. Srry!

Status:Needs work» Needs review
StatusFileSize
new4.17 KB
PASSED: [[SimpleTest]]: [MySQL] 55,968 pass(es).
[ View ]

Changed access modifier to protected as mentioned by aspilicious.

Status:Needs review» Reviewed & tested by the community

No calls are left and code looks fine.

Status:Reviewed & tested by the community» Fixed

Committed 35a7926 and pushed to 8.x. Thanks!

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