diff --git a/module/branch/css/manage.css b/module/branch/css/manage.css index 25f9ddee45..4d9f958f35 100644 --- a/module/branch/css/manage.css +++ b/module/branch/css/manage.css @@ -1,5 +1,3 @@ -#branches .input-group, #newbranches .input-group {margin-bottom: 5px; width: 100%;} -#branches .input-group input, #newbranches .input-group input {border-right: 0px;} -#branches .sort-handler {cursor: move;} -#branches .input-group.drag-shadow {z-index: 5;} -#branches .input-group.drag-row {opacity: 0;} +.c-desc {width: 500px;} +.c-order {width: 70px;} +.c-check {width: 40px} diff --git a/module/branch/model.php b/module/branch/model.php index dcb3032c88..5fbe820de0 100644 --- a/module/branch/model.php +++ b/module/branch/model.php @@ -35,12 +35,12 @@ class branchModel extends model * * @param int $productID * @param string $browseType - * @param int $orderBy - * @param int $pager + * @param string $orderBy + * @param object $pager * @access public - * @return void + * @return array */ - public function getList($productID, $browseType = 'active', $orderBy, $pager = null) + public function getList($productID, $browseType = 'active', $orderBy = 'order_desc', $pager = null) { $branchList = $this->dao->select('*')->from(TABLE_BRANCH) ->where('deleted')->eq(0) @@ -50,27 +50,22 @@ class branchModel extends model ->page($pager) ->fetchAll('id'); - if($browseType == 'closed') - { - return $branchList; - } - else - { - /* Display the main branch under all and active page. */ - $mainBranch = new stdclass(); - $mainBranch->id = MAIN; - $mainBranch->product = $productID; - $mainBranch->name = $this->lang->branch->main; - $mainBranch->default = 1; - $mainBranch->status = 'active'; - $mainBranch->createdDate = ''; - $mainBranch->closedDate = ''; - $mainBranch->desc = $this->lang->branch->mainBranch; - $mainBranch->order = 0; + if($browseType == 'closed') return $branchList; - $pager->recTotal = $pager->recTotal + 1; - return array($mainBranch) + $branchList; - } + /* Display the main branch under all and active page. */ + $mainBranch = new stdclass(); + $mainBranch->id = BRANCH_MAIN; + $mainBranch->product = $productID; + $mainBranch->name = $this->lang->branch->main; + $mainBranch->default = 1; + $mainBranch->status = 'active'; + $mainBranch->createdDate = ''; + $mainBranch->closedDate = ''; + $mainBranch->desc = $this->lang->branch->mainBranch; + $mainBranch->order = 0; + + $pager->recTotal = $pager->recTotal + 1; + return array($mainBranch) + $branchList; } /** diff --git a/module/branch/view/manage.html.php b/module/branch/view/manage.html.php index c09d70279c..f8e22f2a80 100644 --- a/module/branch/view/manage.html.php +++ b/module/branch/view/manage.html.php @@ -12,20 +12,22 @@ ?> + + + +
@@ -33,9 +35,7 @@

branch->noData;?> - - createLink('branch', 'create', "productID=$productID"), " " . $lang->branch->create, '', "class='btn btn-info'");?> - + createLink('branch', 'create', "productID=$productID"), " " . $lang->branch->create, '', "class='btn btn-info'");?>

@@ -44,26 +44,26 @@ recTotal&recPerPage=$pager->recPerPage&pageID=$pager->pageID"; ?> - - + +
- branch->order;?> + branch->order;?> branch->name);?> - branch->status);?> - branch->createdDate);?> - branch->closedDate);?> - branch->desc;?> + branch->status);?> + branch->createdDate);?> + branch->closedDate);?> + branch->desc;?> actions;?> - + - id => ''), '', $branch->id == MAIN ? "disabled='disabled'" : '');?> + id => ''), '', $branch->id == BRANCH_MAIN ? "disabled='disabled'" : '');?> @@ -74,7 +74,7 @@ desc;?> id == MAIN ? 'disabled' : ''; + $disabled = $branch->id == BRANCH_MAIN ? 'disabled' : ''; common::printIcon('branch', 'edit', "branchID=$branch->id", $branch, 'list', '', '', $disabled); if($branch->status == 'active') { @@ -91,7 +91,7 @@