Changes in [508:7dbb15fcd538:509:dbad98223895]

Files:

• admin/comments.php (modified) (7 diffs)
• admin/posts.php (modified) (1 diff)
• inc/admin/class.dc.filter.php (modified) (6 diffs)
• inc/core/class.dc.blog.php (modified) (1 diff)

admin/comments.php

@@ -17 +17 @@
 # Creating filter combo boxes
 # Filter form we'll put in html_block
-$status_combo = array(
-'-' => ''
-);
+$status_combo = array();
 foreach ($core->blog->getAllCommentStatus() as $k => $v) {
      $status_combo[$v] = (string) $k;
@@ -25 +23 @@
 
 $type_combo = array(
-'-' => '',
-__('comment') => 'co',
-__('trackback') => 'tb'
+__('comment') => '0',
+__('trackback') => '1'
 );
 
@@ -43 +40 @@
 
 
-/* Get comments
--------------------------------------------------------- */
-$author = isset($_GET['author']) ? $_GET['author'] : '';
-$status = isset($_GET['status']) ?      $_GET['status'] : '';
-$type = !empty($_GET['type']) ?         $_GET['type'] : '';
-$sortby = !empty($_GET['sortby']) ?     $_GET['sortby'] : 'comment_dt';
-$order = !empty($_GET['order']) ?       $_GET['order'] : 'desc';
-$ip = !empty($_GET['ip']) ?             $_GET['ip'] : '';
-
-$with_spam = $author || $status || $type || $sortby != 'comment_dt' || $order != 'desc' || $ip;
-
-$show_filters = false;
 
 $page = !empty($_GET['page']) ? (integer) $_GET['page'] : 1;
@@ -65 +50 @@
      $nb_per_page = (integer) $_GET['nb'];
 }
-
+$params = new ArrayObject();
 $params['limit'] = array((($page-1)*$nb_per_page),$nb_per_page);
 $params['no_content'] = true;
-
-# Author filter
-if ($author !== '') {
-     $params['q_author'] = $author;
-     $show_filters = true;
-}
-
-# - Type filter
-if ($type == 'tb' || $type == 'co') {
-     $params['comment_trackback'] = ($type == 'tb');
-     $show_filters = true;
-}
-
-# - Status filter
-if ($status !== '' && in_array($status,$status_combo)) {
-     $params['comment_status'] = $status;
-     $show_filters = true;
-} elseif (!$with_spam) {
-     $params['comment_status_not'] = -2;
-}
-
-# - IP filter
-if ($ip) {
-     $params['comment_ip'] = $ip;
-     $show_filters = true;
-}
-
-# Sortby and order filter
-if ($sortby !== '' && in_array($sortby,$sortby_combo)) {
-     if ($order !== '' && in_array($order,$order_combo)) {
-          $params['order'] = $sortby.' '.$order;
-     }
-     
-     if ($sortby != 'comment_dt' || $order != 'desc') {
-          $show_filters = true;
-     }
-}
 
 # Actions combo box
@@ -123 +71 @@
 $core->callBehavior('adminCommentsActionsCombo',array(&$combo_action));
 
+$filterSet = new dcFilterSet('comments','comments.php');
+
+$authorFilter = new textFilter(
+          'author',__('Author'),'q_author',20,255);
+$filterSet
+     ->addFilter(new comboFilter(
+          'status',__('Status'), 'comment_status', $status_combo))
+     ->addFilter(new booleanFilter(
+          'type',__('Type'), 'comment_trackback', $type_combo))
+     ->addFilter($authorFilter)
+     ->addFilter(new textFilter(
+          'ip',__('IP address'),'comment_ip',20,39));
+          
+$core->callBehavior('adminCommentsFilters',$filterSet);
+
+$filterSet->setFormValues($_GET);
+if (isset($_GET['author'])) {
+     $authorFilter->add();
+     $authorFilter->setValue($_GET['author']);
+}
 /* Get comments
 -------------------------------------------------------- */
 try {
+     $nfparams = $params->getArrayCopy();
+     $filtered = $filterSet->applyFilters($params);
+     $core->callBehavior('adminCommentsParams',$params);
      $comments = $core->blog->getComments($params);
      $counter = $core->blog->getComments($params,true);
+     if ($filtered) {
+          $totalcounter = $core->blog->getComments($nfparams,true);
+          $page_title = sprintf(__('Comments and Trackacks / %s filtered out of %s'),$counter->f(0),$totalcounter->f(0));
+     } else {
+          $page_title = __('Comments and Trackacks');
+     }
+
      $comment_list = new adminCommentList($core,$comments,$counter->f(0));
 } catch (Exception $e) {
      $core->error->add($e->getMessage());
 }
+$filterSet->setColumnsForm($comment_list->getColumnsForm());
 
 /* DISPLAY
 -------------------------------------------------------- */
-$starting_script = dcPage::jsLoad('js/_comments.js');
-if (!$show_filters) {
-     $starting_script .= dcPage::jsLoad('js/filter-controls.js');
-}
+$starting_script = dcPage::jsLoad('js/_comments.js').$filterSet->header();;
+
 # --BEHAVIOR-- adminCommentsHeaders
 $starting_script .= $core->callBehavior('adminCommentsHeaders');
