This was found in #2584603: PHP exception on manage fields after enabling Configuration Translation

Problem/Motivation

In WebTestBase::setUp() $this->installParameters() is called and the parameters are then passed into $this->doInstall(). That method does document a $parameters variable but it never actually uses it but instead calls $this->installParameters() itself.

Proposed resolution

Remove the unneeded argument and superfluous method call.

This is 100% backwards compatible and only changes test code so is eligible during the RC.

Remaining tasks

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 2589599-simpletest.patch1.19 KBtstoeckler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler created an issue. See original summary.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler at erdfisch commented
Status: Active » Needs review
FileSize
2589599-simpletest.patch1.19 KB
stevector’s picture

stevector
he/him
CreditAttribution: stevector at Pantheon commented

+1 for removing unused code.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +rc eligible

Its not only that, also tests are not an API so we can change it at any given point.

Nice find @tstoeckler!

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett at Acquia commented

I personally would have removed the second method call and used the parameters, but either way is better than HEAD.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed 2b92ca2 and pushed to 8.0.x. Thanks!

  • alexpott committed 805e1d6 on 
    Issue #2589599 by tstoeckler: Dead code in WebTestBase

Status: Fixed » Closed (fixed)

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