From 48f425f1806d52106378e2e1c0a017273b070be9 Mon Sep 17 00:00:00 2001
From: chenfeiCF <chenfei@qdphp.com>
Date: Thu, 15 Nov 2018 13:43:28 +0800
Subject: [PATCH]  * Finish task #5128.

---
 module/bug/view/browse.html.php          |  8 +++----
 module/product/view/browse.html.php      |  7 +++---
 module/project/view/grouptask.html.php   | 21 ++++++++++--------
 module/project/view/kanban.html.php      |  4 ++--
 module/project/view/task.html.php        | 21 ++++++++++--------
 module/project/view/tree.html.php        |  4 ++--
 module/testcase/view/caseheader.html.php | 28 ++++++++++++++----------
 7 files changed, 52 insertions(+), 41 deletions(-)

diff --git a/module/bug/view/browse.html.php b/module/bug/view/browse.html.php
index b0a4281607..812b3dcf19 100644
--- a/module/bug/view/browse.html.php
+++ b/module/bug/view/browse.html.php
@@ -122,19 +122,19 @@ $currentBrowseType = isset($lang->bug->mySelects[$browseType]) && in_array($brow
   </div>
   <div class="btn-toolbar pull-right">
     <?php common::printIcon('bug', 'report', "productID=$productID&browseType=$browseType&branchID=$branch&moduleID=$moduleID", '', 'button', 'bar-chart muted');?>
-    <?php if(common::hasPriv('bug', 'export')):?>
     <div class='btn-group'>
       <button type='button' class='btn btn-link dropdown-toggle' data-toggle='dropdown'>
         <i class="icon icon-export muted"></i> <span class="text"> <?php echo $lang->export;?></span> <span class="caret"></span></button>
       </button>
       <ul class='dropdown-menu' id='exportActionMenu'>
         <?php
-        $link = $this->createLink('bug', 'export', "productID=$productID&orderBy=$orderBy&browseType=$browseType");
-        echo "<li>" . html::a($link, $lang->bug->export, '', "class='export'") . "</li>";
+        $class = common::hasPriv('bug', 'export') ? "" : "class='disabled'";
+        $misc  = common::hasPriv('bug', 'export') ? "class='export'" : "class='disabled'";
+        $link  = common::hasPriv('bug', 'export') ? $this->createLink('bug', 'export', "productID=$productID&orderBy=$orderBy&browseType=$browseType") : '#';
+        echo "<li $class>" . html::a($link, $lang->bug->export, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
-    <?php endif;?>
     <?php
     common::printLink('bug', 'batchCreate', "productID=$productID&branch=$branch&projectID=0&moduleID=$moduleID", "<i class='icon icon-plus'></i>" . $lang->bug->batchCreate, '', "class='btn btn-secondary'");
     if(commonModel::isTutorialMode())
diff --git a/module/product/view/browse.html.php b/module/product/view/browse.html.php
index f5a4ad6f79..28dbacefa2 100644
--- a/module/product/view/browse.html.php
+++ b/module/product/view/browse.html.php
@@ -96,9 +96,10 @@
       <button class="btn btn-link" data-toggle="dropdown"><i class="icon icon-export muted"></i> <span class="text"><?php echo $lang->export ?></span> <span class="caret"></span></button>
       <ul class="dropdown-menu" id='exportActionMenu'>
         <?php
-        $misc = common::hasPriv('story', 'export') ? "class='export'" : "class=disabled";
-        $link = common::hasPriv('story', 'export') ?  $this->createLink('story', 'export', "productID=$productID&orderBy=$orderBy&projectID=0&browseType=$browseType") : '#';
-        echo "<li>" . html::a($link, $lang->story->export, '', $misc) . "</li>";
+        $class = common::hasPriv('story', 'export') ? '' : "class=disabled";
+        $misc  = common::hasPriv('story', 'export') ? "class='export'" : "class=disabled";
+        $link  = common::hasPriv('story', 'export') ?  $this->createLink('story', 'export', "productID=$productID&orderBy=$orderBy&projectID=0&browseType=$browseType") : '#';
+        echo "<li $class>" . html::a($link, $lang->story->export, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
diff --git a/module/project/view/grouptask.html.php b/module/project/view/grouptask.html.php
index 453c19152e..c5af58f1f6 100644
--- a/module/project/view/grouptask.html.php
+++ b/module/project/view/grouptask.html.php
@@ -46,9 +46,10 @@
       <button class="btn btn-link" data-toggle="dropdown"><i class="icon icon-export muted"></i> <span class="text"><?php echo $lang->export;?></span> <span class="caret"></span></button>
       <ul class="dropdown-menu">
         <?php
-        $misc = common::hasPriv('task', 'export') ? "class='export'" : "class=disabled";
-        $link = common::hasPriv('task', 'export') ? $this->createLink('task', 'export', "project=$projectID&orderBy=$orderBy&type=$browseType") : '#';
-        echo "<li>" . html::a($link, $lang->story->export, '', $misc) . "</li>";
+        $class = common::hasPriv('task', 'export') ? '' : "class=disabled";
+        $misc  = common::hasPriv('task', 'export') ? "class='export'" : "class=disabled";
+        $link  = common::hasPriv('task', 'export') ? $this->createLink('task', 'export', "project=$projectID&orderBy=$orderBy&type=$browseType") : '#';
+        echo "<li $class>" . html::a($link, $lang->story->export, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
@@ -56,13 +57,15 @@
       <button class="btn btn-link" data-toggle="dropdown"><i class="icon icon-import muted"></i> <span class="text"><?php echo $lang->import;?></span> <span class="caret"></span></button>
       <ul class="dropdown-menu">
         <?php
-        $misc = common::hasPriv('project', 'importTask') ? "class='import'" : "class=disabled";
-        $link = common::hasPriv('project', 'importTask') ? $this->createLink('project', 'importTask', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
+        $class = common::hasPriv('project', 'importTask') ? '' : "class=disabled";
+        $misc  = common::hasPriv('project', 'importTask') ? "class='import'" : "class=disabled";
+        $link  = common::hasPriv('project', 'importTask') ? $this->createLink('project', 'importTask', "project=$project->id") : '#';
+        echo "<li $class>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
 
-        $misc = common::hasPriv('project', 'importBug') ? "class='import'" : "class=disabled";
-        $link = common::hasPriv('project', 'importBug') ? $this->createLink('project', 'importBug', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
+        $class = common::hasPriv('project', 'importBug') ? '' : "class=disabled";
+        $misc  = common::hasPriv('project', 'importBug') ? "class='import'" : "class=disabled";
+        $link  = common::hasPriv('project', 'importBug') ? $this->createLink('project', 'importBug', "project=$project->id") : '#';
+        echo "<li $class>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
diff --git a/module/project/view/kanban.html.php b/module/project/view/kanban.html.php
index 8a530afb40..30ee650cce 100644
--- a/module/project/view/kanban.html.php
+++ b/module/project/view/kanban.html.php
@@ -26,11 +26,11 @@
         <?php
         $misc = common::hasPriv('project', 'importTask') ? '' : "class=disabled";
         $link = common::hasPriv('project', 'importTask') ?  $this->createLink('project', 'importTask', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
+        echo "<li $misc>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
 
         $misc = common::hasPriv('project', 'importBug') ? '' : "class=disabled";
         $link = common::hasPriv('project', 'importBug') ?  $this->createLink('project', 'importBug', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
+        echo "<li $misc>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
diff --git a/module/project/view/task.html.php b/module/project/view/task.html.php
index 2dee532eaa..59b0ab256f 100644
--- a/module/project/view/task.html.php
+++ b/module/project/view/task.html.php
@@ -122,9 +122,10 @@ js::set('browseType', $browseType);
       <button class="btn btn-link" data-toggle="dropdown"><i class="icon icon-export muted"></i> <span class="text"><?php echo $lang->export;?></span> <span class="caret"></span></button>
       <ul class="dropdown-menu pull-right" id='exportActionMenu'>
         <?php
-        $misc = common::hasPriv('task', 'export') ? "class='export'" : "class=disabled";
-        $link = common::hasPriv('task', 'export') ? $this->createLink('task', 'export', "project=$projectID&orderBy=$orderBy&type=$browseType") : '#';
-        echo "<li>" . html::a($link, $lang->story->export, '', $misc) . "</li>";
+        $class = common::hasPriv('task', 'export') ? '' : "class=disabled";
+        $misc  = common::hasPriv('task', 'export') ? "class='export'" : "class=disabled";
+        $link  = common::hasPriv('task', 'export') ? $this->createLink('task', 'export', "project=$projectID&orderBy=$orderBy&type=$browseType") : '#';
+        echo "<li $class>" . html::a($link, $lang->story->export, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
@@ -133,13 +134,15 @@ js::set('browseType', $browseType);
       <button class="btn btn-link" data-toggle="dropdown"><i class="icon icon-import muted"></i> <span class="text"><?php echo $lang->import;?></span> <span class="caret"></span></button>
       <ul class="dropdown-menu pull-right" id='importActionMenu'>
         <?php
-        $misc = common::hasPriv('project', 'importTask') ? "class='import'" : "class=disabled";
-        $link = common::hasPriv('project', 'importTask') ? $this->createLink('project', 'importTask', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
+        $class = common::hasPriv('project', 'importTask') ? '' : "class=disabled";
+        $misc  = common::hasPriv('project', 'importTask') ? "class='import'" : "class=disabled";
+        $link  = common::hasPriv('project', 'importTask') ? $this->createLink('project', 'importTask', "project=$project->id") : '#';
+        echo "<li $class>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
 
-        $misc = common::hasPriv('project', 'importBug') ? "class='import'" : "class=disabled";
-        $link = common::hasPriv('project', 'importBug') ? $this->createLink('project', 'importBug', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
+        $class = common::hasPriv('project', 'importBug') ? '' : "class=disabled";
+        $misc  = common::hasPriv('project', 'importBug') ? "class='import'" : "class=disabled";
+        $link  = common::hasPriv('project', 'importBug') ? $this->createLink('project', 'importBug', "project=$project->id") : '#';
+        echo "<li $class>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
diff --git a/module/project/view/tree.html.php b/module/project/view/tree.html.php
index 1b5a6f1a71..c80a212363 100644
--- a/module/project/view/tree.html.php
+++ b/module/project/view/tree.html.php
@@ -38,11 +38,11 @@
         <?php
         $misc = common::hasPriv('project', 'importTask') ? '' : "class=disabled";
         $link = common::hasPriv('project', 'importTask') ?  $this->createLink('project', 'importTask', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
+        echo "<li $misc>" . html::a($link, $lang->project->importTask, '', $misc) . "</li>";
 
         $misc = common::hasPriv('project', 'importBug') ? '' : "class=disabled";
         $link = common::hasPriv('project', 'importBug') ?  $this->createLink('project', 'importBug', "project=$project->id") : '#';
-        echo "<li>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
+        echo "<li $misc>" . html::a($link, $lang->project->importBug, '', $misc) . "</li>";
         ?>
       </ul>
     </div>
diff --git a/module/testcase/view/caseheader.html.php b/module/testcase/view/caseheader.html.php
index 6a24c0ac48..5e7d41e261 100644
--- a/module/testcase/view/caseheader.html.php
+++ b/module/testcase/view/caseheader.html.php
@@ -124,13 +124,15 @@
       </button>
       <ul class='dropdown-menu' id='exportActionMenu'>
       <?php
-      $misc = common::hasPriv('testcase', 'export') ? "class='export'" : "class=disabled";
-      $link = common::hasPriv('testcase', 'export') ?  $this->createLink('testcase', 'export', "productID=$productID&orderBy=$orderBy&taskID=0&browseType=$browseType") : '#';
-      echo "<li>" . html::a($link, $lang->testcase->export, '', $misc) . "</li>";
+      $class = common::hasPriv('testcase', 'export') ? '' : "class=disabled";
+      $misc  = common::hasPriv('testcase', 'export') ? "class='export'" : "class=disabled";
+      $link  = common::hasPriv('testcase', 'export') ?  $this->createLink('testcase', 'export', "productID=$productID&orderBy=$orderBy&taskID=0&browseType=$browseType") : '#';
+      echo "<li $class>" . html::a($link, $lang->testcase->export, '', $misc) . "</li>";
 
-      $misc = common::hasPriv('testcase', 'exportTemplet') ? "class='export'" : "class=disabled";
-      $link = common::hasPriv('testcase', 'exportTemplet') ?  $this->createLink('testcase', 'exportTemplet', "productID=$productID") : '#';
-      echo "<li>" . html::a($link, $lang->testcase->exportTemplet, '', $misc) . "</li>";
+      $class = common::hasPriv('testcase', 'exportTemplet') ? '' : "class=disabled";
+      $misc  = common::hasPriv('testcase', 'exportTemplet') ? "class='export'" : "class=disabled";
+      $link  = common::hasPriv('testcase', 'exportTemplet') ?  $this->createLink('testcase', 'exportTemplet', "productID=$productID") : '#';
+      echo "<li $class>" . html::a($link, $lang->testcase->exportTemplet, '', $misc) . "</li>";
       ?>
       </ul>
     </div>
@@ -138,13 +140,15 @@
       <button type='button' class='btn btn-link dropdown-toggle' data-toggle='dropdown' id='importAction'><i class='icon icon-import muted'></i> <?php echo $lang->import ?><span class='caret'></span></button>
       <ul class='dropdown-menu' id='importActionMenu'>
       <?php
-      $misc = common::hasPriv('testcase', 'import') ? "class='export'" : "class=disabled";
-      $link = common::hasPriv('testcase', 'import') ?  $this->createLink('testcase', 'import', "productID=$productID&branch=$branch") : '#';
-      echo "<li>" . html::a($link, $lang->testcase->importFile, '', $misc) . "</li>";
+      $class = common::hasPriv('testcase', 'import') ? '' : "class=disabled";
+      $misc  = common::hasPriv('testcase', 'import') ? "class='export'" : "class=disabled";
+      $link  = common::hasPriv('testcase', 'import') ?  $this->createLink('testcase', 'import', "productID=$productID&branch=$branch") : '#';
+      echo "<li $class>" . html::a($link, $lang->testcase->importFile, '', $misc) . "</li>";
 
-      $misc = common::hasPriv('testcase', 'importFromLib') ? '' : "class=disabled";
-      $link = common::hasPriv('testcase', 'importFromLib') ?  $this->createLink('testcase', 'importFromLib', "productID=$productID&branch=$branch") : '#';
-      echo "<li>" . html::a($link, $lang->testcase->importFromLib, '', $misc) . "</li>";
+      $class = common::hasPriv('testcase', 'importFromLib') ? '' : "class=disabled";
+      $misc  = common::hasPriv('testcase', 'importFromLib') ? '' : "class=disabled";
+      $link  = common::hasPriv('testcase', 'importFromLib') ?  $this->createLink('testcase', 'importFromLib', "productID=$productID&branch=$branch") : '#';
+      echo "<li $class>" . html::a($link, $lang->testcase->importFromLib, '', $misc) . "</li>";
       ?>
       </ul>
     </div>