Follow up for PR 3421 - some improvements
This commit is contained in:
parent
f639a86d02
commit
99a54df8d8
1 changed files with 17 additions and 12 deletions
|
@ -344,13 +344,13 @@ class Diaspora {
|
||||||
if (dbm::is_result($r)) {
|
if (dbm::is_result($r)) {
|
||||||
foreach ($r as $rr) {
|
foreach ($r as $rr) {
|
||||||
logger("delivering to: ".$rr["username"]);
|
logger("delivering to: ".$rr["username"]);
|
||||||
self::dispatch($rr,$msg);
|
self::dispatch($rr, $msg);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
// Use a dummy importer to import the data for the public copy
|
// Use a dummy importer to import the data for the public copy
|
||||||
// or for comments from unknown people
|
// or for comments from unknown people
|
||||||
$importer = array("uid" => 0, "page-flags" => PAGE_FREELOVE);
|
$importer = array("uid" => 0, "page-flags" => PAGE_FREELOVE);
|
||||||
$message_id = self::dispatch($importer,$msg);
|
$message_id = self::dispatch($importer, $msg);
|
||||||
}
|
}
|
||||||
|
|
||||||
return $message_id;
|
return $message_id;
|
||||||
|
@ -378,7 +378,7 @@ class Diaspora {
|
||||||
$type = $fields->getName();
|
$type = $fields->getName();
|
||||||
|
|
||||||
$social_relay = Config::get('system', 'relay_subscribe', false);
|
$social_relay = Config::get('system', 'relay_subscribe', false);
|
||||||
if (!$social_relay AND ($type == 'message')) {
|
if (!$social_relay AND (in_array($type, array('status_message', 'reshare')))) {
|
||||||
logger("Unwanted message from ".$sender." send by ".$_SERVER["REMOTE_ADDR"]." with ".$_SERVER["HTTP_USER_AGENT"].": ".print_r($msg, true), LOGGER_DEBUG);
|
logger("Unwanted message from ".$sender." send by ".$_SERVER["REMOTE_ADDR"]." with ".$_SERVER["HTTP_USER_AGENT"].": ".print_r($msg, true), LOGGER_DEBUG);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -828,17 +828,17 @@ class Diaspora {
|
||||||
logger("defining user ".$contact["nick"]." as friend");
|
logger("defining user ".$contact["nick"]." as friend");
|
||||||
}
|
}
|
||||||
|
|
||||||
if (($contact["blocked"]) || ($contact["readonly"]) || ($contact["archive"]))
|
if ($contact["blocked"] || $contact["readonly"] || $contact["archive"]) {
|
||||||
return false;
|
return false;
|
||||||
if ($contact["rel"] == CONTACT_IS_SHARING || $contact["rel"] == CONTACT_IS_FRIEND)
|
} elseif (($contact["rel"] == CONTACT_IS_SHARING) || ($contact["rel"] == CONTACT_IS_FRIEND)) {
|
||||||
return true;
|
return true;
|
||||||
if ($contact["rel"] == CONTACT_IS_FOLLOWER)
|
} elseif (($contact["rel"] == CONTACT_IS_FOLLOWER) && ($importer["page-flags"] == PAGE_COMMUNITY)) {
|
||||||
if (($importer["page-flags"] == PAGE_COMMUNITY) OR $is_comment)
|
|
||||||
return true;
|
|
||||||
|
|
||||||
// Messages for the global users are always accepted
|
|
||||||
if ($importer["uid"] == 0)
|
|
||||||
return true;
|
return true;
|
||||||
|
}
|
||||||
|
// Messages for the global users and comments are always accepted
|
||||||
|
if (($importer["uid"] == 0) || $is_comment) {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
@ -1240,7 +1240,7 @@ class Diaspora {
|
||||||
|
|
||||||
if (dbm::is_result($item)) {
|
if (dbm::is_result($item)) {
|
||||||
logger("Found user ".$item['uid']." as owner of item ".$guid, LOGGER_DEBUG);
|
logger("Found user ".$item['uid']." as owner of item ".$guid, LOGGER_DEBUG);
|
||||||
$contact = dba::fetch_first("SELECT * FROM `contact` WHERE `self` AND `uid` = ?", $item['uid']);
|
$contact = dba::fetch_first("SELECT `uid`, `page-flags` FROM `contact` WHERE `self` AND `uid` = ?", $item['uid']);
|
||||||
if (dbm::is_result($contact)) {
|
if (dbm::is_result($contact)) {
|
||||||
$importer = $contact;
|
$importer = $contact;
|
||||||
}
|
}
|
||||||
|
@ -1626,6 +1626,11 @@ class Diaspora {
|
||||||
if (!in_array($parent_type, array("Post", "Comment")))
|
if (!in_array($parent_type, array("Post", "Comment")))
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
|
// Find the best importer when there was no importer found
|
||||||
|
if ($importer["uid"] == 0) {
|
||||||
|
$importer = self::importer_for_comment($importer, $parent_guid);
|
||||||
|
}
|
||||||
|
|
||||||
$contact = self::allowed_contact_by_handle($importer, $sender, true);
|
$contact = self::allowed_contact_by_handle($importer, $sender, true);
|
||||||
if (!$contact)
|
if (!$contact)
|
||||||
return false;
|
return false;
|
||||||
|
|
Loading…
Reference in a new issue