From 4943ce7bbe7fe7bd7e23d9f802716b6886cb0113 Mon Sep 17 00:00:00 2001
From: holan20180123 <56391770@qq.com>
Date: Tue, 17 Nov 2020 15:25:16 +0800
Subject: [PATCH] * Finish task #8387.

---
 module/bug/model.php                    | 2 +-
 module/bug/view/view.html.php           | 2 +-
 module/build/control.php                | 2 +-
 module/common/model.php                 | 6 +++---
 module/doc/model.php                    | 2 +-
 module/doc/view/browse.html.php         | 6 +++---
 module/doc/view/browsebygrid.html.php   | 4 ++--
 module/doc/view/view.html.php           | 2 +-
 module/my/view/bug.html.php             | 2 +-
 module/my/view/story.html.php           | 2 +-
 module/my/view/task.html.php            | 2 +-
 module/my/view/testcase.html.php        | 2 +-
 module/productplan/control.php          | 2 +-
 module/productplan/view/browse.html.php | 2 +-
 module/project/view/bug.html.php        | 2 +-
 module/project/view/build.html.php      | 2 +-
 module/project/view/treestory.html.php  | 2 +-
 module/project/view/treetask.html.php   | 2 +-
 module/release/view/browse.html.php     | 2 +-
 module/release/view/view.html.php       | 2 +-
 module/story/model.php                  | 2 +-
 module/story/view/view.html.php         | 2 +-
 module/task/view/view.html.php          | 2 +-
 module/testcase/model.php               | 2 +-
 module/testcase/view/view.html.php      | 2 +-
 module/testreport/view/browse.html.php  | 2 +-
 module/testreport/view/view.html.php    | 2 +-
 module/testsuite/control.php            | 2 +-
 module/testtask/control.php             | 4 ++--
 module/testtask/view/view.html.php      | 2 +-
 30 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/module/bug/model.php b/module/bug/model.php
index 887dd0ef67..847b1eac10 100644
--- a/module/bug/model.php
+++ b/module/bug/model.php
@@ -2508,7 +2508,7 @@ class bugModel extends model
     public function printCell($col, $bug, $users, $builds, $branches, $modulePairs, $projects = array(), $plans = array(), $stories = array(), $tasks = array(), $mode = 'datatable')
     {
         /* Check the product is closed. */
-        $canBeChanged = common::checkObjectChangeAllowed('bug', $bug);
+        $canBeChanged = common::canBeChanged('bug', $bug);
 
         $canBatchEdit         = ($canBeChanged and common::hasPriv('bug', 'batchEdit'));
         $canBatchConfirm      = ($canBeChanged and common::hasPriv('bug', 'batchConfirm'));
diff --git a/module/bug/view/view.html.php b/module/bug/view/view.html.php
index 540516b9cb..d17743d7d6 100644
--- a/module/bug/view/view.html.php
+++ b/module/bug/view/view.html.php
@@ -58,7 +58,7 @@
         </div>
       </div>
       <?php echo $this->fetch('file', 'printFiles', array('files' => $bug->files, 'fieldset' => 'true', 'object' => $bug));?>
-      <?php if(common::checkObjectChangeAllowed('bug', $bug)) $actionFormLink = $this->createLink('action', 'comment', "objectType=bug&objectID=$bug->id");?>
+      <?php if(common::canBeChanged('bug', $bug)) $actionFormLink = $this->createLink('action', 'comment', "objectType=bug&objectID=$bug->id");?>
     </div>
     <?php $this->printExtendFields($bug, 'div', "position=left&inForm=0&inCell=1");?>
     <div class='cell'><?php include '../../common/view/action.html.php';?></div>
diff --git a/module/build/control.php b/module/build/control.php
index 329160c247..eb8f4f6a2a 100644
--- a/module/build/control.php
+++ b/module/build/control.php
@@ -213,7 +213,7 @@ class build extends control
         if(!$build) die(js::error($this->lang->notFound) . js::locate('back'));
 
         /* Determines whether an object is editable. */
-        $canBeChanged = common::checkObjectChangeAllowed('build', $build);
+        $canBeChanged = common::canBeChanged('build', $build);
 
         $product = $this->loadModel('product')->getById($build->product);
         if($product->type != 'normal') $this->lang->product->branch = sprintf($this->lang->product->branch, $this->lang->product->branchName[$product->type]);
diff --git a/module/common/model.php b/module/common/model.php
index aed653968a..d26cc7001f 100644
--- a/module/common/model.php
+++ b/module/common/model.php
@@ -1490,7 +1490,7 @@ EOD;
         global $app, $lang;
 
         /* Check the parent object is closed. */
-        if(strpos('close|batchClose', $method) === false and !commonModel::checkObjectChangeAllowed($module, $object)) return false;
+        if(strpos('close|batchClose', $method) === false and !commonModel::canBeChanged($module, $object)) return false;
 
         /* Check is the super admin or not. */
         if(!empty($app->user->admin) || strpos($app->company->admins, ",{$app->user->account},") !== false) return true;
@@ -1842,7 +1842,7 @@ EOD;
     }
 
     /**
-     * Check the parent object is closed.
+     * Check the object can be changed.
      *
      * @param  string $module
      * @param  object $object
@@ -1850,7 +1850,7 @@ EOD;
      * @access public
      * @return bool
      */
