From: Florent Bruneau Date: Fri, 5 Nov 2010 22:14:14 +0000 (+0100) Subject: Update following core@7da01959a. X-Git-Tag: xorg/1.0.2~21 X-Git-Url: http://git.polytechnique.org/?a=commitdiff_plain;h=93ded2f47e9fb5e873be559c78a922adb4bcb87d;p=platal.git Update following core@7da01959a. Signed-off-by: Florent Bruneau --- diff --git a/classes/userfilter.php b/classes/userfilter.php index a5fcb2d..dd277ce 100644 --- a/classes/userfilter.php +++ b/classes/userfilter.php @@ -48,7 +48,7 @@ abstract class UserFilterCondition implements PlFilterCondition */ class UFC_HasProfile extends UserFilterCondition { - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); return '$PID IS NOT NULL'; @@ -68,7 +68,7 @@ class UFC_AccountType extends UserFilterCondition $this->types = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireAccounts(); return XDB::format('a.type IN {?}', $this->types); @@ -88,7 +88,7 @@ class UFC_AccountPerm extends UserFilterCondition $this->perms = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requirePerms(); $conds = array(); @@ -119,7 +119,7 @@ class UFC_Hruid extends UserFilterCondition $this->hruids = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireAccounts(); return XDB::format('a.hruid IN {?}', $this->hruids); @@ -140,7 +140,7 @@ class UFC_Hrpid extends UserFilterCondition $this->hrpids = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); return XDB::format('p.hrpid IN {?}', $this->hrpids); @@ -161,7 +161,7 @@ class UFC_Ip extends UserFilterCondition $this->ip = $ip; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addLoggerFilter(); $ip = ip_to_uint($this->ip); @@ -180,7 +180,7 @@ class UFC_Comment extends UserFilterCondition $this->text = $text; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); return $uf->getVisibilityCondition('p.freetext_pub') . ' AND p.freetext ' . XDB::formatWildcards(XDB::WILDCARD_CONTAINS, $this->text); @@ -215,7 +215,7 @@ class UFC_Promo extends UserFilterCondition } } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { if ($this->grade == UserFilter::DISPLAY) { $sub = $uf->addDisplayFilter(); @@ -264,7 +264,7 @@ class UFC_SchoolId extends UserFilterCondition self::assertType($type); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); $ids = $this->ids; @@ -291,7 +291,7 @@ class UFC_EducationSchool extends UserFilterCondition $this->val = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addEducationFilter(); return XDB::format('pe' . $sub . '.eduid IN {?}', $this->val); @@ -309,7 +309,7 @@ class UFC_EducationDegree extends UserFilterCondition $this->diploma = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addEducationFilter(); return XDB::format('pe' . $sub . '.degreeid IN {?}', $this->diploma); @@ -327,7 +327,7 @@ class UFC_EducationField extends UserFilterCondition $this->val = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addEducationFilter(); return XDB::format('pe' . $sub . '.fieldid IN {?}', $this->val); @@ -361,13 +361,13 @@ class UFC_Name extends UserFilterCondition $this->mode = $mode; } - private function buildNameQuery($type, $variant, $where, UserFilter &$uf) + private function buildNameQuery($type, $variant, $where, UserFilter $uf) { $sub = $uf->addNameFilter($type, $variant); return str_replace('$ME', 'pn' . $sub, $where); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $left = '$ME.name'; if (($this->mode & self::PARTICLE) == self::PARTICLE) { @@ -419,7 +419,7 @@ class UFC_NameTokens extends UserFilterCondition $this->exact = $exact; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $conds = array(); foreach ($this->tokens as $i => $token) { @@ -452,7 +452,7 @@ class UFC_Nationality extends UserFilterCondition $this->val = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); $nat = XDB::formatArray($this->val); @@ -482,7 +482,7 @@ class UFC_Dead extends UserFilterCondition $this->date = make_datetime($date); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); $str = 'p.deathdate IS NOT NULL'; @@ -513,7 +513,7 @@ class UFC_Registered extends UserFilterCondition $this->date = make_datetime($date); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireAccounts(); if ($this->active) { @@ -545,7 +545,7 @@ class UFC_ProfileUpdated extends UserFilterCondition $this->date = $date; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); return 'p.last_change ' . $this->comparison . XDB::format(' {?}', date('Y-m-d H:i:s', $this->date)); @@ -569,7 +569,7 @@ class UFC_Birthday extends UserFilterCondition $this->date = $date; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $uf->requireProfiles(); return 'p.next_birthday ' . $this->comparison . XDB::format(' {?}', date('Y-m-d', $this->date)); @@ -589,7 +589,7 @@ class UFC_Sex extends UserFilterCondition $this->sex = $sex; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { if ($this->sex != User::GENDER_MALE && $this->sex != User::GENDER_FEMALE) { return self::COND_FALSE; @@ -616,7 +616,7 @@ class UFC_Group extends UserFilterCondition $this->anim = $anim; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { // Groups have AX visibility. if ($uf->getVisibilityLevel() == ProfileVisibility::VIS_PUBLIC) { @@ -645,7 +645,7 @@ class UFC_Binet extends UserFilterCondition $this->val = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { // Binets are private. if ($uf->getVisibilityLevel() != ProfileVisibility::VIS_PRIVATE) { @@ -670,7 +670,7 @@ class UFC_Section extends UserFilterCondition $this->section = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { // Sections are private. if ($uf->getVisibilityLevel() != ProfileVisibility::VIS_PRIVATE) { @@ -694,7 +694,7 @@ class UFC_Email extends UserFilterCondition $this->emails = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $foreign = array(); $virtual = array(); @@ -837,7 +837,7 @@ class UFC_AddressText extends UFC_Address return XDB::formatWildcards($this->textSearchMode, $txt); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addAddressFilter(); $conds = $this->initConds($sub, $uf->getVisibilityCondition($sub . '.pub')); @@ -894,7 +894,7 @@ class UFC_AddressField extends UFC_Address $this->fieldtype = $fieldtype; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addAddressFilter(); $conds = $this->initConds($sub, $uf->getVisibilityCondition($sub . '.pub')); @@ -944,7 +944,7 @@ class UFC_Corps extends UserFilterCondition $this->type = $type; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { /** Tables shortcuts: * pc for profile_corps, @@ -971,7 +971,7 @@ class UFC_Corps_Rank extends UserFilterCondition $this->rank = $rank; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { /** Tables shortcuts: * pc for profile_corps @@ -1015,7 +1015,7 @@ class UFC_Job_Company extends UserFilterCondition } } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addJobCompanyFilter(); $cond = $sub . '.' . $this->type . XDB::formatWildcards(XDB::WILDCARD_CONTAINS, $this->value); @@ -1043,7 +1043,7 @@ class UFC_Job_Terms extends UserFilterCondition $this->val = $val; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addJobTermsFilter(count($this->val)); $conditions = array(); @@ -1074,7 +1074,7 @@ class UFC_Job_Description extends UserFilterCondition $this->description = $description; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $conds = array(); @@ -1117,7 +1117,7 @@ class UFC_Networking extends UserFilterCondition $this->value = $value; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addNetworkingFilter(); $conds = array(); @@ -1162,7 +1162,7 @@ class UFC_Phone extends UserFilterCondition $this->phone_type = $phone_type; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addPhoneFilter(); $conds = array(); @@ -1197,7 +1197,7 @@ class UFC_Medal extends UserFilterCondition $this->grade = $grade; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $conds = array(); @@ -1220,7 +1220,7 @@ class UFC_Medal extends UserFilterCondition */ class UFC_Photo extends UserFilterCondition { - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addPhotoFilter(); return $sub . '.attach IS NOT NULL AND ' . $uf->getVisibilityCondition($sub . '.pub'); @@ -1231,7 +1231,7 @@ class UFC_Photo extends UserFilterCondition // {{{ class UFC_Mentor class UFC_Mentor extends UserFilterCondition { - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addMentorFilter(UserFilter::MENTOR); return $sub . '.expertise IS NOT NULL'; @@ -1253,7 +1253,7 @@ class UFC_Mentor_Expertise extends UserFilterCondition $this->expertise = $expertise; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addMentorFilter(UserFilter::MENTOR_EXPERTISE); return $sub . '.expertise ' . XDB::formatWildcards(XDB::WILDCARD_CONTAINS, $this->expertise); @@ -1274,7 +1274,7 @@ class UFC_Mentor_Country extends UserFilterCondition $this->country = pl_flatten(func_get_args()); } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addMentorFilter(UserFilter::MENTOR_COUNTRY); return $sub . '.country IN ' . XDB::format('{?}', $this->country); @@ -1295,7 +1295,7 @@ class UFC_Mentor_Terms extends UserFilterCondition $this->val = $val; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addMentorFilter(UserFilter::MENTOR_TERM); return $sub . '.jtid_1 = ' . XDB::escape($this->val); @@ -1322,7 +1322,7 @@ abstract class UFC_UserRelated extends UserFilterCondition */ class UFC_Contact extends UFC_UserRelated { - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $sub = $uf->addContactFilter($this->user->id()); return 'c' . $sub . '.contact IS NOT NULL'; @@ -1335,7 +1335,7 @@ class UFC_Contact extends UFC_UserRelated */ class UFC_WatchRegistration extends UFC_UserRelated { - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { if (!$this->user->watchType('registration')) { return PlFilterCondition::COND_FALSE; @@ -1364,7 +1364,7 @@ class UFC_WatchPromo extends UFC_UserRelated $this->grade = $grade; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $promos = $this->user->watchPromos(); if (count($promos) == 0) { @@ -1383,7 +1383,7 @@ class UFC_WatchPromo extends UFC_UserRelated */ class UFC_WatchContact extends UFC_Contact { - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { if (!$this->user->watchContacts()) { return PlFilterCondition::COND_FALSE; @@ -1406,7 +1406,7 @@ class UFC_MarketingHash extends UserFilterCondition $this->hash = $hash; } - public function buildCondition(PlFilter &$uf) + public function buildCondition(PlFilter $uf) { $table = $uf->addMarketingHash(); return XDB::format('rm.hash = {?}', $this->hash); @@ -1433,7 +1433,7 @@ class UFO_Promo extends PlFilterGroupableOrder $this->grade = $grade; } - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { if (UserFilter::isGrade($this->grade)) { $sub = $uf->addEducationFilter($this->grade); @@ -1467,7 +1467,7 @@ class UFO_Name extends PlFilterOrder $this->particle = $particle; } - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { if (Profile::isDisplayName($this->type)) { $sub = $uf->addDisplayFilter(); @@ -1493,7 +1493,7 @@ class UFO_Name extends PlFilterOrder // {{{ class UFO_Score class UFO_Score extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $toks = $uf->getNameTokens(); $scores = array(); @@ -1516,7 +1516,7 @@ class UFO_Score extends PlFilterOrder */ class UFO_Registration extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireAccounts(); return 'a.registration_date'; @@ -1529,7 +1529,7 @@ class UFO_Registration extends PlFilterOrder */ class UFO_Birthday extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireProfiles(); return 'p.next_birthday'; @@ -1542,7 +1542,7 @@ class UFO_Birthday extends PlFilterOrder */ class UFO_ProfileUpdate extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireProfiles(); return 'p.last_change'; @@ -1555,7 +1555,7 @@ class UFO_ProfileUpdate extends PlFilterOrder */ class UFO_Death extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireProfiles(); return 'p.deathdate'; @@ -1568,7 +1568,7 @@ class UFO_Death extends PlFilterOrder */ class UFO_Uid extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireAccounts(); return '$UID'; @@ -1581,7 +1581,7 @@ class UFO_Uid extends PlFilterOrder */ class UFO_Hruid extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireAccounts(); return 'a.hruid'; @@ -1594,7 +1594,7 @@ class UFO_Hruid extends PlFilterOrder */ class UFO_Pid extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireProfiles(); return '$PID'; @@ -1607,7 +1607,7 @@ class UFO_Pid extends PlFilterOrder */ class UFO_Hrpid extends PlFilterOrder { - protected function getSortTokens(PlFilter &$uf) + protected function getSortTokens(PlFilter $uf) { $uf->requireProfiles(); return 'p.hrpid'; @@ -2046,13 +2046,13 @@ class UserFilter extends PlFilter } } - public function setCondition(PlFilterCondition &$cond) + public function setCondition(PlFilterCondition $cond) { $this->root =& $cond; $this->query = null; } - public function addSort(PlFilterOrder &$sort) + public function addSort(PlFilterOrder $sort) { if (count($this->sort) == 0 && $sort instanceof PlFilterGroupableOrder) {