X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=include%2Fuserset.inc.php;h=23a117ee8c2acc4c2c35d639d4b915f6cfa1c691;hb=87d72675e4a6d5e2ca377cde0f30666f6305ad2b;hp=18d1ee6817a32585a8ae98ff0460551c26563df1;hpb=b5b33b82ba1929364e6dc69ddb2575a2e93f7995;p=platal.git

diff --git a/include/userset.inc.php b/include/userset.inc.php
index 18d1ee6..23a117e 100644
--- a/include/userset.inc.php
+++ b/include/userset.inc.php
@@ -53,8 +53,6 @@ class SearchSet extends ProfileSet
 
     public function __construct($quick = false, $no_search = false, PlFilterCondition $cond = null)
     {
-        Platal::load('search', 'search.inc.php');
-
         if ($no_search) {
             return;
         }
@@ -93,13 +91,11 @@ class SearchSet extends ProfileSet
         $this->conds->addChild($ufc);
 
         $orders = $ufb->getOrders();
-        $orders[] = new UFO_Promo(UserFilter::DISPLAY, true);
-        $orders[] = new UFO_Name(Profile::DN_SORT);
 
         if (S::logged() && Env::has('nonins')) {
             $this->conds = new PFC_And($this->conds,
-                new UFC_Not(new UFC_Dead()),
-                new UFC_Not(new UFC_Registered())
+                new PFC_Not(new UFC_Dead()),
+                new PFC_Not(new UFC_Registered())
             );
         }
 
@@ -119,21 +115,13 @@ class SearchSet extends ProfileSet
         $this->conds->addChild($ufb->getUFC());
     }
 
-    public function &get(PlLimit $limit = null, $orders = array())
+    protected function &getFilterResults(PlFilter &$pf, PlLimit $limit)
     {
-        $orders = array_merge($orders, $this->orders);
-
-        $uf = $this->buildFilter($this->conds, $orders);
-        if (is_null($limit)) {
-            $limit = new PlLimit(20, 0);
-        }
-        $it          = $uf->getProfiles($limit);
-        $this->count = $uf->getTotalCount();
-        return $it;
+        return $pf->getProfiles($limit, Profile::FETCH_MINIFICHES);
     }
 }
 
-class ArraySet extends UserSet
+class ArraySet extends ProfileSet
 {
     public function __construct(array $users)
     {
@@ -147,44 +135,15 @@ class ArraySet extends UserSet
     }
 }
 
