There is a wrong variable name in /core/modules/node/src/Form/DeleteMultiple.php

CommentFileSizeAuthor
#3 2945369-3-wrong_varaible_name.patch483 bytesPasqualle
wrong_varaible_name.patch496 bytesPasqualle
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pasqualle created an issue. See original summary.

Status: Needs review » Needs work

The last submitted patch, wrong_varaible_name.patch, failed testing. View results

Pasqualle’s picture

Pasqualle
Location 🇭🇺 Budapest
CreditAttribution: Pasqualle as a volunteer commented
Status: Needs work » Needs review
FileSize
2945369-3-wrong_varaible_name.patch483 bytes
timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
CreditAttribution: timmillwood at Appnovation for Pfizer, Inc. commented
Status: Needs review » Reviewed & tested by the community

Nice find!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott as a volunteer and commented
Title: Wrong variable name » Wrong variable name in \Drupal\node\Form\DeleteMultiple
Status: Reviewed & tested by the community » Fixed

This only applies to 8.5.x because entity delete forms have changed in 8.6.x - the new 8.6.x version does not have this bug. As this is a form class it is not API and therefore making a property protected is not an API change.

Committed 82500fa and pushed to 8.5.x. Thanks!

  • alexpott committed 82500fa on 8.5.x 
    Issue #2945369 by Pasqualle: Wrong variable name in \Drupal\node\Form\...

Status: Fixed » Closed (fixed)

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