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

File /core/modules/hal/lib/Drupal/hal/Normalizer/EntityNormalizer.php

Line 93: Unused local variable $links

#1 drupal-core-remove-unused-local-variable-2081197-1.patch661 bytessmira
PASSED: [[SimpleTest]]: [MySQL] 58,505 pass(es). View


smira’s picture

Status: Active » Needs review
661 bytes
PASSED: [[SimpleTest]]: [MySQL] 58,505 pass(es). View

my take on the matter though i would probably opt to mark as "wont fix" instead just to be safe...

lorique’s picture

Status: Needs review » Reviewed & tested by the community

The variable is gone from EntityNormalizer->denormalize() method, and is used nowhere in the method. thus it can and has been safely removed.

Changing status. Thank you for your patch @smiro.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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