]> git.mxchange.org Git - friendica.git/blobdiff - include/Contact.php
Merge remote-tracking branch 'upstream/develop' into rewrites/coding-convention
[friendica.git] / include / Contact.php
index 2aab828f8aaf2d8cc169251f05f2af7cb6b722b3..dc29d38257a38a6cca4f0ae60c7f0df152215661 100644 (file)
@@ -1,13 +1,11 @@
 <?php
 
-require_once('include/Probe.php');
-
 // Included here for completeness, but this is a very dangerous operation.
 // It is the caller's responsibility to confirm the requestor's intent and
 // authorisation to do this.
 
 function user_remove($uid) {
-       if(! $uid)
+       if (! $uid)
                return;
        logger('Removing user: ' . $uid);
 
@@ -51,7 +49,7 @@ function user_remove($uid) {
        // Send an update to the directory
        proc_run(PRIORITY_LOW, "include/directory.php", $r[0]['url']);
 
-       if($uid == local_user()) {
+       if ($uid == local_user()) {
                unset($_SESSION['authenticated']);
                unset($_SESSION['uid']);
                goaway(App::get_baseurl());
@@ -124,12 +122,14 @@ function terminate_friendship($user,$self,$contact) {
 // This provides for the possibility that their database is temporarily messed
 // up or some other transient event and that there's a possibility we could recover from it.
 
-function mark_for_death($contact) {
+function mark_for_death(array $contact) {
 
-       if($contact['archive'])
+       if ($contact['archive']) {
                return;
+       }
 
-       if($contact['term-date'] == '0000-00-00 00:00:00') {
+       /// @TODO Comparison of strings this way may lead to bugs/incompatibility, better switch to DateTime
+       if ($contact['term-date'] <= NULL_DATE) {
                q("UPDATE `contact` SET `term-date` = '%s' WHERE `id` = %d",
                                dbesc(datetime_convert()),
                                intval($contact['id'])
@@ -153,7 +153,7 @@ function mark_for_death($contact) {
                /// Check for contact vitality via probing
 
                $expiry = $contact['term-date'] . ' + 32 days ';
-               if(datetime_convert() > datetime_convert('UTC','UTC',$expiry)) {
+               if (datetime_convert() > datetime_convert('UTC','UTC',$expiry)) {
 
                        // relationship is really truly dead.
                        // archive them rather than delete
@@ -187,13 +187,13 @@ function unmark_for_death($contact) {
 
        // It's a miracle. Our dead contact has inexplicably come back to life.
        q("UPDATE `contact` SET `term-date` = '%s' WHERE `id` = %d",
-               dbesc('0000-00-00 00:00:00'),
+               dbesc(NULL_DATE),
                intval($contact['id'])
        );
 
        if ($contact['url'] != '') {
                q("UPDATE `contact` SET `term-date` = '%s' WHERE `nurl` = '%s'",
-                       dbesc('0000-00-00 00:00:00'),
+                       dbesc(NULL_DATE),
                        dbesc(normalise_link($contact['url']))
                );
        }
@@ -355,6 +355,7 @@ function get_contact_details_by_addr($addr, $uid = -1) {
                                dbesc($addr));
 
        if (!dbm::is_result($r)) {
+               require_once('include/Probe.php');
                $data = Probe::uri($addr);
 
                $profile = get_contact_details_by_url($data['url'], $uid);
@@ -484,7 +485,7 @@ function random_profile() {
 
 function contacts_not_grouped($uid,$start = 0,$count = 0) {
 
-       if(! $count) {
+       if (! $count) {
                $r = q("select count(*) as total from contact where uid = %d and self = 0 and id not in (select distinct(`contact-id`) from group_member where uid = %d) ",
                        intval($uid),
                        intval($uid)
@@ -508,72 +509,96 @@ function contacts_not_grouped($uid,$start = 0,$count = 0) {
 /**
  * @brief Fetch the contact id for a given url and user
  *
+ * First lookup in the contact table to find a record matching either `url`, `nurl`,
+ * `addr` or `alias`.
+ *
+ * If there's no record and we aren't looking for a public contact, we quit.
+ * If there's one, we check that it isn't time to update the picture else we
+ * directly return the found contact id.
+ *
+ * Second, we probe the provided $url wether it's http://server.tld/profile or
+ * nick@server.tld. We quit if we can't get any info back.
+ *
+ * Third, we create the contact record if it doesn't exist
+ *
+ * Fourth, we update the existing record with the new data (avatar, alias, nick)
+ * if there's any updates
+ *
  * @param string $url Contact URL
- * @param integer $uid The user id for the contact
+ * @param integer $uid The user id for the contact (0 = public contact)
  * @param boolean $no_update Don't update the contact
  *
  * @return integer Contact ID
  */
 function get_contact($url, $uid = 0, $no_update = false) {
-       require_once("include/Scrape.php");
-
        logger("Get contact data for url ".$url." and user ".$uid." - ".App::callstack(), LOGGER_DEBUG);;
 
        $data = array();
-       $contactid = 0;
-
-       // is it an address in the format user@server.tld?
-       /// @todo use gcontact and/or the addr field for a lookup
-       if (!strstr($url, "http") OR strstr($url, "@")) {
-               $data = probe_url($url);
-               $url = $data["url"];
-               if ($url == "")
-                       return 0;
-       }
+       $contact_id = 0;
 
-       $contact = q("SELECT `id`, `avatar-date` FROM `contact` WHERE `nurl` = '%s' AND `uid` = %d ORDER BY `id` LIMIT 2",
+       // We first try the nurl (http://server.tld/nick), most common case
+       $contacts = q("SELECT `id`, `avatar-date` FROM `contact`
+                                       WHERE `nurl` = '%s'
+                                       AND `uid` = %d",
                        dbesc(normalise_link($url)),
                        intval($uid));
 
-       if (!$contact)
-               $contact = q("SELECT `id`, `avatar-date` FROM `contact` WHERE `alias` IN ('%s', '%s') AND `uid` = %d ORDER BY `id` LIMIT 1",
-                               dbesc($url),
-                               dbesc(normalise_link($url)),
-                               intval($uid));
 
-       if ($contact) {
-               $contactid = $contact[0]["id"];
+       // Then the addr (nick@server.tld)
+       if (! dbm::is_result($contacts)) {
+               $contacts = q("SELECT `id`, `avatar-date` FROM `contact`
+                                       WHERE `addr` = '%s'
+                                       AND `uid` = %d",
+                       dbesc($url),
+                       intval($uid));
+       }
+
+       // Then the alias (which could be anything)
+       if (! dbm::is_result($contacts)) {
+               $contacts = q("SELECT `id`, `avatar-date` FROM `contact`
+                                       WHERE `alias` IN ('%s', '%s')
+                                       AND `uid` = %d",
+                       dbesc($url),
+                       dbesc(normalise_link($url)),
+                       intval($uid));
+       }
+
+       if (dbm::is_result($contacts)) {
+               $contact_id = $contacts[0]["id"];
 
                // Update the contact every 7 days
-               $update_photo = ($contact[0]['avatar-date'] < datetime_convert('','','now -7 days'));
-               //$update_photo = ($contact[0]['avatar-date'] < datetime_convert('','','now -12 hours'));
+               $update_photo = ($contacts[0]['avatar-date'] < datetime_convert('','','now -7 days'));
 
                if (!$update_photo OR $no_update) {
-                       return($contactid);
+                       return $contact_id;
                }
-       } elseif ($uid != 0)
+       } elseif ($uid != 0) {
+               // Non-existing user-specific contact, exiting
                return 0;
+       }
 
-       if (!count($data))
-               $data = probe_url($url);
+       require_once('include/Probe.php');
+       $data = Probe::uri($url);
 
-       // Does this address belongs to a valid network?
-       if (!in_array($data["network"], array(NETWORK_DFRN, NETWORK_OSTATUS, NETWORK_DIASPORA))) {
-               if ($uid != 0)
+       // Last try in gcontact for unsupported networks
+       if (!in_array($data["network"], array(NETWORK_DFRN, NETWORK_OSTATUS, NETWORK_DIASPORA, NETWORK_PUMPIO))) {
+               if ($uid != 0) {
                        return 0;
+               }
 
                // Get data from the gcontact table
-               $r = q("SELECT `name`, `nick`, `url`, `photo`, `addr`, `alias`, `network` FROM `gcontact` WHERE `nurl` = '%s'",
+               $gcontacts = q("SELECT `name`, `nick`, `url`, `photo`, `addr`, `alias`, `network` FROM `gcontact` WHERE `nurl` = '%s'",
                         dbesc(normalise_link($url)));
-               if (!$r)
+               if (!$gcontacts) {
                        return 0;
+               }
 
-               $data = $r[0];
+               $data = $gcontacts[0];
        }
 
        $url = $data["url"];
 
-       if ($contactid == 0) {
+       if (!$contact_id) {
                q("INSERT INTO `contact` (`uid`, `created`, `url`, `nurl`, `addr`, `alias`, `notify`, `poll`,
                                        `name`, `nick`, `photo`, `network`, `pubkey`, `rel`, `priority`,
                                        `batch`, `request`, `confirm`, `poco`, `name-date`, `uri-date`,
@@ -602,45 +627,48 @@ function get_contact($url, $uid = 0, $no_update = false) {
                        dbesc(datetime_convert())
                );
 
-               $contact = q("SELECT `id` FROM `contact` WHERE `nurl` = '%s' AND `uid` = %d ORDER BY `id` LIMIT 2",
+               $contacts = q("SELECT `id` FROM `contact` WHERE `nurl` = '%s' AND `uid` = %d ORDER BY `id` LIMIT 2",
                                dbesc(normalise_link($data["url"])),
                                intval($uid));
-               if (!$contact)
+               if (!dbm::is_result($contacts)) {
                        return 0;
+               }
 
-               $contactid = $contact[0]["id"];
+               $contact_id = $contacts[0]["id"];
 
                // Update the newly created contact from data in the gcontact table
-               $r = q("SELECT `location`, `about`, `keywords`, `gender` FROM `gcontact` WHERE `nurl` = '%s'",
+               $gcontacts = q("SELECT `location`, `about`, `keywords`, `gender` FROM `gcontact` WHERE `nurl` = '%s'",
                         dbesc(normalise_link($data["url"])));
-               if ($r) {
-                       logger("Update contact ".$data["url"]);
+               if (dbm::is_result($gcontacts)) {
+                       logger("Update contact " . $data["url"] . ' from gcontact');
                        q("UPDATE `contact` SET `location` = '%s', `about` = '%s', `keywords` = '%s', `gender` = '%s' WHERE `id` = %d",
-                               dbesc($r["location"]), dbesc($r["about"]), dbesc($r["keywords"]),
-                               dbesc($r["gender"]), intval($contactid));
+                               dbesc($gcontacts[0]["location"]), dbesc($gcontacts[0]["about"]), dbesc($gcontacts[0]["keywords"]),
+                               dbesc($gcontacts[0]["gender"]), intval($contact_id));
                }
        }
 
-       if ((count($contact) > 1) AND ($uid == 0) AND ($contactid != 0) AND ($url != ""))
+       if (count($contacts) > 1 AND $uid == 0 AND $contact_id != 0 AND $url != "") {
                q("DELETE FROM `contact` WHERE `nurl` = '%s' AND `id` != %d AND NOT `self`",
                        dbesc(normalise_link($url)),
-                       intval($contactid));
+                       intval($contact_id));
+       }
 
-       require_once("Photo.php");
+       require_once "Photo.php";
 
-       update_contact_avatar($data["photo"],$uid,$contactid);
+       update_contact_avatar($data["photo"], $uid, $contact_id);
 
-       $r = q("SELECT `addr`, `alias`, `name`, `nick` FROM `contact`  WHERE `id` = %d", intval($contactid));
+       $contacts = q("SELECT `addr`, `alias`, `name`, `nick` FROM `contact` WHERE `id` = %d", intval($contact_id));
 
        // This condition should always be true
-       if (!dbm::is_result($r))
-               return $contactid;
+       if (!dbm::is_result($contacts)) {
+               return $contact_id;
+       }
 
        // Only update if there had something been changed
-       if (($data["addr"] != $r[0]["addr"]) OR
-               ($data["alias"] != $r[0]["alias"]) OR
-               ($data["name"] != $r[0]["name"]) OR
-               ($data["nick"] != $r[0]["nick"]))
+       if ($data["addr"] != $contacts[0]["addr"] OR
+               $data["alias"] != $contacts[0]["alias"] OR
+               $data["name"] != $contacts[0]["name"] OR
+               $data["nick"] != $contacts[0]["nick"]) {
                q("UPDATE `contact` SET `addr` = '%s', `alias` = '%s', `name` = '%s', `nick` = '%s',
                        `name-date` = '%s', `uri-date` = '%s' WHERE `id` = %d",
                        dbesc($data["addr"]),
@@ -649,10 +677,11 @@ function get_contact($url, $uid = 0, $no_update = false) {
                        dbesc($data["nick"]),
                        dbesc(datetime_convert()),
                        dbesc(datetime_convert()),
-                       intval($contactid)
+                       intval($contact_id)
                );
+       }
 
-       return $contactid;
+       return $contact_id;
 }
 
 /**
@@ -675,15 +704,6 @@ function posts_from_gcontact(App $a, $gcontact_id) {
        else
                $sql = "`item`.`uid` = %d";
 
-       if(get_config('system', 'old_pager')) {
-               $r = q("SELECT COUNT(*) AS `total` FROM `item`
-                       WHERE `gcontact-id` = %d and $sql",
-                       intval($gcontact_id),
-                       intval(local_user()));
-
-               $a->set_pager_total($r[0]['total']);
-       }
-
        $r = q("SELECT `item`.`uri`, `item`.*, `item`.`id` AS `item_id`,
                        `author-name` AS `name`, `owner-avatar` AS `photo`,
                        `owner-link` AS `url`, `owner-avatar` AS `thumb`
@@ -697,13 +717,9 @@ function posts_from_gcontact(App $a, $gcontact_id) {
                intval($a->pager['itemspage'])
        );
 
-       $o = conversation($a,$r,'community',false);
+       $o = conversation($a, $r, 'community', false);
 
-       if(!get_config('system', 'old_pager')) {
-               $o .= alt_pager($a,count($r));
-       } else {
-               $o .= paginate($a);
-       }
+       $o .= alt_pager($a, count($r));
 
        return $o;
 }
@@ -736,15 +752,6 @@ function posts_from_contact_url(App $a, $contact_url) {
 
        $author_id = intval($r[0]["author-id"]);
 
-       if (get_config('system', 'old_pager')) {
-               $r = q("SELECT COUNT(*) AS `total` FROM `item`
-                       WHERE `author-id` = %d and $sql",
-                       intval($author_id),
-                       intval(local_user()));
-
-               $a->set_pager_total($r[0]['total']);
-       }
-
        $r = q(item_query()." AND `item`.`author-id` = %d AND ".$sql.
                " ORDER BY `item`.`created` DESC LIMIT %d, %d",
                intval($author_id),
@@ -753,13 +760,9 @@ function posts_from_contact_url(App $a, $contact_url) {
                intval($a->pager['itemspage'])
        );
 
-       $o = conversation($a,$r,'community',false);
+       $o = conversation($a, $r, 'community', false);
 
-       if (!get_config('system', 'old_pager')) {
-               $o .= alt_pager($a,count($r));
-       } else {
-               $o .= paginate($a);
-       }
+       $o .= alt_pager($a, count($r));
 
        return $o;
 }
@@ -774,18 +777,18 @@ function posts_from_contact_url(App $a, $contact_url) {
 function formatted_location($profile) {
        $location = '';
 
-       if($profile['locality'])
+       if ($profile['locality'])
                $location .= $profile['locality'];
 
-       if($profile['region'] AND ($profile['locality'] != $profile['region'])) {
-               if($location)
+       if ($profile['region'] AND ($profile['locality'] != $profile['region'])) {
+               if ($location)
                        $location .= ', ';
 
                $location .= $profile['region'];
        }
 
-       if($profile['country-name']) {
-               if($location)
+       if ($profile['country-name']) {
+               if ($location)
                        $location .= ', ';
 
                $location .= $profile['country-name'];
@@ -807,7 +810,7 @@ function account_type($contact) {
        // "page-flags" is a field in the user table,
        // "forum" and "prv" are used in the contact table. They stand for PAGE_COMMUNITY and PAGE_PRVGROUP.
        // "community" is used in the gcontact table and is true if the contact is PAGE_COMMUNITY or PAGE_PRVGROUP.
-       if((isset($contact['page-flags']) && (intval($contact['page-flags']) == PAGE_COMMUNITY))
+       if ((isset($contact['page-flags']) && (intval($contact['page-flags']) == PAGE_COMMUNITY))
                || (isset($contact['page-flags']) && (intval($contact['page-flags']) == PAGE_PRVGROUP))
                || (isset($contact['forum']) && intval($contact['forum']))
                || (isset($contact['prv']) && intval($contact['prv']))