Show subnavigation for read-only views (CFM-293) (#105)

COmanage · Jun 28, 2023 · ded2ee1 · ded2ee1
1 parent 752bd66
commit ded2ee1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/templates/Standard/add-edit-view.php b/app/templates/Standard/add-edit-view.php
@@ -42,8 +42,8 @@
 
 // Include subnavigation structures on add/edit/view pages
 // XXX: if CFM-218 (Make fields.inc configuration only) is accepted, move the contents of fields-nav.inc into fields.inc
-// When subnav exists, include on all Edit views and on Add/View for items with a parent.
-if($vv_action == 'edit' || !empty($vv_bc_parent_obj) || !empty($vv_primary_link_id)) {
+// When subnav exists, include on all Edit/View views and on Add views for items with a parent.
+if($vv_action == 'edit' || $vv_action == 'view' || !empty($vv_bc_parent_obj) || !empty($vv_primary_link_id)) {
   if(file_exists($templatePath . DS . "fields-nav.inc")) {
     include($templatePath . DS . "fields-nav.inc");
   }  

diff --git a/app/templates/element/subnavigation.php b/app/templates/element/subnavigation.php
@@ -207,7 +207,7 @@
             print $this->Html->link(
               __d('controller', 'People', [1]),
               [ 'controller' => 'people',
-                'action' => 'edit',
+                'action' => $curAction == 'edit' ? 'edit' : 'view',
                 $curId
               ],
               ['class' => $linkClass]
@@ -267,7 +267,7 @@
             print $this->Html->link(
               __d('controller', 'Properties', [99]),
               [ 'controller' => 'groups',
-                'action' => 'edit',
+                'action' => $curAction == 'edit' ? 'edit' : 'view',
                 $curId
               ],
               ['class' => $linkClass]