-    public static function checkObjectChangeAllowed($module, $object = null)
+    public static function canBeChanged($module, $object = null)
     {
         global $app, $config;
 
diff --git a/module/doc/model.php b/module/doc/model.php
index 69c52605e0..c334b329f7 100644
--- a/module/doc/model.php
+++ b/module/doc/model.php
@@ -111,7 +111,7 @@ class docModel extends model
             $docLib          = new stdClass();
             $docLib->product = $productID;
             $docLib->project = $projectID;
-            $canBeChanged    = common::checkObjectChangeAllowed('doc', $docLib);
+            $canBeChanged    = common::canBeChanged('doc', $docLib);
 
             $lib      = isset($lib) ? $lib : new stdClass();
             $actions  = $this->setFastMenu($fastLib);
diff --git a/module/doc/view/browse.html.php b/module/doc/view/browse.html.php
index 4c0978103c..77a774f54f 100644
--- a/module/doc/view/browse.html.php
+++ b/module/doc/view/browse.html.php
@@ -42,7 +42,7 @@ var browseType = '<?php echo $browseType;?>';
             <?php echo html::a('javascript:setBrowseType("bygrid")', "<i class='icon icon-cards-view'></i>", '', "title='{$lang->doc->browseTypeList['grid']}' class='btn btn-icon'");?>
             <?php echo html::a('javascript:setBrowseType("bylist")', "<i class='icon icon-bars'></i>", '', "title='{$lang->doc->browseTypeList['list']}' class='btn btn-icon text-primary'");?>
           </div>
-          <?php if($libID and common::checkObjectChangeAllowed('doc', $currentLib)):?>
+          <?php if($libID and common::canBeChanged('doc', $currentLib)):?>
           <div class="dropdown">
             <button class="btn" type="button" data-toggle="dropdown"><i class='icon-cog'></i> <span class="caret"></span></button>
             <ul class='dropdown-menu'>
@@ -74,7 +74,7 @@ var browseType = '<?php echo $browseType;?>';
         <p>
           <?php if($libID):?>
           <span class="text-muted"><?php echo $lang->doc->noDoc;?></span>
-          <?php if(common::hasPriv('doc', 'create') and common::checkObjectChangeAllowed('doc', $currentLib)):?>
+          <?php if(common::hasPriv('doc', 'create') and common::canBeChanged('doc', $currentLib)):?>
           <?php echo html::a($this->createLink('doc', 'create', "libID={$libID}&moduleID=$moduleID"), "<i class='icon icon-plus'></i> " . $lang->doc->create, '', "class='btn btn-info'");?>
           <?php endif;?>
           <?php elseif($browseType == 'byediteddate'):?>
@@ -164,7 +164,7 @@ var browseType = '<?php echo $browseType;?>';
               <td class="c-datetime"><?php echo formatTime($doc->addedDate, 'y-m-d');?></td>
               <td class="c-datetime"><?php echo formatTime($doc->editedDate, 'y-m-d');?></td>
               <td class="c-actions">
-                <?php if(common::checkObjectChangeAllowed('doc', $doc)):?>
+                <?php if(common::canBeChanged('doc', $doc)):?>
                 <?php if(common::hasPriv('doc', 'collect')):?>
                 <a data-url="<?php echo $this->createLink('doc', 'collect', "objectID=$doc->id&objectType=doc");?>" title="<?php echo $collectTitle;?>" class='btn btn-link ajaxCollect'><i class='icon <?php echo $star;?>'></i></a>
                 <?php endif;?>
diff --git a/module/doc/view/browsebygrid.html.php b/module/doc/view/browsebygrid.html.php
index 698c907977..209a16934a 100644
--- a/module/doc/view/browsebygrid.html.php
+++ b/module/doc/view/browsebygrid.html.php
@@ -16,7 +16,7 @@
             <?php echo html::a('javascript:setBrowseType("bygrid")', "<i class='icon icon-cards-view'></i>", '', "title='{$lang->doc->browseTypeList['grid']}' class='btn btn-icon text-primary'");?>
             <?php echo html::a('javascript:setBrowseType("bylist")', "<i class='icon icon-bars'></i>", '', "title='{$lang->doc->browseTypeList['list']}' class='btn btn-icon'");?>
           </div>
-          <?php if($libID and common::checkObjectChangeAllowed('doc', $currentLib)):?>
+          <?php if($libID and common::canBeChanged('doc', $currentLib)):?>
           <div class="dropdown">
             <button class="btn" type="button" data-toggle="dropdown"><i class='icon-cog'></i> <span class="caret"></span></button>
             <ul class='dropdown-menu'>
@@ -133,7 +133,7 @@
             <div class="text-primary file-info"><?php echo zget($users, $doc->addedBy);?></div>
           </a>
           <div class="actions">
-            <?php if(common::checkObjectChangeAllowed('doc', $doc)):?>
+            <?php if(common::canBeChanged('doc', $doc)):?>
             <?php if(common::hasPriv('doc', 'collect')):?>
             <a data-url="<?php echo $this->createLink('doc', 'collect', "objectID={$doc->id}&objectType=doc");?>" title="<?php echo $collectTitle;?>" class='btn btn-link ajaxCollect'><i class='icon <?php echo $star;?>'></i></a>
             <?php endif;?>
diff --git a/module/doc/view/view.html.php b/module/doc/view/view.html.php
index 91ccc77061..48765c0b54 100644
--- a/module/doc/view/view.html.php
+++ b/module/doc/view/view.html.php
@@ -214,7 +214,7 @@ js::set('docID', $doc->id);
       </details>
     </div>
     <div class='cell'>
-      <?php if(common::checkObjectChangeAllowed('doc', $doc)) $actionFormLink = $this->createLink('action', 'comment', "objectType=doc&objectID=$doc->id");?>
+      <?php if(common::canBeChanged('doc', $doc)) $actionFormLink = $this->createLink('action', 'comment', "objectType=doc&objectID=$doc->id");?>
       <?php include '../../common/view/action.html.php';?>
     </div>
   </div>
diff --git a/module/my/view/bug.html.php b/module/my/view/bug.html.php
index beb4899cd3..a05cc1336d 100644
--- a/module/my/view/bug.html.php
+++ b/module/my/view/bug.html.php
@@ -73,7 +73,7 @@
       ?>
       <tbody>
         <?php foreach($bugs as $bug):?>
-        <?php $canBeChanged = common::checkObjectChangeAllowed('bug', $bug);?>
+        <?php $canBeChanged = common::canBeChanged('bug', $bug);?>
         <tr>
           <td class="c-id">
             <?php if($canBatchAction):?>
diff --git a/module/my/view/story.html.php b/module/my/view/story.html.php
index 860cc550a1..86bdf366f3 100644
--- a/module/my/view/story.html.php
+++ b/module/my/view/story.html.php
@@ -64,7 +64,7 @@
         <?php foreach($stories as $story):?>
         <?php
         $storyLink    = $this->createLink('story', 'view', "id=$story->id");
-        $canBeChanged = common::checkObjectChangeAllowed('story', $story);
+        $canBeChanged = common::canBeChanged('story', $story);
         ?>
         <tr>
           <td class="c-id">
diff --git a/module/my/view/task.html.php b/module/my/view/task.html.php
index 3d472fb57a..1460409a22 100644
--- a/module/my/view/task.html.php
+++ b/module/my/view/task.html.php
@@ -61,7 +61,7 @@
       </thead>
       <tbody>
         <?php foreach($tasks as $task):?>
-        <?php $canBeChanged = common::checkObjectChangeAllowed('task', $task);?>
+        <?php $canBeChanged = common::canBeChanged('task', $task);?>
         <tr>
           <td class="c-id">
             <?php if($canBatchEdit or $canBatchClose):?>
diff --git a/module/my/view/testcase.html.php b/module/my/view/testcase.html.php
index 0b84fc312f..736899b7eb 100644
--- a/module/my/view/testcase.html.php
+++ b/module/my/view/testcase.html.php
@@ -69,7 +69,7 @@
         <?php
         $caseID       = $type == 'assigntome' ? $case->case : $case->id;
         $runID        = $type == 'assigntome' ? $case->id   : 0;
-        $canBeChanged = common::checkObjectChangeAllowed('testcase', $case);
+        $canBeChanged = common::canBeChanged('testcase', $case);
         ?>
         <tr>
           <td class="c-id">
diff --git a/module/productplan/control.php b/module/productplan/control.php
index c85fb01f13..55482e9721 100644
--- a/module/productplan/control.php
+++ b/module/productplan/control.php
@@ -250,7 +250,7 @@ class productplan extends control
         $this->session->set('bugList', $this->app->getURI(true) . '&type=' . 'bug');
 
         /* Determines whether an object is editable. */
-        $canBeChanged = common::checkObjectChangeAllowed('plan', $plan);
+        $canBeChanged = common::canBeChanged('plan', $plan);
 
         /* Load pager. */
         $this->app->loadClass('pager', $static = true);
diff --git a/module/productplan/view/browse.html.php b/module/productplan/view/browse.html.php
index 68e197cac0..7c732cfa8a 100644
--- a/module/productplan/view/browse.html.php
+++ b/module/productplan/view/browse.html.php
@@ -75,7 +75,7 @@
       <?php $this->loadModel('file');?>
       <?php foreach($plans as $plan):?>
       <?php
-      $canBeChanged = common::checkObjectChangeAllowed('plan', $plan);
+      $canBeChanged = common::canBeChanged('plan', $plan);
       $attribute    = $canBeChanged ? '' : 'disabled';
 
       $plan = $this->file->replaceImgURL($plan, 'desc');
diff --git a/module/project/view/bug.html.php b/module/project/view/bug.html.php
index dc3bf5a839..c94559635d 100644
--- a/module/project/view/bug.html.php
+++ b/module/project/view/bug.html.php
@@ -75,7 +75,7 @@
       <tbody>
       <?php foreach($bugs as $bug):?>
       <?php
-      $canBeChanged = common::checkObjectChangeAllowed('bug', $bug);
+      $canBeChanged = common::canBeChanged('bug', $bug);
       $arrtibute    = $canBeChanged ? '' : 'disabled';
       ?>
       <tr>
diff --git a/module/project/view/build.html.php b/module/project/view/build.html.php
index 047249dd44..35c0ea8330 100644
--- a/module/project/view/build.html.php
+++ b/module/project/view/build.html.php
@@ -74,7 +74,7 @@
           <td class="c-user em"><?php echo zget($users, $build->builder);?></td>
           <td class="c-actions">
             <?php
-            if(common::hasPriv('build', 'linkstory') and common::hasPriv('build', 'view') and common::checkObjectChangeAllowed('build', $build))
+            if(common::hasPriv('build', 'linkstory') and common::hasPriv('build', 'view') and common::canBeChanged('build', $build))
             {
                 echo html::a($this->createLink('build', 'view', "buildID=$build->id&type=story&link=true"), "<i class='icon icon-link'></i>", '', "class='btn' title='{$lang->build->linkStory}'");
             }
diff --git a/module/project/view/treestory.html.php b/module/project/view/treestory.html.php
index 436360f244..364650ce38 100644
--- a/module/project/view/treestory.html.php
+++ b/module/project/view/treestory.html.php
@@ -341,7 +341,7 @@
     </table>
   </div>
 </details>
-<?php if(common::checkObjectChangeAllowed('story', $story)) $actionFormLink = $this->createLink('action', 'comment', "objectType=story&objectID=$story->id");?>
+<?php if(common::canBeChanged('story', $story)) $actionFormLink = $this->createLink('action', 'comment', "objectType=story&objectID=$story->id");?>
 <?php include '../../common/view/action.html.php';?>
 <script>
 <?php if(isset($pageJS)) echo $pageJS;?>
diff --git a/module/project/view/treetask.html.php b/module/project/view/treetask.html.php
index 33433ca643..50443d6ed9 100644
--- a/module/project/view/treetask.html.php
+++ b/module/project/view/treetask.html.php
@@ -79,7 +79,7 @@
 <?php endif;?>
 <?php endif;?>
 <?php echo $this->fetch('file', 'printFiles', array('files' => $task->files, 'fieldset' => 'true', 'object' => $task));?>
-<?php if(common::checkObjectChangeAllowed('task', $task)) $actionFormLink = $this->createLink('action', 'comment', "objectType=task&objectID=$task->id");?>
+<?php if(common::canBeChanged('task', $task)) $actionFormLink = $this->createLink('action', 'comment', "objectType=task&objectID=$task->id");?>
 <?php include '../../common/view/action.html.php';?>
 <script>
 <?php if(isset($pageJS)) echo $pageJS;?>
diff --git a/module/release/view/browse.html.php b/module/release/view/browse.html.php
index 48ddcbc5fd..e2ec9a29d4 100644
--- a/module/release/view/browse.html.php
+++ b/module/release/view/browse.html.php
@@ -58,7 +58,7 @@
     </thead>
     <tbody>
       <?php foreach($releases as $release):?>
-      <?php $canBeChanged = common::checkObjectChangeAllowed('release', $release);?>
+      <?php $canBeChanged = common::canBeChanged('release', $release);?>
       <tr>
         <td><?php echo html::a(helper::createLink('release', 'view', "releaseID=$release->id"), sprintf('%03d', $release->id));?></td>
         <td>
diff --git a/module/release/view/view.html.php b/module/release/view/view.html.php
index 074249bf1d..d2f193d38f 100644
--- a/module/release/view/view.html.php
+++ b/module/release/view/view.html.php
@@ -31,7 +31,7 @@
   </div>
   <div class='btn-toolbar pull-right'>
     <?php
-    $canBeChanged = common::checkObjectChangeAllowed('release', $release);
+    $canBeChanged = common::canBeChanged('release', $release);
 
     if(!$release->deleted and $canBeChanged)
     {
diff --git a/module/story/model.php b/module/story/model.php
index ca59702fbf..46c0580e70 100644
--- a/module/story/model.php
+++ b/module/story/model.php
@@ -3028,7 +3028,7 @@ class storyModel extends model
     public function printCell($col, $story, $users, $branches, $storyStages, $modulePairs = array(), $storyTasks = array(), $storyBugs = array(), $storyCases = array(), $mode = 'datatable', $storyType = 'story')
     {
         /* Check the product is closed. */
-        $canBeChanged = common::checkObjectChangeAllowed('story', $story);
+        $canBeChanged = common::canBeChanged('story', $story);
 
         $canBatchEdit         = common::hasPriv('story', 'batchEdit');
         $canBatchClose        = common::hasPriv('story', 'batchClose');
diff --git a/module/story/view/view.html.php b/module/story/view/view.html.php
index 19a9a8a205..f406c7b713 100644
--- a/module/story/view/view.html.php
+++ b/module/story/view/view.html.php
@@ -65,7 +65,7 @@
         <div class="detail-content article-content"><?php echo $story->verify;?></div>
       </div>
       <?php echo $this->fetch('file', 'printFiles', array('files' => $story->files, 'fieldset' => 'true', 'object' => $story));?>
-      <?php if(common::checkObjectChangeAllowed('story', $story)) $actionFormLink = $this->createLink('action', 'comment', "objectType=story&objectID=$story->id");?>
+      <?php if(common::canBeChanged('story', $story)) $actionFormLink = $this->createLink('action', 'comment', "objectType=story&objectID=$story->id");?>
       <?php if(!empty($story->children)):?>
       <div class='detail'>
         <div class='detail-title'><?php echo $this->lang->story->children;?></div>
diff --git a/module/task/view/view.html.php b/module/task/view/view.html.php
index 330d9f153d..68ac744cd3 100644
--- a/module/task/view/view.html.php
+++ b/module/task/view/view.html.php
@@ -143,7 +143,7 @@
       </div>
       <?php endif;?>
       <?php echo $this->fetch('file', 'printFiles', array('files' => $task->files, 'fieldset' => 'true', 'object' => $task));?>
-      <?php if(common::checkObjectChangeAllowed('task', $task)) $actionFormLink = $this->createLink('action', 'comment', "objectType=task&objectID=$task->id");?>
+      <?php if(common::canBeChanged('task', $task)) $actionFormLink = $this->createLink('action', 'comment', "objectType=task&objectID=$task->id");?>
     </div>
     <?php $this->printExtendFields($task, 'div', "position=left&inForm=0&inCell=1");?>
     <div class="cell"><?php include '../../common/view/action.html.php';?></div>
diff --git a/module/testcase/model.php b/module/testcase/model.php
index cd9204a889..6512797a05 100644
--- a/module/testcase/model.php
+++ b/module/testcase/model.php
@@ -1411,7 +1411,7 @@ class testcaseModel extends model
     public function printCell($col, $case, $users, $branches, $modulePairs = array(), $browseType = '', $mode = 'datatable')
     {
         /* Check the product is closed. */
-        $canBeChanged = common::checkObjectChangeAllowed('case', $case);
+        $canBeChanged = common::canBeChanged('case', $case);
 
         $canBatchRun                = common::hasPriv('testtask', 'batchRun');
         $canBatchEdit               = common::hasPriv('testcase', 'batchEdit');
diff --git a/module/testcase/view/view.html.php b/module/testcase/view/view.html.php
index 583a82d508..05c268411a 100644
--- a/module/testcase/view/view.html.php
+++ b/module/testcase/view/view.html.php
@@ -50,7 +50,7 @@
   <?php if(!isonlybody()):?>
   <div class='btn-toolbar pull-right'>
     <button type='button' class='btn btn-secondary fullscreen-btn' title='<?php echo $lang->retrack;?>'><i class='icon icon-fullscreen'></i><?php echo ' ' . $lang->retrack;?></button>
-    <?php if(common::checkObjectChangeAllowed('testcase', $case)) common::printLink('testcase', 'create', "productID={$case->product}&branch={$case->branch}&moduleID={$case->module}", "<i class='icon icon-plus'></i>" . $lang->testcase->create, '', "class='btn btn-primary'"); ?>
+    <?php if(common::canBeChanged('testcase', $case)) common::printLink('testcase', 'create', "productID={$case->product}&branch={$case->branch}&moduleID={$case->module}", "<i class='icon icon-plus'></i>" . $lang->testcase->create, '', "class='btn btn-primary'"); ?>
   </div>
   <?php endif;?>
 </div>
diff --git a/module/testreport/view/browse.html.php b/module/testreport/view/browse.html.php
index 0fffa7fe23..3c7517e323 100644
--- a/module/testreport/view/browse.html.php
+++ b/module/testreport/view/browse.html.php
@@ -64,7 +64,7 @@
         <td class='text-left' title='<?php echo $taskName?>'><?php echo $taskName;?></td>
         <td class='c-actions'>
           <?php
-          if(common::checkObjectChangeAllowed('report', $report))
+          if(common::canBeChanged('report', $report))
           {
               common::printIcon('testreport', 'edit', "id=$report->id", '', 'list');
               common::printIcon('testreport', 'delete', "id=$report->id", '', 'list', 'trash', 'hiddenwin');
diff --git a/module/testreport/view/view.html.php b/module/testreport/view/view.html.php
index 4f4a1f83fa..363066263b 100644
--- a/module/testreport/view/view.html.php
+++ b/module/testreport/view/view.html.php
@@ -112,7 +112,7 @@
     <?php if(!$report->deleted):?>
     <div class='divider'></div>
     <?php
-    if(common::checkObjectChangeAllowed('report', $report))
+    if(common::canBeChanged('report', $report))
     {
         if(common::hasPriv('testreport', 'create')) echo html::a(inLink('create', "objectID=$report->objectID&objectType=$report->objectType"),  "<i class='icon-refresh'></i>", '', "class='btn' title='{$lang->testreport->recreate}'");
         common::printIcon('testreport', 'edit', "reportID=$report->id", '', 'button');
diff --git a/module/testsuite/control.php b/module/testsuite/control.php
index 8b92010fe2..c902087647 100644
--- a/module/testsuite/control.php
+++ b/module/testsuite/control.php
@@ -161,7 +161,7 @@ class testsuite extends control
         $this->view->pager        = $pager;
         $this->view->modules      = $this->loadModel('tree')->getOptionMenu($suite->product, 'case');
         $this->view->branches     = $this->loadModel('branch')->getPairs($suite->product, 'noempty');
-        $this->view->canBeChanged = common::checkObjectChangeAllowed('testsuite', $suite);
+        $this->view->canBeChanged = common::canBeChanged('testsuite', $suite);
 
         $this->display();
     }
diff --git a/module/testtask/control.php b/module/testtask/control.php
index 2a4010a113..f96bed702a 100644
--- a/module/testtask/control.php
+++ b/module/testtask/control.php
@@ -402,7 +402,7 @@ class testtask extends control
         setcookie('preTaskID', $taskID, $this->config->cookieLife, $this->config->webRoot, '', false, true);
 
         /* Determines whether an object is editable. */
-        $canBeChanged = common::checkObjectChangeAllowed('testtask', $task);
+        $canBeChanged = common::canBeChanged('testtask', $task);
 
         if($this->cookie->preTaskID != $taskID)
         {
@@ -553,7 +553,7 @@ class testtask extends control
         $this->testtask->setMenu($this->products, $productID, $task->branch, $taskID);
 
         /* Determines whether an object is editable. */
-        $canBeChanged = common::checkObjectChangeAllowed('testtask', $task);
+        $canBeChanged = common::canBeChanged('testtask', $task);
 
         $runs = $this->testtask->getRuns($taskID, 0, $groupBy);
         $this->loadModel('common')->saveQueryCondition($this->dao->get(), 'testcase', false);
diff --git a/module/testtask/view/view.html.php b/module/testtask/view/view.html.php
index 5de9475449..25f6aea05d 100644
--- a/module/testtask/view/view.html.php
+++ b/module/testtask/view/view.html.php
@@ -39,7 +39,7 @@
         <div class="detail-content article-content"><?php echo $task->report;?></div>
       </div>
       <?php endif;?>
-      <?php if(common::checkObjectChangeAllowed('testtask', $task)) $actionFormLink = $this->createLink('action', 'comment', "objectType=testtask&objectID=$task->id");?>
+      <?php if(common::canBeChanged('testtask', $task)) $actionFormLink = $this->createLink('action', 'comment', "objectType=testtask&objectID=$task->id");?>
     </div>
     <?php $this->printExtendFields($task, 'div', "position=left&inForm=0&inCell=1");?>
     <div class='cell'><?php include '../../common/view/action.html.php';?></div>