Part of meta-issue #2002650: [meta, no patch] improve maintainability by removing unused local variables

File /core/modules/editor/lib/Drupal/editor/Tests/EditorLoadingTest.php

Line 82: Unused local variable $settings
Line 147: Unused local variable $settings

CommentFileSizeAuthor
#1 drupal-core-remove-unused-local-variable-2080425.patch1.63 KBmrsinguyen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrsinguyen’s picture

mrsinguyen CreditAttribution: mrsinguyen commented
Status: Active » Needs review
FileSize
drupal-core-remove-unused-local-variable-2080425.patch1.63 KB
enhdless’s picture

enhdless CreditAttribution: enhdless commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Tested patch, it works great!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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