diff --git a/module/product/control.php b/module/product/control.php
index dbaf4c6139..50cbf60214 100644
--- a/module/product/control.php
+++ b/module/product/control.php
@@ -731,7 +731,7 @@ class product extends control
      * @access public
      * @return void
      */
-    public function all($programID = 0, $browseType = 'noclosed', $param = 0, $orderBy = 'order_desc', $recTotal = 0, $recPerPage = 15, $pageID = 1)
+    public function all($programID = 0, $browseType = 'noclosed', $orderBy = 'order_desc', $recTotal = 0, $recPerPage = 15, $pageID = 1)
     {
         $this->session->set('program', $programID);
         $this->loadModel('program');
diff --git a/module/product/view/all.html.php b/module/product/view/all.html.php
index e53a147a4a..cc5a666b54 100644
--- a/module/product/view/all.html.php
+++ b/module/product/view/all.html.php
@@ -39,7 +39,7 @@
       <?php $canOrder = common::hasPriv('product', 'updataOrder');?>
       <?php $canBatchEdit = common::hasPriv('product', 'batchEdit');?>
       <table id="productList" class="table has-sort-head table-fixed">
-        <?php $vars = "programID&$programID&browseType=$browseType&recTotal={$pager->recTotal}&recPerPage={$pager->recPerPage}&pageID={$pager->pageID}";?>
+        <?php $vars = "programID=$programID&browseType=$browseType&orderBy=%s&recTotal={$pager->recTotal}&recPerPage={$pager->recPerPage}&pageID={$pager->pageID}";?>
         <thead>
           <tr>
             <th class='c-id'>