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

File /core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Display.php

Line 64: Unused local variable $section
Line 68: Unused local variable $display

CommentFileSizeAuthor
#2 drupal-core-remove-unused-local-variable-2080033-2.patch633 bytesroyal121
#1 drupal-core-remove-unused-local-variable-2080033.patch755 bytesmrsinguyen

Comments

mrsinguyen’s picture

mrsinguyen commented
Status: Active » Needs review
StatusFileSize
new drupal-core-remove-unused-local-variable-2080033.patch755 bytes

Attached patch.

royal121’s picture

royal121 commented
Issue summary: View changes
StatusFileSize
new drupal-core-remove-unused-local-variable-2080033-2.patch633 bytes

The above patch does not apply. Here is a new patch.

deneo’s picture

deneo commented
Status: Needs review » Reviewed & tested by the community

The patch applies without problem and solve the issue. Thank you!

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.