From bee083a8b097e199dffef98b485286f37781f6d3 Mon Sep 17 00:00:00 2001
From: Liam Morland <lkmorlan@uwaterloo.ca>
Date: Fri, 26 Feb 2021 13:38:27 -0500
Subject: [PATCH] ISTWCMS-4229: Refactor
 UwNodeAccessRouteSubscriber::alterRoutes()

---
 src/Routing/UwNodeAccessRouteSubscriber.php | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/Routing/UwNodeAccessRouteSubscriber.php b/src/Routing/UwNodeAccessRouteSubscriber.php
index b74c3eb8..143543ac 100644
--- a/src/Routing/UwNodeAccessRouteSubscriber.php
+++ b/src/Routing/UwNodeAccessRouteSubscriber.php
@@ -15,10 +15,16 @@ class UwNodeAccessRouteSubscriber extends RouteSubscriberBase {
    * {@inheritdoc}
    */
   protected function alterRoutes(RouteCollection $collection) {
-    // Change the route associated with node (/node/{nid}).
-    if ($route = $collection->get('entity.node.canonical')) {
-      $route->setRequirement('_custom_access', 'Drupal\uw_cfg_common\Access\UwNodeAccessCheck::access');
+    $access_route_names = [
+      // Node pages (/node/{nid}).
+      'entity.node.canonical',
+    ];
+    foreach ($access_route_names as $route_name) {
+      if ($route = $collection->get($route_name)) {
+        $route->setRequirement('_custom_access', 'Drupal\uw_cfg_common\Access\UwNodeAccessCheck::access');
+      }
     }
+  }
 
   /**
    * {@inheritdoc}
-- 
GitLab