Make node aware of the latest_version route.

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Re-roll patch need to 11.x.

Rerolled for 11.x.

This will need test coverage. Why is the one test being removed?

The test that was removed was specifically testing the CM preprocessor which was removed so the test was no longer needed. Instead I've repurposed it to test the node_is_page function.

I was in 2 minds about doing this or 📌 Deprecate node_is_page and move into a service Active first but this might be easier to get in first.

Alternatively if we make it a service I guess CM could decorate it to add the condition to its own route 🤔

Can we update the issue summary to reflect the new scope and findings please.

Hiding patches

@smustgrave what exactly is missing from the IS? It explains exactly what we're doing?

left some comments, still marking as the changes to the test didn't break.

I don't think we should hardcode this. How about we add something to route so we have an API here that other modules could use.

Something like
if ($route_match->getRouteName() == 'entity.node.canonical' || $route_match->getRouteObject()->getOption('node.is_full_page_view')) {

And then in \Drupal\content_moderation\Entity\Routing\EntityModerationRouteProvider::getLatestVersionRoute() do ->setOption($entity_type_id . '.is_full_page_view', TRUE) on the route it returns. This way any entity that has similar needs to node can do things in the same way.

We should also update the docs in template_preprocess_node() as the following is not correct...

  // The 'page' variable is set to TRUE in two occasions:
  // - The view mode is 'full' and we are on the 'node.view' route.
  // - The node is in preview and view mode is either 'full' or 'default'.

We're actually trying to deprecate this variable entirely so I guess this should be postponed 📌 Deprecate $variables['page'] for node.html.twig Needs review

I do like the idea in #35 though!