Convert node_page_edit() to a new style controller

Sometimes it's just too easy.

This is beautiful RTBC if green.

/me shakes fist at EntityBCDecorator.

This is beautiful RTBC if green.

hehe

:/

Borrowing berdir's fix from #1947880-17: Replace node_access() by $entity->access().

diff --git a/core/modules/node/lib/Drupal/node/NodeAccessController.php b/core/modules/node/lib/Drupal/node/NodeAccessController.php
index ec68b7e..c636ebb 100644
--- a/core/modules/node/lib/Drupal/node/NodeAccessController.php
+++ b/core/modules/node/lib/Drupal/node/NodeAccessController.php
@@ -113,7 +113,7 @@ protected function accessGrants(EntityInterface $node, $operation, $langcode = L
     $query->condition('grant_' . $operation, 1, '>=');
     // Check for grants for this node and the correct langcode.
     $nids = db_and()
-      ->condition('nid', $node->nid)
+      ->condition('nid', $node->id())
       ->condition('langcode', $langcode);
     // If the node is published, also take the default grant into account. The
     // default is saved with a node ID of 0.

Its green. Edit page works fine. It is ready now.

Needs a reroll

curl https://drupal.org/files/node-1987766-8.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4963  100  4963    0     0   4835      0  0:00:01  0:00:01 --:--:--  5811
error: core/modules/node/node.routing.yml: already exists in index

Reroll
Conflict

++<<<<<<< HEAD
 +node_multiple_delete_confirm:
 +  pattern: '/admin/content/node/delete'
 +  defaults:
 +    _form: '\Drupal\node\Form\DeleteMultiple'
 +  requirements:
 +    _permission: 'administer nodes'
++=======
+ node_page_edit:
+   pattern: '/node/{node}/edit'
+   defaults:
+     _entity_form: 'node.edit'
+   requirements:
+     _entity_access: 'node.update'
++>>>>>>> 8

Resolved

+node_multiple_delete_confirm:
 +  pattern: '/admin/content/node/delete'
 +  defaults:
 +    _form: '\Drupal\node\Form\DeleteMultiple'
 +  requirements:
 +    _permission: 'administer nodes'
+ node_page_edit:
+   pattern: '/node/{node}/edit'
+   defaults:
+     _entity_form: 'node.edit'
+   requirements:
+     _entity_access: 'node.update'

Back to RTBC as no change in code.

Committed 2581366 and pushed to 8.x. Thanks!