@@ -144 +121 @@
 dcPage::open(__('Comments and trackbacks'),$starting_script);
 
-echo '<h2>'.html::escapeHTML($core->blog->name).' &rsaquo; '.__('Comments and trackbacks').'</h2>';
+echo '<h2>'.html::escapeHTML($core->blog->name).' &rsaquo; '.$page_title.'</h2>';
 
 if (!$core->error->flag())
@@ -153 +130 @@
           __('Filters').'</a></p>';
      }
-     
-     echo
-     '<form action="comments.php" method="get" id="filters-form">'.
-     '<fieldset><legend>'.__('Filters').'</legend>'.
-     '<div class="three-cols">'.
-     '<div class="col">'.
-     '<label for="type">'.__('Type:').' '.
-     form::combo('type',$type_combo,$type).
-     '</label> '.
-     '<label for="status">'.__('Status:').' '.
-     form::combo('status',$status_combo,$status).
-     '</label>'.
-     '</div>'.
-     
-     '<div class="col">'.
-     '<p><label for="sortby">'.__('Order by:').' '.
-     form::combo('sortby',$sortby_combo,$sortby).
-     '</label> '.
-     '<label for="order">'.__('Sort:').' '.
-     form::combo('order',$order_combo,$order).
-     '</label></p>'.
-     '<p><label for="nb" class="classic">'.  form::field('nb',3,3,$nb_per_page).' '.
-     __('Comments per page').'</label></p>'.
-     '</div>'.
-     
-     '<div class="col">'.
-     '<p><label for="author">'.__('Comment author:').' '.
-     form::field('author',20,255,html::escapeHTML($author)).
-     '</label>'.
-     '<label for="ip">'.__('IP address:').' '.
-     form::field('ip',20,39,html::escapeHTML($ip)).
-     '</label></p>'.
-     '<p><input type="submit" value="'.__('Apply filters').'" /></p>'.
-     '</div>'.
-     
-     '</div>'.
-     '<br class="clear" />'. //Opera sucks
-     '</fieldset>'.
-     '</form>';
+     $filterSet->display();
      
      if (!$with_spam) {

admin/posts.php

@@ -166 +166 @@
 $core->callBehavior('adminPostsFilters',$filterSet);
 
-$filterSet->setValues($_GET);
+$filterSet->setFormValues($_GET);
 
 # Get posts

inc/admin/class.dc.filter.php

@@ -91 +91 @@
      @param    form_data <b>array</b>   form values (usually $_GET or $_POST)
      */
-     public function setValues ($form_data) {
+     public function setFormValues ($form_data) {
           $this->hideform = true;
           if (isset($form_data['clear_filters'])) {
@@ -101 +101 @@
           }
           foreach ($this->filters as $filter) {
-               $filter->setValues ($form_data);
+               $filter->setFormValues ($form_data);
                if ($filter->isEnabled()) {
                     $this->hideform=false;
@@ -337 +337 @@
      @param    $form_data     <b>array</b>   form data
      */
-     public function setValues($form_data) {
+     public function setFormValues($form_data) {
           $count=0;
           while (isset($form_data[$this->getFieldId($count)])) {
@@ -389 +389 @@
      }
      
+     public function setValues($value) {
+          $this->values = $value;
+     }
+     
+     public function getValue() {
+          return $this->values;
+     }
+     
 }
 
@@ -436 +444 @@
      }
      
-     public function setValues($form_data) {
-          parent::setValues($form_data);
+     public function setFormValues($form_data) {
+          parent::setFormValues($form_data);
           if (isset($form_data[$this->field_id."_v"])) {
                $this->verb = ($form_data[$this->field_id."_v"] == 'is') ? 'is' : 'isnot';
@@ -514 +522 @@
 }
 
+
+class textFilter extends Filter {
+     protected $size;
+     protected $max;
+     
+     public function __construct($id,$desc,$request_param,$size,$max) {
+          parent::__construct($id,$desc,$request_param);
+          $this->options = $options;
+          $this->values=array();
+          $this->size = $size;
+          $this->max = $max;
+     }
+     
+     
+     public function getType() {
+          return "text";
+     }
+     public function add() {
+          parent::add();
+          $this->values[]='';
+     }
+
+     public function getFormFields($pos=0) {
+          return '<span class="'.$labelclass.'">'.$this->desc.'</span>'.
+               form::field($this->getFieldId($pos),$this->size,$this->max,html::escapeHTML($this->values[0]));
+     }
+     
+     public function applyFilter($params) {
+          $params[$this->request_param]=$this->values[0];
+     }
+     
+     public function setValues($value) {
+          parent::setValues(array($value));
+     }
+
+     public function getValue() {
+          $v = parent::getValue();
+          return $v[0];
+     }
+     
+}
 ?>

inc/core/class.dc.blog.php

@@ -1812 +1812 @@
           
           if (isset($params['comment_status'])) {
-               $strReq .= 'AND comment_status = '.(integer) $params['comment_status'].' ';
-          }
-          
-          if (!empty($params['comment_status_not']))
+               $strReq .= 'AND '.$this->getInParamStr($params,'comment_status','comment_status');
+          } 
+          elseif (!empty($params['comment_status_not'])) // older method, deprecated
           {
                $strReq .= 'AND comment_status <> '.(integer) $params['comment_status_not'].' ';