From 57590fef864c524ccf0698d01e868a49dc6b16db Mon Sep 17 00:00:00 2001
From: Uwe Steinmann <steinm@debian.org>
Date: Tue, 30 Apr 2013 18:14:05 +0200
Subject: [PATCH] show review/approval tab only if version has reviewer or
 approver

---
 views/bootstrap/class.ViewDocument.php | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/views/bootstrap/class.ViewDocument.php b/views/bootstrap/class.ViewDocument.php
index e1057ad9a..57fd819c9 100644
--- a/views/bootstrap/class.ViewDocument.php
+++ b/views/bootstrap/class.ViewDocument.php
@@ -121,6 +121,11 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style {
 		if($needwkflaction) {
 			$this->infoMsg(getMLText('needs_workflow_action'));
 		}
+
+		$status = $latestContent->getStatus();
+		$reviewStatus = $latestContent->getReviewStatus();
+		$approvalStatus = $latestContent->getApprovalStatus();
+
 ?>
     <ul class="nav nav-tabs" id="docinfotab">
 		  <li class="active"><a data-target="#docinfo" data-toggle="tab"><?php printMLText('document_infos'); ?> / <?php printMLText('current_version'); ?></a></li>
@@ -129,9 +134,12 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style {
 <?php
 			}
 			if($workflowmode == 'traditional') {
+				if((is_array($reviewStatus) && count($reviewStatus)>0) ||
+					(is_array($approvalStatus) && count($approvalStatus)>0)) {
 ?>
 		  <li><a data-target="#revapp" data-toggle="tab"><?php echo getMLText('reviewers')."/".getMLText('approvers'); ?></a></li>
 <?php
+				}
 			} else {
 				if($workflow) {
 ?>
@@ -274,10 +282,6 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style {
 			exit;
 		}
 
-		$status = $latestContent->getStatus();
-		$reviewStatus = $latestContent->getReviewStatus();
-		$approvalStatus = $latestContent->getApprovalStatus();
-
 		// verify if file exists
 		$file_exists=file_exists($dms->contentDir . $latestContent->getPath());
 
@@ -443,6 +447,8 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style {
 		</div>
 <?php
 		if($workflowmode == 'traditional') {
+			if((is_array($reviewStatus) && count($reviewStatus)>0) ||
+				(is_array($approvalStatus) && count($approvalStatus)>0)) {
 ?>
 		  <div class="tab-pane" id="revapp">
 <?php
@@ -583,6 +589,7 @@ class SeedDMS_View_ViewDocument extends SeedDMS_Bootstrap_Style {
 ?>
 		  </div>
 <?php
+		}
 		} else {
 			if($workflow) {
 ?>