Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kenianbei’s picture

Assigned: Unassigned » kenianbei
kenianbei’s picture

Status: Active » Needs review
FileSize
3.1 KB

Status: Needs review » Needs work

The last submitted patch, drupal-rename-value-submit-2003236-2.patch, failed testing.

SpartyDan’s picture

@kenianbei - It looks like you should resubmit your patch.

The failure message was: Setup environment - The testbot client is probably malfunctioning.

Your patch looks good to me.

aspilicious’s picture

+++ b/core/modules/taxonomy/lib/Drupal/taxonomy/Plugin/views/filter/TaxonomyIndexTid.phpundefined
@@ -330,7 +330,7 @@ function validate_term_strings(&$form, $values) {
+  function valueSubmit($form, &$form_state) {

should be public function valueSubmit (in all cases)

zschmid’s picture

updated patch

zschmid’s picture

Status: Needs work » Needs review
aspilicious’s picture

Status: Needs review » Needs work

Looking at this again, this should be protected :s. Srry!
The function is only used internally by filter plugins.

SpartyDan’s picture

Status: Needs work » Needs review
FileSize
3.15 KB

functions changed to protected

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed c93be24 and pushed to 8.x. Thanks!

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