Problem/Motivation

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

Proposed resolution

Use grep -r "render_exposed_form(" . to find all instances of render_exposed_form(
Replace render_exposed_form( with renderExposedForm(

Remaining tasks

reviews needed

[META] Rename Views methods to core standards

Files: 
CommentFileSizeAuthor
#7 2003346-render_exposed_form-7.patch3.13 KBJimSmith
PASSED: [[SimpleTest]]: [MySQL] 56,827 pass(es).
[ View ]
#2 2003346-render_exposed_form.patch3.12 KBJimSmith
PASSED: [[SimpleTest]]: [MySQL] 55,916 pass(es).
[ View ]

Comments

JimSmith’s picture

Assigned:Unassigned» JimSmith
JimSmith’s picture

Status:Active» Needs review
StatusFileSize
new3.12 KB
PASSED: [[SimpleTest]]: [MySQL] 55,916 pass(es).
[ View ]

Patch for review.

Status:Needs review» Needs work
Issue tags:-Novice, -VDC

The last submitted patch, 2003346-render_exposed_form.patch, failed testing.

JimSmith’s picture

Status:Needs work» Needs review
Issue tags:+Novice, +VDC

#2: 2003346-render_exposed_form.patch queued for re-testing.

Previous test failed because it failed to checkout from git. I'm hoping that was just a result of so many patches being submitted at the same time during the Drupalcon Portland code sprint.

JimSmith’s picture

Okay. Passed this time. Ready for review.

oenie’s picture

Status:Needs review» Needs work
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/exposed_form/ExposedFormPluginBase.phpundefined
@@ -115,7 +115,7 @@ public function buildOptionsForm(&$form, &$form_state) {
+  function renderExposedForm($block = FALSE) {

Add public access modifier in front of the function to adhere to the new OOP standards.

JimSmith’s picture

Status:Needs work» Needs review
StatusFileSize
new3.13 KB
PASSED: [[SimpleTest]]: [MySQL] 56,827 pass(es).
[ View ]

Alrighty then. Public access modifier added to front of the function to adhere to the new OOP standards and re-rolled.

oenie’s picture

Status:Needs review» Reviewed & tested by the community

Looks good to me now !

alexpott’s picture

Status:Reviewed & tested by the community» Fixed

Committed 15d26ab and pushed to 8.x. Thanks!

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

Anonymous’s picture

Issue summary:View changes

updated summary of issue