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

File /core/modules/language/lib/Drupal/language/HttpKernel/PathProcessorLanguage.php

Line 143: Unused local variable $host

CommentFileSizeAuthor
#3 drupal-core-remove-unused-local-variable-2081189.patch894 bytesmrsinguyen
#1 drupal8.other_.2081189-1.patch891 bytessmira

Comments

smira’s picture

smira commented
Status: Active » Needs review
StatusFileSize
new drupal8.other_.2081189-1.patch891 bytes

bye bye $host

Status: Needs review » Needs work

The last submitted patch, drupal8.other_.2081189-1.patch, failed testing.

mrsinguyen’s picture

mrsinguyen commented
Status: Needs work » Needs review
StatusFileSize
new drupal-core-remove-unused-local-variable-2081189.patch894 bytes
parthipanramesh’s picture

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

Looks fine!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
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.