]> git.mxchange.org Git - friendica.git/blobdiff - include/socgraph.php
Merge remote-tracking branch 'upstream/develop' into rewrites/coding-convention-split...
[friendica.git] / include / socgraph.php
index 953bd0da784be9fb65d99e18525190e66b509884..7fa73a45ca2d8ce572d734bd18192dba890e7553 100644 (file)
@@ -150,7 +150,7 @@ function poco_load_worker($cid, $uid, $zcid, $url) {
                        $gender = $entry->gender;
                }
 
-               if (isset($entry->generation) AND ($entry->generation > 0)) {
+               if (isset($entry->generation) && ($entry->generation > 0)) {
                        $generation = ++$entry->generation;
                }
 
@@ -160,7 +160,7 @@ function poco_load_worker($cid, $uid, $zcid, $url) {
                        }
                }
 
-               if (isset($entry->contactType) AND ($entry->contactType >= 0))
+               if (isset($entry->contactType) && ($entry->contactType >= 0))
                        $contact_type = $entry->contactType;
 
                $gcontact = array("url" => $profile_url,
@@ -238,7 +238,7 @@ function sanitize_gcontact($gcontact) {
        $alternate = poco_alternate_ostatus_url($gcontact['url']);
 
        // The global contacts should contain the original picture, not the cached one
-       if (($gcontact['generation'] != 1) AND stristr(normalise_link($gcontact['photo']), normalise_link(App::get_baseurl()."/photo/"))) {
+       if (($gcontact['generation'] != 1) && stristr(normalise_link($gcontact['photo']), normalise_link(App::get_baseurl()."/photo/"))) {
                $gcontact['photo'] = "";
        }
 
@@ -250,7 +250,7 @@ function sanitize_gcontact($gcontact) {
                        $gcontact['network'] = $r[0]["network"];
                }
 
-               if (($gcontact['network'] == "") OR ($gcontact['network'] == NETWORK_OSTATUS)) {
+               if (($gcontact['network'] == "") || ($gcontact['network'] == NETWORK_OSTATUS)) {
                        $r = q("SELECT `network`, `url` FROM `contact` WHERE `uid` = 0 AND `alias` IN ('%s', '%s') AND `network` != '' AND `network` != '%s' LIMIT 1",
                                dbesc($gcontact['url']), dbesc(normalise_link($gcontact['url'])), dbesc(NETWORK_STATUSNET)
                        );
@@ -268,13 +268,13 @@ function sanitize_gcontact($gcontact) {
        );
 
        if (count($x)) {
-               if (!isset($gcontact['network']) AND ($x[0]["network"] != NETWORK_STATUSNET)) {
+               if (!isset($gcontact['network']) && ($x[0]["network"] != NETWORK_STATUSNET)) {
                        $gcontact['network'] = $x[0]["network"];
                }
                if ($gcontact['updated'] <= NULL_DATE) {
                        $gcontact['updated'] = $x[0]["updated"];
                }
-               if (!isset($gcontact['server_url']) AND (normalise_link($x[0]["server_url"]) != normalise_link($x[0]["url"]))) {
+               if (!isset($gcontact['server_url']) && (normalise_link($x[0]["server_url"]) != normalise_link($x[0]["url"]))) {
                        $gcontact['server_url'] = $x[0]["server_url"];
                }
                if (!isset($gcontact['addr'])) {
@@ -282,8 +282,8 @@ function sanitize_gcontact($gcontact) {
                }
        }
 
-       if ((!isset($gcontact['network']) OR !isset($gcontact['name']) OR !isset($gcontact['addr']) OR !isset($gcontact['photo']) OR !isset($gcontact['server_url']) OR $alternate)
-               AND poco_reachable($gcontact['url'], $gcontact['server_url'], $gcontact['network'], false)) {
+       if ((!isset($gcontact['network']) || !isset($gcontact['name']) || !isset($gcontact['addr']) || !isset($gcontact['photo']) || !isset($gcontact['server_url']) || $alternate)
+               && poco_reachable($gcontact['url'], $gcontact['server_url'], $gcontact['network'], false)) {
                $data = Probe::uri($gcontact['url']);
 
                if ($data["network"] == NETWORK_PHANTOM) {
@@ -296,7 +296,7 @@ function sanitize_gcontact($gcontact) {
 
                $gcontact = array_merge($gcontact, $data);
 
-               if ($alternate AND ($gcontact['network'] == NETWORK_OSTATUS)) {
+               if ($alternate && ($gcontact['network'] == NETWORK_OSTATUS)) {
                        // Delete the old entry - if it exists
                        $r = q("SELECT `id` FROM `gcontact` WHERE `nurl` = '%s'", dbesc(normalise_link($orig_profile)));
                        if ($r) {
@@ -306,7 +306,7 @@ function sanitize_gcontact($gcontact) {
                }
        }
 
-       if (!isset($gcontact['name']) OR !isset($gcontact['photo'])) {
+       if (!isset($gcontact['name']) || !isset($gcontact['photo'])) {
                throw new Exception('No name and photo for URL '.$gcontact['url']);
        }
 
@@ -483,11 +483,11 @@ function poco_last_updated($profile, $force = false) {
                $server_url = normalise_link(poco_detect_server($profile));
        }
 
-       if (($server_url == '') AND ($gcontacts[0]["server_url"] != "")) {
+       if (($server_url == '') && ($gcontacts[0]["server_url"] != "")) {
                $server_url = $gcontacts[0]["server_url"];
        }
 
-       if (!$force AND (($server_url == '') OR ($gcontacts[0]["server_url"] == $gcontacts[0]["nurl"]))) {
+       if (!$force && (($server_url == '') || ($gcontacts[0]["server_url"] == $gcontacts[0]["nurl"]))) {
                $server_url = normalise_link(poco_detect_server($profile));
        }
 
@@ -521,7 +521,7 @@ function poco_last_updated($profile, $force = false) {
        }
 
        // noscrape is really fast so we don't cache the call.
-       if (($server_url != "") AND ($gcontacts[0]["nick"] != "")) {
+       if (($server_url != "") && ($gcontacts[0]["nick"] != "")) {
 
                //  Use noscrape if possible
                $server = q("SELECT `noscrape`, `network` FROM `gserver` WHERE `nurl` = '%s' AND `noscrape` != ''", dbesc(normalise_link($server_url)));
@@ -529,7 +529,7 @@ function poco_last_updated($profile, $force = false) {
                if ($server) {
                        $noscraperet = z_fetch_url($server[0]["noscrape"]."/".$gcontacts[0]["nick"]);
 
-                       if ($noscraperet["success"] AND ($noscraperet["body"] != "")) {
+                       if ($noscraperet["success"] && ($noscraperet["body"] != "")) {
 
                                $noscrape = json_decode($noscraperet["body"], true);
 
@@ -593,7 +593,7 @@ function poco_last_updated($profile, $force = false) {
        }
 
        // If we only can poll the feed, then we only do this once a while
-       if (!$force AND !poco_do_update($gcontacts[0]["created"], $gcontacts[0]["updated"], $gcontacts[0]["last_failure"], $gcontacts[0]["last_contact"])) {
+       if (!$force && !poco_do_update($gcontacts[0]["created"], $gcontacts[0]["updated"], $gcontacts[0]["last_failure"], $gcontacts[0]["last_contact"])) {
                logger("Profile ".$profile." was last updated at ".$gcontacts[0]["updated"]." (cached)", LOGGER_DEBUG);
 
                update_gcontact($contact);
@@ -604,8 +604,8 @@ function poco_last_updated($profile, $force = false) {
 
        // Is the profile link the alternate OStatus link notation? (http://domain.tld/user/4711)
        // Then check the other link and delete this one
-       if (($data["network"] == NETWORK_OSTATUS) AND poco_alternate_ostatus_url($profile) AND
-               (normalise_link($profile) == normalise_link($data["alias"])) AND
+       if (($data["network"] == NETWORK_OSTATUS) && poco_alternate_ostatus_url($profile) &&
+               (normalise_link($profile) == normalise_link($data["alias"])) &&
                (normalise_link($profile) != normalise_link($data["url"]))) {
 
                // Delete the old entry
@@ -629,7 +629,7 @@ function poco_last_updated($profile, $force = false) {
                return false;
        }
 
-       if (($data["poll"] == "") OR (in_array($data["network"], array(NETWORK_FEED, NETWORK_PHANTOM)))) {
+       if (($data["poll"] == "") || (in_array($data["network"], array(NETWORK_FEED, NETWORK_PHANTOM)))) {
                q("UPDATE `gcontact` SET `last_failure` = '%s' WHERE `nurl` = '%s'",
                        dbesc(datetime_convert()), dbesc(normalise_link($profile)));
 
@@ -717,24 +717,24 @@ function poco_do_update($created, $updated, $last_failure,  $last_contact) {
                return false;
 
        // If the last contact was less than a week ago and the last failure is older than a week then don't update
-       //if ((($now - $contact_time) < (60 * 60 * 24 * 7)) AND ($contact_time > $failure_time))
+       //if ((($now - $contact_time) < (60 * 60 * 24 * 7)) && ($contact_time > $failure_time))
        //      return false;
 
        // If the last contact time was more than a week ago and the contact was created more than a week ago, then only try once a week
-       if ((($now - $contact_time) > (60 * 60 * 24 * 7)) AND (($now - $created_time) > (60 * 60 * 24 * 7)) AND (($now - $failure_time) < (60 * 60 * 24 * 7)))
+       if ((($now - $contact_time) > (60 * 60 * 24 * 7)) && (($now - $created_time) > (60 * 60 * 24 * 7)) && (($now - $failure_time) < (60 * 60 * 24 * 7)))
                return false;
 
        // If the last contact time was more than a month ago and the contact was created more than a month ago, then only try once a month
-       if ((($now - $contact_time) > (60 * 60 * 24 * 30)) AND (($now - $created_time) > (60 * 60 * 24 * 30)) AND (($now - $failure_time) < (60 * 60 * 24 * 30)))
+       if ((($now - $contact_time) > (60 * 60 * 24 * 30)) && (($now - $created_time) > (60 * 60 * 24 * 30)) && (($now - $failure_time) < (60 * 60 * 24 * 30)))
                return false;
 
        return true;
 }
 
 function poco_to_boolean($val) {
-       if (($val == "true") OR ($val == 1))
+       if (($val == "true") || ($val == 1))
                return(true);
-       if (($val == "false") OR ($val == 0))
+       if (($val == "false") || ($val == 0))
                return(false);
 
        return ($val);
@@ -825,7 +825,7 @@ function poco_fetch_nodeinfo($server_url) {
 
        $server['register_policy'] = REGISTER_CLOSED;
 
-       if (is_bool($nodeinfo->openRegistrations) AND $nodeinfo->openRegistrations) {
+       if (is_bool($nodeinfo->openRegistrations) && $nodeinfo->openRegistrations) {
                $server['register_policy'] = REGISTER_OPEN;
        }
 
@@ -977,7 +977,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
                $info = $servers[0]["info"];
                $register_policy = $servers[0]["register_policy"];
 
-               if (!$force AND !poco_do_update($servers[0]["created"], "", $last_failure, $last_contact)) {
+               if (!$force && !poco_do_update($servers[0]["created"], "", $last_failure, $last_contact)) {
                        logger("Use cached data for server ".$server_url, LOGGER_DEBUG);
                        return ($last_contact >= $last_failure);
                }
@@ -1009,7 +1009,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
 
        // Quit if there is a timeout.
        // But we want to make sure to only quit if we are mostly sure that this server url fits.
-       if (dbm::is_result($servers) AND ($orig_server_url == $server_url) AND
+       if (dbm::is_result($servers) && ($orig_server_url == $server_url) &&
                ($serverret['errno'] == CURLE_OPERATION_TIMEDOUT)) {
                logger("Connection to server ".$server_url." timed out.", LOGGER_DEBUG);
                dba::p("UPDATE `gserver` SET `last_failure` = ? WHERE `nurl` = ?", datetime_convert(), normalise_link($server_url));
@@ -1018,7 +1018,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
 
        // Maybe the page is unencrypted only?
        $xmlobj = @simplexml_load_string($serverret["body"],'SimpleXMLElement',0, "http://docs.oasis-open.org/ns/xri/xrd-1.0");
-       if (!$serverret["success"] OR ($serverret["body"] == "") OR (@sizeof($xmlobj) == 0) OR !is_object($xmlobj)) {
+       if (!$serverret["success"] || ($serverret["body"] == "") || (@sizeof($xmlobj) == 0) || !is_object($xmlobj)) {
                $server_url = str_replace("https://", "http://", $server_url);
 
                // We set the timeout to 20 seconds since this operation should be done in no time if the server was vital
@@ -1034,7 +1034,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
                $xmlobj = @simplexml_load_string($serverret["body"],'SimpleXMLElement',0, "http://docs.oasis-open.org/ns/xri/xrd-1.0");
        }
 
-       if (!$serverret["success"] OR ($serverret["body"] == "") OR (sizeof($xmlobj) == 0) OR !is_object($xmlobj)) {
+       if (!$serverret["success"] || ($serverret["body"] == "") || (sizeof($xmlobj) == 0) || !is_object($xmlobj)) {
                // Workaround for bad configured servers (known nginx problem)
                if (!in_array($serverret["debug"]["http_code"], array("403", "404"))) {
                        $failure = true;
@@ -1073,7 +1073,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
                // Test for Diaspora, Hubzilla, Mastodon or older Friendica servers
                $serverret = z_fetch_url($server_url);
 
-               if (!$serverret["success"] OR ($serverret["body"] == "")) {
+               if (!$serverret["success"] || ($serverret["body"] == "")) {
                        $failure = true;
                } else {
                        $server = poco_detect_server_type($serverret["body"]);
@@ -1106,13 +1106,13 @@ function poco_check_server($server_url, $network = "", $force = false) {
                }
        }
 
-       if (!$failure AND ($poco == "")) {
+       if (!$failure && ($poco == "")) {
                // Test for Statusnet
                // Will also return data for Friendica and GNU Social - but it will be overwritten later
                // The "not implemented" is a special treatment for really, really old Friendica versions
                $serverret = z_fetch_url($server_url."/api/statusnet/version.json");
-               if ($serverret["success"] AND ($serverret["body"] != '{"error":"not implemented"}') AND
-                       ($serverret["body"] != '') AND (strlen($serverret["body"]) < 30)) {
+               if ($serverret["success"] && ($serverret["body"] != '{"error":"not implemented"}') &&
+                       ($serverret["body"] != '') && (strlen($serverret["body"]) < 30)) {
                        $platform = "StatusNet";
                        // Remove junk that some GNU Social servers return
                        $version = str_replace(chr(239).chr(187).chr(191), "", $serverret["body"]);
@@ -1122,8 +1122,8 @@ function poco_check_server($server_url, $network = "", $force = false) {
 
                // Test for GNU Social
                $serverret = z_fetch_url($server_url."/api/gnusocial/version.json");
-               if ($serverret["success"] AND ($serverret["body"] != '{"error":"not implemented"}') AND
-                       ($serverret["body"] != '') AND (strlen($serverret["body"]) < 30)) {
+               if ($serverret["success"] && ($serverret["body"] != '{"error":"not implemented"}') &&
+                       ($serverret["body"] != '') && (strlen($serverret["body"]) < 30)) {
                        $platform = "GNU Social";
                        // Remove junk that some GNU Social servers return
                        $version = str_replace(chr(239).chr(187).chr(191), "", $serverret["body"]);
@@ -1133,7 +1133,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
 
                // Test for Mastodon
                $serverret = z_fetch_url($server_url."/api/v1/instance");
-               if ($serverret["success"] AND ($serverret["body"] != '')) {
+               if ($serverret["success"] && ($serverret["body"] != '')) {
                        $data = json_decode($serverret["body"]);
                        if (isset($data->version)) {
                                $platform = "Mastodon";
@@ -1188,9 +1188,9 @@ function poco_check_server($server_url, $network = "", $force = false) {
                                $data->site->private = poco_to_boolean($data->site->private);
                                $data->site->inviteonly = poco_to_boolean($data->site->inviteonly);
 
-                               if (!$data->site->closed AND !$data->site->private and $data->site->inviteonly) {
+                               if (!$data->site->closed && !$data->site->private and $data->site->inviteonly) {
                                        $register_policy = REGISTER_APPROVE;
-                               } elseif (!$data->site->closed AND !$data->site->private) {
+                               } elseif (!$data->site->closed && !$data->site->private) {
                                        $register_policy = REGISTER_OPEN;
                                } else {
                                        $register_policy = REGISTER_CLOSED;
@@ -1255,7 +1255,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
 
        // Check for noscrape
        // Friendica servers could be detected as OStatus servers
-       if (!$failure AND in_array($network, array(NETWORK_DFRN, NETWORK_OSTATUS))) {
+       if (!$failure && in_array($network, array(NETWORK_DFRN, NETWORK_OSTATUS))) {
                $serverret = z_fetch_url($server_url."/friendica/json");
 
                if (!$serverret["success"]) {
@@ -1290,7 +1290,7 @@ function poco_check_server($server_url, $network = "", $force = false) {
                }
        }
 
-       if ($possible_failure AND !$failure) {
+       if ($possible_failure && !$failure) {
                $failure = true;
        }
 
@@ -1302,9 +1302,9 @@ function poco_check_server($server_url, $network = "", $force = false) {
                $last_failure = $orig_last_failure;
        }
 
-       if (($last_contact <= $last_failure) AND !$failure) {
+       if (($last_contact <= $last_failure) && !$failure) {
                logger("Server ".$server_url." seems to be alive, but last contact wasn't set - could be a bug", LOGGER_DEBUG);
-       } elseif (($last_contact >= $last_failure) AND $failure) {
+       } else if (($last_contact >= $last_failure) && $failure) {
                logger("Server ".$server_url." seems to be dead, but last failure wasn't set - could be a bug", LOGGER_DEBUG);
        }
 
@@ -1746,7 +1746,7 @@ function poco_discover_single_server($id) {
                                $success = poco_discover_server(json_decode($retdata["body"]));
                        }
 
-                       if (!$success AND (get_config('system','poco_discovery') > 2)) {
+                       if (!$success && (get_config('system','poco_discovery') > 2)) {
                                logger("Fetch contacts from users of the server ".$server["nurl"], LOGGER_DEBUG);
                                poco_discover_server_users($data, $server);
                        }
@@ -1793,7 +1793,7 @@ function poco_discover($complete = false) {
                        logger('Update directory from server '.$server['url'].' with ID '.$server['id'], LOGGER_DEBUG);
                        proc_run(PRIORITY_LOW, "include/discover_poco.php", "update_server_directory", intval($server['id']));
 
-                       if (!$complete AND (--$no_of_queries == 0)) {
+                       if (!$complete && (--$no_of_queries == 0)) {
                                break;
                        }
                }
@@ -1833,7 +1833,7 @@ function poco_discover_server_users($data, $server) {
 
 function poco_discover_server($data, $default_generation = 0) {
 
-       if (!isset($data->entry) OR !count($data->entry)) {
+       if (!isset($data->entry) || !count($data->entry)) {
                return false;
        }
 
@@ -1897,11 +1897,11 @@ function poco_discover_server($data, $default_generation = 0) {
                        $gender = $entry->gender;
                }
 
-               if (isset($entry->generation) AND ($entry->generation > 0)) {
+               if(isset($entry->generation) && ($entry->generation > 0)) {
                        $generation = ++$entry->generation;
                }
 
-               if (isset($entry->contactType) AND ($entry->contactType >= 0)) {
+               if(isset($entry->contactType) && ($entry->contactType >= 0)) {
                        $contact_type = $entry->contactType;
                }
 
@@ -1951,7 +1951,7 @@ function poco_discover_server($data, $default_generation = 0) {
 function clean_contact_url($url) {
        $parts = parse_url($url);
 
-       if (!isset($parts["scheme"]) OR !isset($parts["host"])) {
+       if (!isset($parts["scheme"]) || !isset($parts["host"])) {
                return $url;
        }
 
@@ -1978,7 +1978,7 @@ function clean_contact_url($url) {
  * @param arr $contact contact array (called by reference)
  */
 function fix_alternate_contact_address(&$contact) {
-       if (($contact["network"] == NETWORK_OSTATUS) AND poco_alternate_ostatus_url($contact["url"])) {
+       if (($contact["network"] == NETWORK_OSTATUS) && poco_alternate_ostatus_url($contact["url"])) {
                $data = probe_url($contact["url"]);
                if ($contact["network"] == NETWORK_OSTATUS) {
                        logger("Fix primary url from ".$contact["url"]." to ".$data["url"]." - Called by: ".App::callstack(), LOGGER_DEBUG);
@@ -2023,7 +2023,8 @@ function get_gcontact_id($contact) {
                $contact["url"] = clean_contact_url($contact["url"]);
        }
 
-       $r = q("SELECT `id`, `last_contact`, `last_failure`, `network` FROM `gcontact` WHERE `nurl` = '%s' LIMIT 2",
+       dba::lock('gcontact');
+       $r = q("SELECT `id`, `last_contact`, `last_failure`, `network` FROM `gcontact` WHERE `nurl` = '%s' LIMIT 1",
                dbesc(normalise_link($contact["url"])));
 
        if (dbm::is_result($r)) {
@@ -2035,7 +2036,7 @@ function get_gcontact_id($contact) {
                        $last_failure = strtotime($r[0]["last_failure"]);
                        $last_contact_str = $r[0]["last_contact"];
                        $last_contact = strtotime($r[0]["last_contact"]);
-                       $doprobing = (((time() - $last_contact) > (90 * 86400)) AND ((time() - $last_failure) > (90 * 86400)));
+                       $doprobing = (((time() - $last_contact) > (90 * 86400)) && ((time() - $last_failure) > (90 * 86400)));
                }
        } else {
                q("INSERT INTO `gcontact` (`name`, `nick`, `addr` , `network`, `url`, `nurl`, `photo`, `created`, `updated`, `location`, `about`, `hide`, `generation`)
@@ -2064,18 +2065,13 @@ function get_gcontact_id($contact) {
                        $doprobing = in_array($r[0]["network"], array(NETWORK_DFRN, NETWORK_DIASPORA, NETWORK_OSTATUS, ""));
                }
        }
+       dba::unlock();
 
        if ($doprobing) {
                logger("Last Contact: ". $last_contact_str." - Last Failure: ".$last_failure_str." - Checking: ".$contact["url"], LOGGER_DEBUG);
                proc_run(PRIORITY_LOW, 'include/gprobe.php', bin2hex($contact["url"]));
        }
 
-       if ((dbm::is_result($r)) AND (count($r) > 1) AND ($gcontact_id > 0) AND ($contact["url"] != "")) {
-               q("DELETE FROM `gcontact` WHERE `nurl` = '%s' AND `id` != %d",
-                       dbesc(normalise_link($contact["url"])),
-                       intval($gcontact_id));
-       }
-
        return $gcontact_id;
 }
 
@@ -2088,7 +2084,7 @@ function get_gcontact_id($contact) {
 function update_gcontact($contact) {
 
        // Check for invalid "contact-type" value
-       if (isset($contact['contact-type']) AND (intval($contact['contact-type']) < 0)) {
+       if (isset($contact['contact-type']) && (intval($contact['contact-type']) < 0)) {
                $contact['contact-type'] = 0;
        }
 
@@ -2126,8 +2122,8 @@ function update_gcontact($contact) {
        }
 
        // assign all unassigned fields from the database entry
-       foreach ($fields AS $field => $data) {
-               if (!isset($contact[$field]) OR ($contact[$field] == "")) {
+       foreach ($fields as $field => $data)
+               if (!isset($contact[$field]) || ($contact[$field] == "")) {
                        $contact[$field] = $r[0][$field];
                }
        }
@@ -2170,7 +2166,7 @@ function update_gcontact($contact) {
                $contact["server_url"] = normalise_link($contact["server_url"]);
        }
 
-       if (($contact["addr"] == "") AND ($contact["server_url"] != "") AND ($contact["nick"] != "")) {
+       if (($contact["addr"] == "") && ($contact["server_url"] != "") && ($contact["nick"] != "")) {
                $hostname = str_replace("http://", "", $contact["server_url"]);
                $contact["addr"] = $contact["nick"]."@".$hostname;
        }
@@ -2179,7 +2175,7 @@ function update_gcontact($contact) {
        $update = false;
        unset($fields["generation"]);
 
-       if ((($contact["generation"] > 0) AND ($contact["generation"] <= $r[0]["generation"])) OR ($r[0]["generation"] == 0)) {
+       if ((($contact["generation"] > 0) && ($contact["generation"] <= $r[0]["generation"])) || ($r[0]["generation"] == 0)) {
                foreach ($fields AS $field => $data) {
                        if ($contact[$field] != $r[0][$field]) {
                                logger("Difference for contact ".$contact["url"]." in field '".$field."'. New value: '".$contact[$field]."', old value '".$r[0][$field]."'", LOGGER_DEBUG);
@@ -2286,7 +2282,7 @@ function update_gcontact_for_user($uid) {
                        "gender" => $r[0]["gender"], "keywords" => $r[0]["pub_keywords"],
                        "birthday" => $r[0]["dob"], "photo" => $r[0]["photo"],
                        "notify" => $r[0]["notify"], "url" => $r[0]["url"],
-                       "hide" => ($r[0]["hidewall"] OR !$r[0]["net-publish"]),
+                       "hide" => ($r[0]["hidewall"] || !$r[0]["net-publish"]),
                        "nick" => $r[0]["nickname"], "addr" => $addr,
                        "connect" => $addr, "server_url" => App::get_baseurl(),
                        "generation" => 1, "network" => NETWORK_DFRN);