[META] Remove menu_get_object()

Updated: Comment #0

Problem/Motivation

menu_get_object() only works with menu_get_item() and the {menu_router} table.
#2089635: Convert non-test non-form page callbacks to routes and controllers moves most of the paths menu_get_object() is called on to routes, and it breaks.
It includes a generic hack.

Proposed resolution

menu_get_object() is used for two main reasons:

1. To determine if it is on the "full" page of an entity, like node/%node, user/%user, taxonomy/term/%taxonomy_term
2. To retrieve the object if its on the "full" page of an entity.

For case #1, it should check the route name, like if (\Drupal::request()->attributes->get(RouteObjectInterface::ROUTE_NAME) == 'node.view')
We can consider adding a helper function for that.

In the case of #2, more knowledge about the route is needed.
For example, for node/%node you would call $node = \Drupal::request()->attributes->get('node');, but for
node/%node/revisions/%node_revision/view you would need to do $vid = \Drupal::request()->attributes->get('node_revision'); $node = entity_revision_load('node', $vid); (at least until we add a node_revision upcaster, if we so choose).

Remaining tasks

Replace all usage of menu_get_object() and remove it

User interface changes

N/A

API changes

menu_get_object() is removed
Possible new function like drupal_current_route_name() if desired

Related Issues