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 bytessmiro
PASSED: [[SimpleTest]]: [MySQL] 58,505 pass(es). View


smiro’s picture

Status:Active» Needs review
new661 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.