<?php
/***************************************************************************
- * Copyright (C) 2003-2011 Polytechnique.org *
+ * Copyright (C) 2003-2014 Polytechnique.org *
* http://opensource.polytechnique.org/ *
* *
* This program is free software; you can redistribute it and/or modify *
{
$address = $this->text;
if ($this->type == self::LINK_PROFILE || $this->type == self::LINK_JOB) {
- static $pubs = array('public' => 'publique', 'ax' => 'annuaire AX', 'private' => 'privé');
+ static $pubs = array('public' => 'publique', 'ax' => 'annuaire papier', 'private' => 'privé', 'hidden' => 'administrateurs');
$address .= ' (affichage ' . $pubs[$this->pub];
}
if ($this->type == self::LINK_PROFILE) {
$this->types, $this->formatted_address, $this->location_type, $this->partial_match, $this->latitude, $this->longitude,
$this->southwest_latitude, $this->southwest_longitude, $this->northeast_latitude, $this->northeast_longitude, $this->geocoding_calls, $this->postal_code_fr);
+ // In an ideal world there would not be any components to clean up before insertion.
+ // In real world, it happens that some addresses are badly removed and this query removes the leaked trash.
+ XDB::execute('DELETE FROM profile_addresses_components
+ WHERE pid = {?} AND jobid = {?} AND groupid = {?} AND type = {?} AND id = {?}',
+ $this->pid, $this->jobid, $this->groupid, $this->type, $this->id);
if ($this->componentsIds) {
foreach (explode(',', $this->componentsIds) as $component_id) {
XDB::execute('INSERT IGNORE INTO profile_addresses_components (pid, jobid, groupid, type, id, component_id)
$req = new AddressReq(S::user(), $this->toFormArray(), $this->pid, $this->jobid, $this->groupid, $this->type, $this->id);
$req->submit();
}
+
+ if ($this->pid != 0) {
+ self::updateBestMail($this->pid);
+ }
}
}
- public function updateGeocoding($text)
+ /**
+ * Upate the denormalized flag which is used to mark the best mail to use
+ * when sending postal mail
+ *
+ * Call with $fake to true to only get which address would be selected,
+ * without updating anything in the database.
+ * Returns an array describing the selected profile address
+ */
+ static public function updateBestMail($pid, $fake=false)
+ {
+ if (!$fake) {
+ XDB::execute("UPDATE profile_addresses
+ SET flags = REPLACE(flags, 'dn_best_mail', '')
+ WHERE pid = {?}",
+ $pid);
+ }
+
+ /* Following order is selected to find the best mail:
+ * * Use addresses without the deliveryIssue flag if possible.
+ * * Among these, use addresses flagged as "current".
+ * * If there is no such addresses or several ones, prefer those
+ * without "secondary" flag.
+ * * If there are still several addresses in the selection, try not
+ * to select the ones with "job" type.
+ */
+ $best_mail = XDB::fetchOneAssoc("SELECT pid, jobid, groupid, type, id, flags
+ FROM profile_addresses
+ WHERE FIND_IN_SET('mail', flags) AND pid = {?}
+ ORDER BY FIND_IN_SET('deliveryIssue', flags),
+ NOT FIND_IN_SET('current', flags),
+ FIND_IN_SET('secondary', flags), type = 'job'
+ LIMIT 1",
+ $pid);
+
+ if (!$fake && $best_mail) {
+ XDB::execute("UPDATE profile_addresses
+ SET flags = CONCAT(flags, ',dn_best_mail')
+ WHERE pid = {?} AND jobid = {?} AND groupid = {?} AND type = {?} AND id = {?}",
+ $best_mail['pid'], $best_mail['jobid'], $best_mail['groupid'], $best_mail['type'], $best_mail['id']);
+ }
+ return $best_mail;
+ }
+
+ public function updateGeocoding()
{
XDB::execute('UPDATE profile_addresses
SET text = {?}, postalText = {?}, types = {?}, formatted_address = {?},
{
$where = '';
if (!is_null($pid)) {
- $where = XDB::format(' AND pid = {?}', $pid);
+ $where .= XDB::format(' AND pid = {?}', $pid);
}
if (!is_null($jobid)) {
- $where = XDB::format(' AND jobid = {?}', $jobid);
+ $where .= XDB::format(' AND jobid = {?}', $jobid);
}
if (!is_null($groupid)) {
- $where = XDB::format(' AND groupid = {?}', $groupid);
+ $where .= XDB::format(' AND groupid = {?}', $groupid);
+ }
+ // Prevent buggy callers from accidentally dropping profile_addresses
+ if (!$where) {
+ throw new Exception("Unable to delete all addresses of the given type, the request was too generic");
+ }
+ $where_pub = $where . (($deletePrivate) ? '' : ' AND pub IN (\'public\', \'ax\')');
+
+ if ($deletePrivate) {
+ XDB::execute('DELETE FROM profile_addresses_components
+ WHERE type = {?}' . $where,
+ $type);
+ } else {
+ // Delete address components one by one if $deletePrivate is not requested
+ $res = XDB::iterator('SELECT pid, jobid, groupid, type, id
+ FROM profile_addresses
+ WHERE type = {?}' . $where_pub,
+ $type);
+ while (($values = $res->next())) {
+ XDB::execute('DELETE FROM profile_addresses_components
+ WHERE pid = {?} AND jobid = {?} AND groupid = {?} AND type = {?} AND id = {?}',
+ $values['pid'], $values['jobid'], $values['groupid'], $values['type'], $values['id']);
+ }
}
XDB::execute('DELETE FROM profile_addresses
- WHERE type = {?}' . $where . (($deletePrivate) ? '' : ' AND pub IN (\'public\', \'ax\')'),
+ WHERE type = {?}' . $where_pub,
$type);
if ($type == self::LINK_PROFILE) {
Phone::deletePhones($pid, Phone::LINK_ADDRESS, null, $deletePrivate);
}
}
-/** Iterator over a set of Phones
+/** Iterator over a set of Addresses
*
* @param $pid, $type, $jobid, $pub
*
- * The iterator contains the phones that correspond to the value stored in the
- * parameters' arrays.
+ * The iterator contains the addresses that correspond to the value stored in
+ * the parameters' arrays.
*/
class AddressIterator implements PlIterator
{
}
}
-// vim:set et sw=4 sts=4 sws=4 foldmethod=marker enc=utf-8:
+// vim:set et sw=4 sts=4 sws=4 foldmethod=marker fenc=utf-8:
?>