Part of #1971384: [META] Convert page callbacks to controllers

For instructions on how to convert a page callback into a controller, see the WSCCI Conversion Guide.

Files: 
CommentFileSizeAuthor
#11 node-1987766-11.patch4.99 KBjibran
PASSED: [[SimpleTest]]: [MySQL] 55,175 pass(es).
[ View ]
#8 node-1987766-8.patch4.85 KBtim.plunkett
PASSED: [[SimpleTest]]: [MySQL] 56,067 pass(es).
[ View ]
#4 interdiff.txt850 bytestim.plunkett
#4 node-1987766-4.patch4.08 KBtim.plunkett
FAILED: [[SimpleTest]]: [MySQL] 55,844 pass(es), 1 fail(s), and 68 exception(s).
[ View ]
#1 node-1987766-1.patch3.25 KBtim.plunkett
FAILED: [[SimpleTest]]: [MySQL] 56,104 pass(es), 150 fail(s), and 167 exception(s).
[ View ]

Comments

Assigned:Unassigned» tim.plunkett
Status:Active» Needs review
Issue tags:+FormInterface
StatusFileSize
new3.25 KB
FAILED: [[SimpleTest]]: [MySQL] 56,104 pass(es), 150 fail(s), and 167 exception(s).
[ View ]

Sometimes it's just too easy.

Status:Needs review» Reviewed & tested by the community

This is beautiful RTBC if green.

Status:Reviewed & tested by the community» Needs work

The last submitted patch, node-1987766-1.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new4.08 KB
FAILED: [[SimpleTest]]: [MySQL] 55,844 pass(es), 1 fail(s), and 68 exception(s).
[ View ]
new850 bytes

/me shakes fist at EntityBCDecorator.

Status:Needs review» Reviewed & tested by the community
Issue tags:-WSCCI-conversion

This is beautiful RTBC if green.

hehe

Issue tags:+WSCCI-conversion

:/

Status:Reviewed & tested by the community» Needs work

The last submitted patch, node-1987766-4.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new4.85 KB
PASSED: [[SimpleTest]]: [MySQL] 56,067 pass(es).
[ View ]

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.

Status:Needs review» Reviewed & tested by the community

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

Status:Reviewed & tested by the community» Needs work
Issue tags:+Needs reroll

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

Status:Needs work» Reviewed & tested by the community
Issue tags:-Needs reroll
StatusFileSize
new4.99 KB
PASSED: [[SimpleTest]]: [MySQL] 55,175 pass(es).
[ View ]

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.

Status:Reviewed & tested by the community» Fixed

Committed 2581366 and pushed to 8.x. Thanks!

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