Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sam Moore’s picture

Assigned: Unassigned » Sam Moore

I'll have a look.

Sam Moore’s picture

Assigned: Sam Moore » Unassigned
Status: Active » Needs review
FileSize
1.68 KB

See patch attached. This has passed local testing on Views Plugins.
Please review.

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

The last submitted patch, renamefunction-2003364-2.patch, failed testing.

Sam Moore’s picture

Status: Needs work » Needs review

#2: renamefunction-2003364-2.patch queued for re-testing.

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

The last submitted patch, renamefunction-2003364-2.patch, failed testing.

Sam Moore’s picture

Status: Needs work » Needs review
FileSize
1.68 KB

See re-uploaded patch attached. This has passed local testing on Views Plugins.
Please review.

oenie’s picture

Status: Needs review » Needs work
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/filter/FilterPluginBase.phpundefined
@@ -1151,16 +1151,16 @@ function exposed_translate(&$form, $type) {
+  function prepareFilterSelectOptions(&$options) {

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

All occurences seem to be replaced. The corrected patch should then be RTBC...

Sam Moore’s picture

Status: Needs work » Needs review
FileSize
1.69 KB

Added "protected" access modifier to method

oenie’s picture

Status: Needs review » Reviewed & tested by the community

Looks fine to me now !

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed a2b14f1 and pushed to 8.x. Thanks!

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