From 9eefa1934bb212c51978d5c1e12014e89b97c392 Mon Sep 17 00:00:00 2001
From: "chencongzhi520@gmail.com" <azhi@cnezsoft.com>
Date: Mon, 30 Apr 2012 03:33:50 +0000
Subject: [PATCH] * Refactory checkall.

---
 module/group/view/managemember.html.php | 4 ++--
 module/group/view/privbygroup.html.php  | 4 ++--
 module/project/view/linkstory.html.php  | 9 ++++++++-
 module/testtask/view/cases.html.php     | 2 +-
 module/testtask/view/linkcase.html.php  | 2 +-
 5 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/module/group/view/managemember.html.php b/module/group/view/managemember.html.php
index 08e216091d..67c7395ecb 100644
--- a/module/group/view/managemember.html.php
+++ b/module/group/view/managemember.html.php
@@ -15,7 +15,7 @@
   <table align='center' class='table-1 a-left'> 
     <caption><?php echo $group->name . $lang->colon . $lang->group->manageMember;?></caption>
     <tr>
-      <th class='rowhead'><?php echo $lang->group->inside;?><input type='checkbox' onclick='checkall(this, "group");'></th>
+      <th class='rowhead'><?php echo $lang->group->inside;?><?php echo html::checkAll('checkall', $lang->selectAll, 'group');?> </th>
       <td id='group' class='f-14px pv-10px'><?php $i = 1;?>
         <?php foreach($groupUsers as $account => $realname):?>
         <div class='w-p10 f-left'><?php echo '<span>' . html::checkbox('members', array($account => $realname), $account) . '</span>';?></div>
@@ -24,7 +24,7 @@
       </td>
     </tr>
     <tr>
-      <th class='rowhead'><?php echo $lang->group->outside;?><input type='checkbox' onclick='checkall(this, "other");'></th>
+      <th class='rowhead'><?php echo $lang->group->outside;?><?php echo html::checkAll('checkall', $lang->selectAll, 'other');?> </th>
       <td id='other' class='f-14px pv-10px'><?php $i = 1;?>
         <?php foreach($otherUsers as $account => $realname):?>
         <div class='w-p10 f-left'><?php echo '<span>' . html::checkbox('members', array($account => $realname), '') . '</span>';?></div>
diff --git a/module/group/view/privbygroup.html.php b/module/group/view/privbygroup.html.php
index e8a3a1ecb6..306f1ce662 100644
--- a/module/group/view/privbygroup.html.php
+++ b/module/group/view/privbygroup.html.php
@@ -25,7 +25,7 @@
     </tr>  
     <?php foreach($lang->resource as $moduleName => $moduleActions):?>
     <tr class='f-14px <?php echo cycle('even, bg-yellow');?>'>
-      <th class='a-right'><?php echo $this->lang->$moduleName->common;?> <input type='checkbox' onclick='checkall(this, "<?php echo $moduleName;?>")'></td>
+      <th class='a-right'><?php echo $this->lang->$moduleName->common;?><?php echo html::checkAll('checkall', $lang->selectAll, $moduleName)?></td>
       <td id='<?php echo $moduleName;?>' class='pv-10px'>
         <?php $i = 1;?>
         <?php foreach($moduleActions as $action => $actionLabel):?>
@@ -39,7 +39,7 @@
     </tr>
     <?php endforeach;?>
     <tr>
-      <th class='rowhead'><?php echo $lang->selectAll;?><input type='checkbox' onclick='checkall(this);'></th>
+      <th class='rowhead'><?php echo $lang->selectAll;?><?php echo html::checkAll('checkall', $lang->selectAll)?></th>
       <td class='a-center'>
         <?php 
         echo html::submitButton($lang->save);
diff --git a/module/project/view/linkstory.html.php b/module/project/view/linkstory.html.php
index b0409f6846..ac0c695a98 100644
--- a/module/project/view/linkstory.html.php
+++ b/module/project/view/linkstory.html.php
@@ -47,7 +47,14 @@
     <?php $storyCount ++;?>
     <?php endforeach;?>
     </tbody>
-    <tfoot><tr><td colspan='8' class='f-14px'><input type='checkbox' onclick='checkall(this);'><?php print($storyCount? ($lang->selectAll . html::submitButton()) : $lang->project->whyNoStories);?></td></tr></tfoot>
+    <tfoot>
+      <tr>
+        <td colspan='8' class='f-14px'>
+        <?php echo html::checkAll('checkall', $lang->selectAll); ?>
+        <?php print($storyCount? html::submitButton() : $lang->project->whyNoStories);?>
+        </td>
+      </tr>
+    </tfoot>
   </table>
 </form>
 <?php include '../../common/view/footer.html.php';?>
diff --git a/module/testtask/view/cases.html.php b/module/testtask/view/cases.html.php
index c59497df55..7303fb205c 100644
--- a/module/testtask/view/cases.html.php
+++ b/module/testtask/view/cases.html.php
@@ -85,7 +85,7 @@ var moduleID   = '<?php echo $moduleID;?>';
       </table>
       <table class='table-1'>
         <tr>
-          <td><nobr><?php echo "<input type='checkbox' onclick='checkall(this);'> " . $lang->selectAll;?></nobr></td>
+          <td><nobr><?php echo html::checkAll('checkall', $lang->selectAll);?></nobr></td>
           <td colspan='9'>
             <?php
             echo html::select('assignedTo', $users);
diff --git a/module/testtask/view/linkcase.html.php b/module/testtask/view/linkcase.html.php
index 551fae6920..58b9a1b8f5 100644
--- a/module/testtask/view/linkcase.html.php
+++ b/module/testtask/view/linkcase.html.php
@@ -60,7 +60,7 @@
   <tfoot> 
   <tr><td colspan='7' class='a-right'><?php $pager->show();?></td></tr>
   <tr>
-    <td colspan='7' class='a-center'><input type='checkbox' onclick='checkall(this);'><?php echo $lang->selectAll; echo html::submitButton();?></td>
+    <td colspan='7' class='a-center'><?php echo html::checkAll('checkall', $lang->selectAll);?><?php echo html::submitButton();?></td>
   </tr>
   </tfoot>
 </table>