X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=modules%2Fprofile%2Fpage.inc.php;h=de655ec98c78ec54f2a5ac8df97de765d70e49cf;hb=f280f651c1ea4059896655b132ddaec1e39cbd1b;hp=628a9498a85e98b8df646ec50e823a6c5a9bbdfd;hpb=d66464d5ea0a69c47f2f49ace9de2f4b7b8c4d62;p=platal.git diff --git a/modules/profile/page.inc.php b/modules/profile/page.inc.php index 628a949..de655ec 100644 --- a/modules/profile/page.inc.php +++ b/modules/profile/page.inc.php @@ -1,6 +1,6 @@ id = $id; - } else { - $this->id = S::i('uid'); - } $this->tel = new ProfileTel(); $this->pub = new ProfilePub(); $this->link_type = $type; @@ -139,17 +133,28 @@ class ProfilePhones implements ProfileSetting public function value(ProfilePage &$page, $field, $value, &$success) { $success = true; - if (is_null($value) || !is_array($value)) { + if (is_null($value)) { $value = array(); - $res = XDB::iterator("SELECT t.display_tel AS tel, t.tel_type AS type, t.pub, t.comment - FROM profile_phones AS t - WHERE t.uid = {?} AND t.link_type = {?} - ORDER BY t.tel_id", - $this->id, $this->link_type); - $value = $res->fetchAllAssoc(); + $res = XDB::iterator('SELECT display_tel AS tel, tel_type AS type, pub, comment + FROM profile_phones + WHERE pid = {?} AND link_type = {?} + ORDER BY tel_id', + $page->pid(), $this->link_type); + if ($res->numRows() > 0) { + $value = $res->fetchAllAssoc(); + } else { + $value = array( + 0 => array( + 'type' => 'fixed', + 'tel' => '', + 'pub' => 'private', + 'comment' => '', + ) + ); + } } foreach ($value as $key=>&$phone) { - if (@$phone['removed']) { + if (isset($phone['removed']) && $phone['removed']) { unset($value[$key]); } else { unset($phone['removed']); @@ -171,14 +176,14 @@ class ProfilePhones implements ProfileSetting return $value; } - private function saveTel($telid, array &$phone) + private function saveTel($pid, $telid, array &$phone) { if ($phone['tel'] != '') { - XDB::execute("INSERT INTO profile_phones (uid, link_type, link_id, tel_id, tel_type, + XDB::execute("INSERT INTO profile_phones (pid, link_type, link_id, tel_id, tel_type, search_tel, display_tel, pub, comment) VALUES ({?}, {?}, {?}, {?}, {?}, {?}, {?}, {?}, {?})", - $this->id, $this->link_type, $this->link_id, $telid, $phone['type'], + $pid, $this->link_type, $this->link_id, $telid, $phone['type'], format_phone_number($phone['tel']), $phone['tel'], $phone['pub'], $phone['comment']); } } @@ -186,16 +191,16 @@ class ProfilePhones implements ProfileSetting public function save(ProfilePage &$page, $field, $value) { XDB::execute("DELETE FROM profile_phones - WHERE uid = {?} AND link_type = {?} AND link_id = {?}", - $this->id, $this->link_type, $this->link_id); - $this->saveTels($field, $value); + WHERE pid = {?} AND link_type = {?} AND link_id = {?}", + $page->pid(), $this->link_type, $this->link_id); + $this->saveTels($page->pid(), $field, $value); } //Only saves phones without a delete operation - public function saveTels($field, $value) + public function saveTels($pid, $field, $value) { foreach ($value as $telid=>&$phone) { - $this->saveTel($telid, $phone); + $this->saveTel($pid, $telid, $phone); } } } @@ -260,12 +265,15 @@ abstract class ProfileGeocoding implements ProfileSetting { require_once 'geocoding.inc.php'; $success = true; - if ($address['changed'] == 1) { + if (isset($address['changed']) && $address['changed'] == 1) { $gmapsGeocoder = new GMapsGeocoder(); $address = $gmapsGeocoder->getGeocodedAddress($address); if (isset($address['geoloc'])) { $success = false; } + } elseif (@$address['changed'] && !@$address['text']) { + $address = empty_address(); + $address['pub'] = 'private'; } if (isset($address['geoloc_choice']) && ($address['geoloc_choice'] == 0)) { $mailer = new PlMailer('geoloc/geoloc.mail.tpl'); @@ -290,10 +298,14 @@ abstract class ProfilePage implements PlWizardPage public $orig = array(); public $values = array(); + public $profile = null; + public $owner = null; public function __construct(PlWizard &$wiz) { $this->wizard =& $wiz; + $this->profile = $this->wizard->getUserData('profile'); + $this->owner = $this->wizard->getUserData('owner'); } protected function _fetchData() @@ -332,19 +344,17 @@ abstract class ProfilePage implements PlWizardPage $setting->save($this, $field, $this->values[$field]); } if ($this->changed[$field] && @$this->watched[$field]) { - register_profile_update(S::i('uid'), $field); + WatchProfileUpdate::register($this->profile, $field); } } $this->_saveData(); // Update the last modification date - XDB::execute('REPLACE INTO user_changes - SET user_id = {?}', S::v('uid')); - if (!S::has('suid')) { - register_watch_op(S::i('uid'), WATCH_FICHE); - } + XDB::execute('UPDATE profiles + SET last_change = NOW() + WHERE pid = {?}', $this->pid()); global $platal; - S::logger()->log('profil', $platal->pl_self(1)); + S::logger()->log('profil', $platal->pl_self(2)); } protected function checkChanges() @@ -374,6 +384,16 @@ abstract class ProfilePage implements PlWizardPage return 'profile/base.tpl'; } + public function pid() + { + return $this->profile->id(); + } + + public function hrpid() + { + return $this->profile->hrpid(); + } + protected function _prepare(PlPage &$page, $id) { } @@ -387,11 +407,13 @@ abstract class ProfilePage implements PlWizardPage $page->assign($field, $value); } $this->_prepare($page, $id); + $page->assign('profile', $this->profile); + $page->assign('owner', $this->owner); $page->assign('profile_page', $this->pg_template); $page->assign('errors', $this->errors); } - public function process() + public function process(&$global_success) { $global_success = true; $this->fetchData(); @@ -414,9 +436,14 @@ abstract class ProfilePage implements PlWizardPage return Post::has('next_page') ? PlWizard::NEXT_PAGE : PlWizard::CURRENT_PAGE; } Platal::page()->trigError("Certains champs n'ont pas pu être validés, merci de corriger les informations " - . "de ton profil et de revalider ta demande"); + . "de ton profil et de revalider ta demande."); return PlWizard::CURRENT_PAGE; } + + public function success() + { + return 'Ton profil a bien été mis à jour.'; + } } require_once dirname(__FILE__) . '/general.inc.php';