Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Part of meta-issue #2002650: [meta, no patch] improve maintainability by removing unused local variables
File /core/modules/taxonomy/lib/Drupal/taxonomy/Tests/Views/TaxonomyTestBase.php
Line 70: Unused local variable $type
Line 94: Unused local variable $instance
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | drupal_core-remove-unused-variable-2081131-4.patch | 969 bytes | deneo2 |
| #2 | drupal-core-remove-unused-local-variable-2081131.patch | 2.96 KB | mrsinguyen |
| #1 | drupal-core-remove-unused-local-variable-2081131.patch | 969 bytes | mrsinguyen |
Comments
Comment #1
mrsinguyen CreditAttribution: mrsinguyen commentedComment #2
mrsinguyen CreditAttribution: mrsinguyen commentedCombine issues:
- #2081129: Remove Unused local variable $key from /core/modules/taxonomy/lib/Drupal/taxonomy/Tests/VocabularyTest.php
- #2081125: Remove Unused local variable $config from /core/modules/taxonomy/taxonomy.install
- #2081123: Remove Unused local variable $delta from /core/modules/taxonomy/taxonomy.module
- #2081121: Remove Unused local variable $matches from /core/modules/taxonomy/taxonomy.pages.inc
Comment #3
areke CreditAttribution: areke commentedThe patch no longer applies.
Comment #4
deneo2 CreditAttribution: deneo2 commentedIt only solve this issue
Comment #5
areke CreditAttribution: areke commentedThe patch looks good. Thank you!
Comment #6
webchickCommitted and pushed to 8.x. Thanks!