Merge branch 'fixbug_wangxiaomeng_#21773_#21650' into 'master'

Fixbug wangxiaomeng #21773 ,21650.

See merge request easycorp/zentaopms!2990

module/search/model.php

@@ -334,7 +334,7 @@ class searchModel extends model
      *
      * @param  int    $queryID
      * @access public
-     * @return string
+     * @return object
      */
     public function getQuery($queryID)
     {
@@ -357,6 +357,20 @@ class searchModel extends model
         return $query;
     }
 
+    /**
+     * Get a query.
+     *
+     * @param  int    $queryID
+     * @access public
+     * @return object
+     */
+    public function getByID($queryID)
+    {
+        $query = $this->dao->findByID($queryID)->from(TABLE_USERQUERY)->fetch();
+        if(!$query) return false;
+        return $query;
+    }
+
     /**
      * Save current query to db.
      *

module/search/view/buildform.html.php

@@ -228,7 +228,7 @@ foreach($fieldParams as $fieldName => $param)
         <ul>
           <?php foreach($queries as $queryID => $queryName):?>
           <?php if(empty($queryID)) continue;?>
-          <?php $query = $this->search->getQuery($queryID);?>
+          <?php $query = $this->search->getByID($queryID);?>
           <li><?php echo html::a("javascript:executeQuery($queryID)", $queryName . ((common::hasPriv('search', 'deleteQuery') and $this->app->user->account == $query->account) ? '<i class="icon icon-close"></i>' : ''), '', "class='label user-query' data-query-id='$queryID' title='{$queryName}'");?></li>
           <?php endforeach;?>
         </ul>
@@ -363,7 +363,6 @@ $(function()
     var setDateField = function(query, fieldNO)
     {
         var $period = $('#selectPeriod');
-        if($period.length) $period.remove();
 
         <?php
         $selectPeriod  = "<ul id='selectPeriod' class='dropdown-menu'>";