Don't display ignored pending contacts, link to contact request page

This commit is contained in:
Michael 2019-09-09 05:29:33 +00:00
parent 591ba9c68e
commit 2dbd843ed2
4 changed files with 29 additions and 17 deletions

View file

@ -98,9 +98,14 @@ function notifications_content(App $a)
if ((($a->argc > 1) && ($a->argv[1] == 'intros')) || (($a->argc == 1))) { if ((($a->argc > 1) && ($a->argv[1] == 'intros')) || (($a->argc == 1))) {
Nav::setSelected('introductions'); Nav::setSelected('introductions');
$id = 0;
if (!empty($a->argv[2]) && intval($a->argv[2]) != 0) {
$id = (int)$a->argv[2];
}
$all = (($a->argc > 2) && ($a->argv[2] == 'all')); $all = (($a->argc > 2) && ($a->argv[2] == 'all'));
$notifs = $nm->introNotifs($all, $startrec, $perpage); $notifs = $nm->introNotifs($all, $startrec, $perpage, $id);
// Get the network notifications // Get the network notifications
} elseif (($a->argc > 1) && ($a->argv[1] == 'network')) { } elseif (($a->argc > 1) && ($a->argv[1] == 'network')) {

View file

@ -549,6 +549,7 @@ class NotificationsManager extends BaseObject
* which aren't marked as ignored * which aren't marked as ignored
* @param int $start Start the query at this point * @param int $start Start the query at this point
* @param int $limit Maximum number of query results * @param int $limit Maximum number of query results
* @param int $id When set, only the introduction with this id is displayed
* *
* @return array with * @return array with
* string 'ident' => Notification identifier * string 'ident' => Notification identifier
@ -556,14 +557,20 @@ class NotificationsManager extends BaseObject
* @throws \Friendica\Network\HTTPException\InternalServerErrorException * @throws \Friendica\Network\HTTPException\InternalServerErrorException
* @throws \ImagickException * @throws \ImagickException
*/ */
public function introNotifs($all = false, $start = 0, $limit = 80) public function introNotifs($all = false, $start = 0, $limit = 80, $id = 0)
{ {
$ident = 'introductions'; $ident = 'introductions';
$notifs = []; $notifs = [];
$sql_extra = ""; $sql_extra = "";
if (!$all) { if (empty($id)) {
$sql_extra = " AND NOT `ignore` "; if (!$all) {
$sql_extra = " AND NOT `ignore` ";
}
$sql_extra .= " AND NOT `intro`.`blocked` ";
} else {
$sql_extra = sprintf(" AND `intro`.`id` = %d ", intval($id));
} }
/// @todo Fetch contact details by "Contact::getDetailsByUrl" instead of queries to contact, fcontact and gcontact /// @todo Fetch contact details by "Contact::getDetailsByUrl" instead of queries to contact, fcontact and gcontact
@ -578,7 +585,7 @@ class NotificationsManager extends BaseObject
LEFT JOIN `contact` ON `contact`.`id` = `intro`.`contact-id` LEFT JOIN `contact` ON `contact`.`id` = `intro`.`contact-id`
LEFT JOIN `gcontact` ON `gcontact`.`nurl` = `contact`.`nurl` LEFT JOIN `gcontact` ON `gcontact`.`nurl` = `contact`.`nurl`
LEFT JOIN `fcontact` ON `intro`.`fid` = `fcontact`.`id` LEFT JOIN `fcontact` ON `intro`.`fid` = `fcontact`.`id`
WHERE `intro`.`uid` = ? $sql_extra AND `intro`.`blocked` = 0 WHERE `intro`.`uid` = ? $sql_extra
LIMIT ?, ?", LIMIT ?, ?",
$_SESSION['uid'], $_SESSION['uid'],
$start, $start,

View file

@ -1175,7 +1175,7 @@ class Contact extends BaseObject
} }
$sparkle = false; $sparkle = false;
if (($contact['network'] === Protocol::DFRN) && !$contact['self']) { if (($contact['network'] === Protocol::DFRN) && !$contact['self'] && !$contact['pending']) {
$sparkle = true; $sparkle = true;
$profile_link = System::baseUrl() . '/redir/' . $contact['id'] . '?url=' . $contact['url']; $profile_link = System::baseUrl() . '/redir/' . $contact['id'] . '?url=' . $contact['url'];
} else { } else {
@ -1192,11 +1192,11 @@ class Contact extends BaseObject
$profile_link = $profile_link . '?tab=profile'; $profile_link = $profile_link . '?tab=profile';
} }
if (self::canReceivePrivateMessages($contact)) { if (self::canReceivePrivateMessages($contact) && !$contact['pending']) {
$pm_url = System::baseUrl() . '/message/new/' . $contact['id']; $pm_url = System::baseUrl() . '/message/new/' . $contact['id'];
} }
if (($contact['network'] == Protocol::DFRN) && !$contact['self']) { if (($contact['network'] == Protocol::DFRN) && !$contact['self'] && !$contact['pending']) {
$poke_link = System::baseUrl() . '/poke/?f=&c=' . $contact['id']; $poke_link = System::baseUrl() . '/poke/?f=&c=' . $contact['id'];
} }
@ -1231,6 +1231,13 @@ class Contact extends BaseObject
'pm' => [L10n::t('Send PM'), $pm_url, false], 'pm' => [L10n::t('Send PM'), $pm_url, false],
'poke' => [L10n::t('Poke'), $poke_link, false], 'poke' => [L10n::t('Poke'), $poke_link, false],
]; ];
if ($contact['pending']) {
$intro = DBA::selectFirst('intro', ['id'], ['contact-id' => $contact['id']]);
if (DBA::isResult($intro)) {
$menu['follow'] = [L10n::t('Approve'), 'notifications/intros/' . $intro['id'], true];
}
}
} }
$args = ['contact' => $contact, 'menu' => &$menu]; $args = ['contact' => $contact, 'menu' => &$menu];

View file

@ -642,7 +642,8 @@ class Contact extends BaseModule
$sql_extra = " AND `archive` AND NOT `blocked`"; $sql_extra = " AND `archive` AND NOT `blocked`";
break; break;
case 'pending': case 'pending':
$sql_extra = " AND `pending` AND NOT `archive`"; $sql_extra = " AND `pending` AND NOT `archive`
AND NOT EXISTS (SELECT `id` FROM `intro` WHERE `contact-id` = `contact`.`id` AND `ignore`)";
break; break;
default: default:
$sql_extra = " AND NOT `archive` AND NOT `blocked` AND NOT `pending`"; $sql_extra = " AND NOT `archive` AND NOT `blocked` AND NOT `pending`";
@ -993,14 +994,6 @@ class Contact extends BaseModule
$alt_text = L10n::t('Pending outgoing contact request'); $alt_text = L10n::t('Pending outgoing contact request');
} else { } else {
$alt_text = L10n::t('Pending incoming contact request'); $alt_text = L10n::t('Pending incoming contact request');
$intro = DBA::selectFirst('intro', ['blocked', 'ignore'], ['contact-id' => $rr['id']]);
if (DBA::isResult($intro)) {
if ($intro['blocked']) {
$alt_text = L10n::t('Blocked incoming contact request');
} elseif ($intro['ignore']) {
$alt_text = L10n::t('Ignored incoming contact request');
}
}
} }
} }