From 61f61261c9f8a1ab1796a88871d903c1f31ea55a Mon Sep 17 00:00:00 2001
From: =?utf8?q?Rapha=C3=ABl=20Barrois?= <raphael.barrois@polytechnique.org>
Date: Thu, 4 Feb 2010 22:03:45 +0100
Subject: [PATCH] Minor bugfixes
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>
---
 classes/userfilter.php | 44 +++++++++++++++++++++-----------------------
 include/common.inc.php |  4 +++-
 2 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/classes/userfilter.php b/classes/userfilter.php
index c2442f0..843de8f 100644
--- a/classes/userfilter.php
+++ b/classes/userfilter.php
@@ -818,7 +818,7 @@ class UFC_AddressField extends UFC_Address
             $field = 'postalCode';
             break;
         default:
-            Platal::page()->killError('Invalid address field type : ' . $this->fieldtype);
+            Platal::page()->killError('Invalid address field type: ' . $this->fieldtype);
         }
         $conds[] = $sub . '.' . $field . ' IN ' . XDB::formatArray($this->val);
 
@@ -964,12 +964,6 @@ class UFC_Job_Sectorization implements UserFilterCondition
 class UFC_Job_Description implements UserFilterCondition
 {
 
-    /** Meta-filters
-     * Built with binary OR on UserFilter::JOB_*
-     */
-    const ANY = 63;
-    const SECTORIZATION = 15;
-
     private $description;
     private $fields;
 
@@ -1291,7 +1285,7 @@ abstract class UserFilterOrder extends PlFilterOrder
      * @param &$uf The UserFilter whose results must be ordered
      * @return The name of the field to use for ordering results
      */
-    abstract protected function getSortTokens(UserFilter &$uf);
+//    abstract protected function getSortTokens(UserFilter &$uf);
 }
 // }}}
 
@@ -1310,7 +1304,7 @@ class UFO_Promo extends UserFilterOrder
         $this->grade = $grade;
     }
 
-    protected function getSortTokens(UserFilter &$uf)
+    protected function getSortTokens(PlFilter &$uf)
     {
         if (UserFilter::isGrade($this->grade)) {
             $sub = $uf->addEducationFilter($this->grade);
@@ -1344,7 +1338,7 @@ class UFO_Name extends UserFilterOrder
         $this->particle = $particle;
     }
 
-    protected function getSortTokens(UserFilter &$uf)
+    protected function getSortTokens(PlFilter &$uf)
     {
         if (UserFilter::isDisplayName($this->type)) {
             $sub = $uf->addDisplayFilter();
@@ -1364,7 +1358,7 @@ class UFO_Name extends UserFilterOrder
 // {{{ class UFO_Score
 class UFO_Score extends UserFilterOrder
 {
-    protected function getSortTokens(UserFilter &$uf)
+    protected function getSortTokens(PlFilter &$uf)
     {
         $sub = $uf->addNameTokensFilter();
         return 'SUM(' . $sub . '.score)';
@@ -1377,7 +1371,7 @@ class UFO_Score extends UserFilterOrder
  */
 class UFO_Registration extends UserFilterOrder
 {
-    protected function getSortTokens(UserFilter &$uf)
+    protected function getSortTokens(PlFilter &$uf)
     {
         return 'a.registration_date';
     }
@@ -1389,7 +1383,7 @@ class UFO_Registration extends UserFilterOrder
  */
 class UFO_Birthday extends UserFilterOrder
 {
-    protected function getSortTokens(UserFilter &$uf)
+    protected function getSortTokens(PlFilter &$uf)
     {
         return 'p.next_birthday';
     }
@@ -1401,7 +1395,7 @@ class UFO_Birthday extends UserFilterOrder
  */
 class UFO_ProfileUpdate extends UserFilterOrder
 {
-    protected function getSortTokens(UserFilter &$uf)
+    protected function getSortTokens(PlFilter &$uf)
     {
         return 'p.last_change';
     }
@@ -1413,7 +1407,7 @@ class UFO_ProfileUpdate extends UserFilterOrder
  */
 class UFO_Death extends UserFilterOrder
 {
-    protected function getSortTokens(UserFilter &$uf)
+    protected function getSortTokens(PlFilter &$uf)
     {
         return 'p.deathdate';
     }
@@ -1472,8 +1466,9 @@ class UserFilter extends PlFilter
 {
     protected $joinMethods = array();
 
-    protected $joinMetas = array('$PID' => 'p.pid',
-                                 '$UID' => 'a.uid',
+    protected $joinMetas = array(
+                                '$PID' => 'p.pid',
+                                '$UID' => 'a.uid',
                                 );
 
     private $root;
@@ -2215,12 +2210,15 @@ class UserFilter extends PlFilter
     /** JOBS
      */
 
-    const JOB_SECTOR = 1;
-    const JOB_SUBSECTOR = 2;
-    const JOB_SUBSUBSECTOR = 4;
-    const JOB_ALTERNATES = 8;
-    const JOB_USERDEFINED = 16;
-    const JOB_CV = 32;
+    const JOB_SECTOR        = 0x0001;
+    const JOB_SUBSECTOR     = 0x0002;
+    const JOB_SUBSUBSECTOR  = 0x0004;
+    const JOB_ALTERNATES    = 0x0008;
+    const JOB_USERDEFINED   = 0x0010;
+    const JOB_CV            = 0x0020;
+
+    const JOB_SECTORIZATION = 0x000F;
+    const JOB_ANY           = 0x003F;
 
     /** Joins :
      * pj => profile_job
diff --git a/include/common.inc.php b/include/common.inc.php
index 27209e6..e9f5217 100644
--- a/include/common.inc.php
+++ b/include/common.inc.php
@@ -26,7 +26,9 @@ function __autoload($cls)
         if (substr($cls, 0, 4) == 'ufc_' || substr($cls, 0, 4) == 'ufo_') {
             __autoload('userfilter');
             return;
-        } else if (substr($cls, 0, 4) == 'pfc_' || substr($cls, 0, 4) == 'pfo_' || substr($cls, 0, 8) == 'plfilter') {
+        } else if (substr($cls, 0, 4) == 'pfc_'
+                || substr($cls, 0, 4) == 'pfo_'
+                || substr($cls, 0, 8) == 'plfilter') {
             __autoload('plfilter');
             return;
         } else if (substr($cls, -3, 3) == 'req') {
-- 
2.1.4