-class MinificheView extends MultipageView
+abstract class ProfileView extends MultipageView
 {
-    public function __construct(PlSet &$set, $data, array $params)
-    {
-        require_once 'education.func.inc.php';
-        global $globals;
-        $this->entriesPerPage = $globals->search->per_page;
-        if (@$params['with_score']) {
-            $this->addSort(new PlViewOrder('score', array(
-                    new UFO_Score(true),
-                    new UFO_ProfileUpdate(true),
-                    new UFO_Promo(UserFilter::DISPLAY, true),
-                    new UFO_Name(Profile::DN_SORT),
-                ), 'pertinence'));
-        }
-        $this->addSort(new PlViewOrder(
-                            'name',
-                            array(new UFO_Name(Profile::DN_SORT)),
-                            'nom'));
-        $this->addSort(new PlViewOrder('promo', array(
-                    new UFO_Promo(UserFilter::DISPLAY, true),
-                    new UFO_Name(Profile::DN_SORT),
-                ), 'promotion'));
-        $this->addSort(new PlViewOrder('date_mod', array(
-                    new UFO_ProfileUpdate(true),
-                    new UFO_Promo(UserFilter::DISPLAY, true),
-                    new UFO_Name(Profile::DN_SORT),
-                ), 'dernière modification'));
-        parent::__construct($set, $data, $params);
-    }
-
     protected function getBoundValue($obj)
     {
         if ($obj instanceof Profile) {
             switch ($this->bound_field) {
             case 'name':
                 $name = $obj->name('%l');
-                return strtoupper($name[0]);
+                return strtoupper($name);
             case 'promo':
                 return $obj->promo();
             default:
@@ -210,6 +169,38 @@ class MinificheView extends MultipageView
         }
         return $show_bounds;
     }
+}
+
+class MinificheView extends ProfileView
+{
+    public function __construct(PlSet &$set, $data, array $params)
+    {
+        require_once 'education.func.inc.php';
+        global $globals;
+        $this->entriesPerPage = $globals->search->per_page;
+        if (@$params['with_score']) {
+            $this->addSort(new PlViewOrder('score', array(
+                    new UFO_Score(true),
+                    new UFO_ProfileUpdate(true),
+                    new UFO_Promo(UserFilter::DISPLAY, true),
+                    new UFO_Name(Profile::DN_SORT),
+                ), 'pertinence'));
+        }
+        $this->addSort(new PlViewOrder(
+                            'name',
+                            array(new UFO_Name(Profile::DN_SORT)),
+                            'nom'));
+        $this->addSort(new PlViewOrder('promo', array(
+                    new UFO_Promo(UserFilter::DISPLAY, true),
+                    new UFO_Name(Profile::DN_SORT),
+                ), 'promotion'));
+        $this->addSort(new PlViewOrder('date_mod', array(
+                    new UFO_ProfileUpdate(true),
+                    new UFO_Promo(UserFilter::DISPLAY, true),
+                    new UFO_Name(Profile::DN_SORT),
+                ), 'dernière modification'));
+        parent::__construct($set, $data, $params);
+    }
 
     public function templateName()
     {
@@ -217,7 +208,7 @@ class MinificheView extends MultipageView
     }
 }
 
-class MentorView extends MultipageView
+class MentorView extends ProfileView
 {
     public function __construct(PlSet &$set, $data, array $params)
     {
@@ -236,50 +227,18 @@ class MentorView extends MultipageView
         parent::__construct($set, $data, $params);
     }
 
-    protected function getBoundValue($obj)
-    {
-        if ($obj instanceof Profile) {
-            switch ($this->bound_field) {
-            case 'name':
-                $name = $obj->name('%l');
-                return strtoupper($name[0]);
-            case 'promo':
-                return $obj->promo();
-            default:
-                return null;
-            }
-        }
-        return null;
-    }
-
-    public function bounds()
-    {
-        $order = Env::v('order', $this->defaultkey);
-        $show_bounds = 0;
-        if (($order == "name") || ($order == "-name")) {
-            $this->bound_field = "nom";
-            $show_bounds = 1;
-        } elseif (($order == "promo") || ($order == "-promo")) {
-            $this->bound_field = "promo";
-            $show_bounds = -1;
-        }
-        if ($order{0} == '-') {
-            $show_bounds = -$show_bounds;
-        }
-        return $show_bounds;
-    }
-
     public function templateName()
     {
         return 'include/plview.referent.tpl';
     }
 }
 
-class TrombiView extends MultipageView
+class TrombiView extends ProfileView
 {
     public function __construct(PlSet &$set, $data, array $params)
     {
         $this->entriesPerPage = 24;
+        $this->defaultkey = 'name';
         if (@$params['with_score']) {
             $this->addSort(new PlViewOrder('score', array(
                             new UFO_Score(true),
@@ -288,6 +247,7 @@ class TrombiView extends MultipageView
                             new UFO_Name(Profile::DN_SORT),
             ), 'pertinence'));
         }
+        $set->addCond(new UFC_Photo());
         $this->addSort(new PlViewOrder('name', array(new UFO_Name(Profile::DN_SORT)), 'nom'));
         $this->addSort(new PlViewOrder('promo', array(
                         new UFO_Promo(UserFilter::DISPLAY, true),
@@ -296,39 +256,6 @@ class TrombiView extends MultipageView
         parent::__construct($set, $data, $params);
     }
 
-    protected function getBoundValue($obj)
-    {
-        if ($obj instanceof Profile) {
-            switch ($this->bound_field) {
-            case 'name':
-                $name = $obj->name('%l');
-                return strtoupper($name[0]);
-            case 'promo':
-                return $obj->promo();
-            default:
-                return null;
-            }
-        }
-        return null;
-    }
-
-    public function bounds()
-    {
-        $order = Env::v('order', $this->defaultkey);
-        $show_bounds = 0;
-        if (($order == "name") || ($order == "-name")) {
-            $this->bound_field = "nom";
-            $show_bounds = 1;
-        } elseif (($order == "promo") || ($order == "-promo")) {
-            $this->bound_field = "promo";
-            $show_bounds = -1;
-        }
-        if ($order{0} == '-') {
-            $show_bounds = -$show_bounds;
-        }
-        return $show_bounds;
-    }
-
     public function templateName()
     {
         return 'include/plview.trombi.tpl';