Part of #1998638: Replace almost all remaining superglobals ($_GET, $_POST, etc.) with Symfony Request object

Files that need converting are:

  • core/modules/menu/menu.module
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chertzog’s picture

Assigned: Unassigned » chertzog
Status: Active » Needs review
FileSize
615 bytes
kim.pepper’s picture

Looks good. RTBC from me.

Crell’s picture

Status: Needs review » Reviewed & tested by the community
Crell’s picture

Issue tags: +Quick fix

Tagging.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed b90377b and pushed to 8.x. Thanks!

tim.plunkett’s picture

For future reference, PHP's count() function is only useful if you want to know the specific number of items of a thing you know is countable.

Because if you call count(NULL), it will return 1. Yes. I'm serious, it's awful.

This whole thing was already fixed in #1984702: Convert menu.module's page callbacks to Controllers, I'll just reroll that.

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