friendica
/
friendica
mirror of https://github.com/friendica/friendica
6
0
Fork 1
Code Issues Packages Projects Releases 1 Wiki Activity

Merge pull request #5759 from annando/issue-5733 

Issue 5733: Removing contacts does work now on the same machine as well
This commit is contained in:
Hypolite Petovan 2018-09-12 19:40:03 -04:00 committed by GitHub
parent f3a97a0828 c4825a8b53
commit 70c94abd8e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 15 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
mod/contacts.php
View File

@ -47,6 +47,11 @@ function contacts_init(App $a)
if (!DBA::isResult($contact)) { if (!DBA::isResult($contact)) {
$contact = DBA::selectFirst('contact', [], ['id' => $contact_id, 'uid' => 0]); $contact = DBA::selectFirst('contact', [], ['id' => $contact_id, 'uid' => 0]);
} }
// Don't display contacts that are about to be deleted
if ($contact['network'] == Protocol::PHANTOM) {
$contact = false;
}
} }
if (DBA::isResult($contact)) { if (DBA::isResult($contact)) {
@ -719,6 +724,8 @@ function contacts_content(App $a, $update = 0)
$sql_extra = " AND `blocked` = 0 "; $sql_extra = " AND `blocked` = 0 ";
} }
$sql_extra .= sprintf(" AND `network` != '%s' ", Protocol::PHANTOM);
$search = x($_GET, 'search') ? notags(trim($_GET['search'])) : ''; $search = x($_GET, 'search') ? notags(trim($_GET['search'])) : '';
$nets = x($_GET, 'nets' ) ? notags(trim($_GET['nets'])) : ''; $nets = x($_GET, 'nets' ) ? notags(trim($_GET['nets'])) : '';

12
mod/dfrn_notify.php
View File

@ -79,13 +79,13 @@ function dfrn_notify_post(App $a) {
$condition = []; $condition = [];
switch ($direction) { switch ($direction) {
case (-1): case (-1):
$condition = ["`issued-id` = ? OR `dfrn-id` = ?", $dfrn_id, $dfrn_id]; $condition = ["(`issued-id` = ? OR `dfrn-id` = ?) AND `uid` = ?", $dfrn_id, $dfrn_id, $user['uid']];
break; break;
case 0: case 0:
$condition = ['issued-id' => $dfrn_id, 'duplex' => true]; $condition = ['issued-id' => $dfrn_id, 'duplex' => true, 'uid' => $user['uid']];
break; break;
case 1: case 1:
$condition = ['dfrn-id' => $dfrn_id, 'duplex' => true]; $condition = ['dfrn-id' => $dfrn_id, 'duplex' => true, 'uid' => $user['uid']];
break; break;
default: default:
System::xmlExit(3, 'Invalid direction'); System::xmlExit(3, 'Invalid direction');
@ -287,15 +287,15 @@ function dfrn_notify_content(App $a) {
$condition = []; $condition = [];
switch ($direction) { switch ($direction) {
case (-1): case (-1):
$condition = ["`issued-id` = ? OR `dfrn-id` = ?", $dfrn_id, $dfrn_id]; $condition = ["(`issued-id` = ? OR `dfrn-id` = ?) AND `uid` = ?", $dfrn_id, $dfrn_id, $user['uid']];
$my_id = $dfrn_id; $my_id = $dfrn_id;
break; break;
case 0: case 0:
$condition = ['issued-id' => $dfrn_id, 'duplex' => true]; $condition = ['issued-id' => $dfrn_id, 'duplex' => true, 'uid' => $user['uid']];
$my_id = '1:' . $dfrn_id; $my_id = '1:' . $dfrn_id;
break; break;
case 1: case 1:
$condition = ['dfrn-id' => $dfrn_id, 'duplex' => true]; $condition = ['dfrn-id' => $dfrn_id, 'duplex' => true, 'uid' => $user['uid']];
$my_id = '0:' . $dfrn_id; $my_id = '0:' . $dfrn_id;
break; break;
default: default:

4
src/Worker/RemoveContact.php
View File

@ -13,8 +13,8 @@ require_once 'include/dba.php';
class RemoveContact { class RemoveContact {
public static function execute($id) { public static function execute($id) {
// Only delete if the contact is archived // Only delete if the contact is to be deleted
$condition = ['archive' => true, 'network' => Protocol::PHANTOM, 'id' => $id]; $condition = ['network' => Protocol::PHANTOM, 'id' => $id];
$r = DBA::exists('contact', $condition); $r = DBA::exists('contact', $condition);
if (!DBA::isResult($r)) { if (!DBA::isResult($r)) {
return; return;