Merge pull request #5923 from MrPetovan/task/improve-contact-module

Improve Contact module
This commit is contained in:
Michael Vogel 2018-10-15 06:02:55 +02:00 committed by GitHub
commit 0080d117ee
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 193 additions and 210 deletions

View file

@ -5,7 +5,7 @@
use Friendica\App;
use Friendica\Core\L10n;
use Friendica\Core\PConfig;
use Friendica\Module\Contacts;
use Friendica\Module\Contact;
function update_contacts_content(App $a)
{

View file

@ -349,7 +349,7 @@ class Group extends BaseObject
* @param int $cid
* @return string
*/
public static function sidebarWidget($every = 'contacts', $each = 'group', $editmode = 'standard', $group_id = '', $cid = 0)
public static function sidebarWidget($every = 'contact', $each = 'group', $editmode = 'standard', $group_id = '', $cid = 0)
{
$o = '';
@ -404,7 +404,7 @@ class Group extends BaseObject
'newgroup' => $editmode == 'extended' || $editmode == 'full' ? 1 : '',
'grouppage' => 'group/',
'$edittext' => L10n::t('Edit group'),
'$ungrouped' => $every === 'contacts' ? L10n::t('Contacts not in any group') : '',
'$ungrouped' => $every === 'contact' ? L10n::t('Contacts not in any group') : '',
'$ungrouped_selected' => (($group_id === 'none') ? 'group-selected' : ''),
'$createtext' => L10n::t('Create a new group'),
'$creategroup' => L10n::t('Group Name: '),

View file

@ -37,8 +37,8 @@ class Contact extends BaseModule
}
$nets = defaults($_GET, 'nets', '');
if ($nets == "all") {
$nets = "";
if ($nets == 'all') {
$nets = '';
}
if (!x($a->page, 'aside')) {
@ -47,7 +47,9 @@ class Contact extends BaseModule
$contact_id = null;
$contact = null;
if ((($a->argc == 2) && intval($a->argv[1])) || (($a->argc == 3) && intval($a->argv[1]) && in_array($a->argv[2], ['posts', 'conversations']))) {
if ($a->argc == 2 && intval($a->argv[1])
|| $a->argc == 3 && intval($a->argv[1]) && in_array($a->argv[2], ['posts', 'conversations'])
) {
$contact_id = intval($a->argv[1]);
$contact = DBA::selectFirst('contact', [], ['id' => $contact_id, 'uid' => local_user()]);
@ -72,21 +74,21 @@ class Contact extends BaseModule
$a->data['contact'] = $contact;
if (($a->data['contact']['network'] != "") && ($a->data['contact']['network'] != Protocol::DFRN)) {
$networkname = format_network_name($a->data['contact']['network'], $a->data['contact']['url']);
if (($contact['network'] != '') && ($contact['network'] != Protocol::DFRN)) {
$networkname = format_network_name($contact['network'], $contact['url']);
} else {
$networkname = '';
}
/// @TODO Add nice spaces
$vcard_widget = replace_macros(get_markup_template("vcard-widget.tpl"), [
'$name' => htmlentities($a->data['contact']['name']),
'$photo' => $a->data['contact']['photo'],
'$url' => Model\Contact::MagicLink($a->data['contact']['url']),
'$addr' => (($a->data['contact']['addr'] != "") ? ($a->data['contact']['addr']) : ""),
$vcard_widget = replace_macros(get_markup_template('vcard-widget.tpl'), [
'$name' => htmlentities($contact['name']),
'$photo' => $contact['photo'],
'$url' => Model\Contact::MagicLink($contact['url']),
'$addr' => defaults($contact, 'addr', ''),
'$network_name' => $networkname,
'$network' => L10n::t('Network:'),
'$account_type' => Model\Contact::getAccountType($a->data['contact'])
'$account_type' => Model\Contact::getAccountType($contact)
]);
$findpeople_widget = '';
@ -94,7 +96,7 @@ class Contact extends BaseModule
$networks_widget = '';
} else {
$vcard_widget = '';
$networks_widget = Widget::networks('contacts', $nets);
$networks_widget = Widget::networks('contact', $nets);
if (isset($_GET['add'])) {
$follow_widget = Widget::follow($_GET['add']);
} else {
@ -105,12 +107,12 @@ class Contact extends BaseModule
}
if ($contact['uid'] != 0) {
$groups_widget = Model\Group::sidebarWidget('contacts', 'group', 'full', 'everyone', $contact_id);
$groups_widget = Model\Group::sidebarWidget('contact', 'group', 'full', 'everyone', $contact_id);
} else {
$groups_widget = null;
}
$a->page['aside'] .= replace_macros(get_markup_template("contacts-widget-sidebar.tpl"), [
$a->page['aside'] .= replace_macros(get_markup_template('contacts-widget-sidebar.tpl'), [
'$vcard_widget' => $vcard_widget,
'$findpeople_widget' => $findpeople_widget,
'$follow_widget' => $follow_widget,
@ -119,7 +121,7 @@ class Contact extends BaseModule
]);
$base = $a->getBaseURL();
$tpl = get_markup_template("contacts-head.tpl");
$tpl = get_markup_template('contacts-head.tpl');
$a->page['htmlhead'] .= replace_macros($tpl, [
'$baseurl' => System::baseUrl(true),
'$base' => $base
@ -140,31 +142,30 @@ class Contact extends BaseModule
$count_actions = 0;
foreach ($orig_records as $orig_record) {
$contact_id = $orig_record['id'];
if (defaults($_POST, 'contacts_batch_update', '')) {
if (!empty($_POST['contacts_batch_update'])) {
self::updateContactFromPoll($contact_id);
$count_actions++;
}
if (defaults($_POST, 'contacts_batch_block', '')) {
if (!empty($_POST['contacts_batch_block'])) {
self::blockContact($contact_id);
$count_actions++;
}
if (defaults($_POST, 'contacts_batch_ignore', '')) {
if (!empty($_POST['contacts_batch_ignore'])) {
self::ignoreContact($contact_id);
$count_actions++;
}
if (defaults($_POST, 'contacts_batch_archive', '')) {
$r = self::archiveContact($contact_id, $orig_record);
if ($r) {
if (!empty($_POST['contacts_batch_archive'])
&& self::archiveContact($contact_id, $orig_record)
) {
$count_actions++;
}
}
if (defaults($_POST, 'contacts_batch_drop', '')) {
if (!empty($_POST['contacts_batch_drop'])) {
self::dropContact($orig_record);
$count_actions++;
}
}
if ($count_actions > 0) {
info(L10n::tt("%d contact edited.", "%d contacts edited.", $count_actions));
info(L10n::tt('%d contact edited.', '%d contacts edited.', $count_actions));
}
goaway('contact');
@ -178,7 +179,7 @@ class Contact extends BaseModule
return;
}
if ($a->argv[1] === "batch") {
if ($a->argv[1] === 'batch') {
self::batchActions($a);
return;
}
@ -204,9 +205,9 @@ class Contact extends BaseModule
}
}
$hidden = defaults($_POST['hidden']);
$hidden = !empty($_POST['hidden']);
$notify = defaults($_POST['notify']);
$notify = !empty($_POST['notify']);
$fetch_further_information = intval(defaults($_POST, 'fetch_further_information', 0));
@ -219,15 +220,16 @@ class Contact extends BaseModule
$info = escape_tags(trim($_POST['info']));
$r = DBA::update('contact',
['profile-id' => $profile_id,
$r = DBA::update('contact', [
'profile-id' => $profile_id,
'priority' => $priority,
'info' => $info,
'hidden' => $hidden,
'notify_new_posts' => $notify,
'fetch_further_information' => $fetch_further_information,
'ffi_keyword_blacklist' => $ffi_keyword_blacklist],
['id' => $contact_id, 'uid' => local_user()]);
['id' => $contact_id, 'uid' => local_user()]
);
if (DBA::isResult($r)) {
info(L10n::t('Contact updated.') . EOL);
@ -252,17 +254,17 @@ class Contact extends BaseModule
return;
}
$uid = $contact["uid"];
$uid = $contact['uid'];
if ($contact["network"] == Protocol::OSTATUS) {
$result = Model\Contact::createFromProbe($uid, $contact["url"], false, $contact["network"]);
if ($contact['network'] == Protocol::OSTATUS) {
$result = Model\Contact::createFromProbe($uid, $contact['url'], false, $contact['network']);
if ($result['success']) {
DBA::update('contact', ['subhub' => 1], ['id' => $contact_id]);
}
} else {
// pull feed and consume it, which should subscribe to the hub.
Worker::add(PRIORITY_HIGH, "OnePoll", $contact_id, "force");
Worker::add(PRIORITY_HIGH, 'OnePoll', $contact_id, 'force');
}
}
@ -273,60 +275,49 @@ class Contact extends BaseModule
return;
}
$uid = $contact["uid"];
$uid = $contact['uid'];
$data = Probe::uri($contact["url"], "", 0, false);
$data = Probe::uri($contact['url'], '', 0, false);
// "Feed" or "Unknown" is mostly a sign of communication problems
if ((in_array($data["network"], [Protocol::FEED, Protocol::PHANTOM])) && ($data["network"] != $contact["network"])) {
// 'Feed' or 'Unknown' is mostly a sign of communication problems
if ((in_array($data['network'], [Protocol::FEED, Protocol::PHANTOM])) && ($data['network'] != $contact['network'])) {
return;
}
$updatefields = ["name", "nick", "url", "addr", "batch", "notify", "poll", "request", "confirm",
"poco", "network", "alias"];
$update = [];
$updatefields = ['name', 'nick', 'url', 'addr', 'batch', 'notify', 'poll', 'request', 'confirm', 'poco', 'network', 'alias'];
$fields = [];
if ($data["network"] == Protocol::OSTATUS) {
$result = Model\Contact::createFromProbe($uid, $data["url"], false);
if ($data['network'] == Protocol::OSTATUS) {
$result = Model\Contact::createFromProbe($uid, $data['url'], false);
if ($result['success']) {
$update["subhub"] = true;
$fields['subhub'] = true;
}
}
foreach ($updatefields AS $field) {
if (isset($data[$field]) && ($data[$field] != "")) {
$update[$field] = $data[$field];
if (!empty($data[$field])) {
$fields[$field] = $data[$field];
}
}
$update["nurl"] = normalise_link($data["url"]);
$fields['nurl'] = normalise_link($data['url']);
$query = "";
if (isset($data["priority"]) && ($data["priority"] != 0)) {
$query = "'priority' => '" . intval($data["priority"]) . "'";
if (!empty($data['priority'])) {
$fields['priority'] = intval($data['priority']);
}
foreach ($update AS $key => $value) {
if ($query != "") {
$query .= ", ";
}
$query .= "'" . $key . "' => '" . DBA::escape($value) . "'";
}
if ($query == "") {
if (empty($fields)) {
return;
}
$r = DBA::update('contact', $query, ['id' => $contact_id, 'uid' => local_user()]);
$r = DBA::update('contact', $fields, ['id' => $contact_id, 'uid' => local_user()]);
// Update the entry in the contact table
Model\Contact::updateAvatar($data['photo'], local_user(), $contact_id, true);
// Update the entry in the gcontact table
Model\GContact::updateFromProbe($data["url"]);
Model\GContact::updateFromProbe($data['url']);
}
private static function blockContact($contact_id)
@ -351,17 +342,12 @@ class Contact extends BaseModule
private static function dropContact($orig_record)
{
$a = get_app();
$r = q("SELECT `contact`.*, `user`.* FROM `contact` INNER JOIN `user` ON `contact`.`uid` = `user`.`uid`
WHERE `user`.`uid` = %d AND `contact`.`self` LIMIT 1",
intval($a->user['uid'])
);
if (!DBA::isResult($r)) {
$owner = Model\User::getOwnerDataById(local_user());
if (!DBA::isResult($owner)) {
return;
}
Model\Contact::terminateFriendship($r[0], $orig_record, true);
Model\Contact::terminateFriendship($owner, $orig_record, true);
Model\Contact::remove($orig_record['id']);
}
@ -437,8 +423,8 @@ class Contact extends BaseModule
if ($cmd === 'drop' && ($orig_record['uid'] != 0)) {
// Check if we should do HTML-based delete confirmation
if (defaults($_REQUEST, 'confirm')) {
// <form> can't take arguments in its "action" parameter
if (!empty($_REQUEST['confirm'])) {
// <form> can't take arguments in its 'action' parameter
// so add any arguments as hidden inputs
$query = explode_querystring($a->query_string);
$inputs = [];
@ -464,7 +450,7 @@ class Contact extends BaseModule
]);
}
// Now check how the user responded to the confirmation query
if (defaults($_REQUEST, 'canceled')) {
if (!empty($_REQUEST['canceled'])) {
goaway('contact');
}
@ -484,7 +470,7 @@ class Contact extends BaseModule
$_SESSION['return_url'] = $a->query_string;
if ((defaults($a->data, 'contact')) && (is_array($a->data['contact']))) {
if (!empty($a->data['contact']) && is_array($a->data['contact'])) {
$contact_id = $a->data['contact']['id'];
$contact = $a->data['contact'];
@ -522,7 +508,7 @@ class Contact extends BaseModule
}
if (!in_array($contact['network'], [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::OSTATUS, Protocol::DIASPORA])) {
$relation_text = "";
$relation_text = '';
}
$relation_text = sprintf($relation_text, htmlentities($contact['name']));
@ -539,13 +525,13 @@ class Contact extends BaseModule
$last_update = (($contact['last-update'] <= NULL_DATE) ? L10n::t('Never') : DateTimeFormat::local($contact['last-update'], 'D, j M Y, g:i A'));
if ($contact['last-update'] > NULL_DATE) {
$last_update .= ' ' . (($contact['last-update'] <= $contact['success_update']) ? L10n::t("\x28Update was successful\x29") : L10n::t("\x28Update was not successful\x29"));
$last_update .= ' ' . (($contact['last-update'] <= $contact['success_update']) ? L10n::t('(Update was successful)') : L10n::t('(Update was not successful)'));
}
$lblsuggest = (($contact['network'] === Protocol::DFRN) ? L10n::t('Suggest friends') : '');
$poll_enabled = in_array($contact['network'], [Protocol::DFRN, Protocol::OSTATUS, Protocol::FEED, Protocol::MAIL]);
$nettype = L10n::t('Network type: %s', ContactSelector::networkToName($contact['network'], $contact["url"]));
$nettype = L10n::t('Network type: %s', ContactSelector::networkToName($contact['network'], $contact['url']));
// tabs
$tab_str = self::getTabsHTML($a, $contact, 3);
@ -558,8 +544,9 @@ class Contact extends BaseModule
'fetch_further_information',
L10n::t('Fetch further information for feeds'),
$contact['fetch_further_information'],
L10n::t("Fetch information like preview pictures, title and teaser from the feed item. You can activate this if the feed doesn't contain much text. Keywords are taken from the meta header in the feed item and are posted as hash tags."),
['0' => L10n::t('Disabled'),
L10n::t('Fetch information like preview pictures, title and teaser from the feed item. You can activate this if the feed doesn\'t contain much text. Keywords are taken from the meta header in the feed item and are posted as hash tags.'),
[
'0' => L10n::t('Disabled'),
'1' => L10n::t('Fetch information'),
'3' => L10n::t('Fetch keywords'),
'2' => L10n::t('Fetch information and keywords')
@ -569,12 +556,12 @@ class Contact extends BaseModule
$poll_interval = null;
if (in_array($contact['network'], [Protocol::FEED, Protocol::MAIL])) {
$poll_interval = ContactSelector::pollInterval($contact['priority'], (!$poll_enabled));
$poll_interval = ContactSelector::pollInterval($contact['priority'], !$poll_enabled);
}
$profile_select = null;
if ($contact['network'] == Protocol::DFRN) {
$profile_select = ContactSelector::profileAssign($contact['profile-id'], (($contact['network'] !== Protocol::DFRN) ? true : false));
$profile_select = ContactSelector::profileAssign($contact['profile-id'], $contact['network'] !== Protocol::DFRN);
}
/// @todo Only show the following link with DFRN when the remote version supports it
@ -582,12 +569,12 @@ class Contact extends BaseModule
$follow_text = '';
if (in_array($contact['rel'], [Model\Contact::FRIEND, Model\Contact::SHARING])) {
if (in_array($contact['network'], Protocol::NATIVE_SUPPORT)) {
$follow = $a->getBaseURL(true) . "/unfollow?url=" . urlencode($contact["url"]);
$follow_text = L10n::t("Disconnect/Unfollow");
$follow = $a->getBaseURL(true) . '/unfollow?url=' . urlencode($contact['url']);
$follow_text = L10n::t('Disconnect/Unfollow');
}
} else {
$follow = $a->getBaseURL(true) . "/follow?url=" . urlencode($contact["url"]);
$follow_text = L10n::t("Connect/Follow");
$follow = $a->getBaseURL(true) . '/follow?url=' . urlencode($contact['url']);
$follow_text = L10n::t('Connect/Follow');
}
// Load contactact related actions like hide, suggest, delete and others
@ -603,9 +590,9 @@ class Contact extends BaseModule
$contact_settings_label = null;
}
$tpl = get_markup_template("contact_edit.tpl");
$tpl = get_markup_template('contact_edit.tpl');
$o .= replace_macros($tpl, [
'$header' => L10n::t("Contact"),
'$header' => L10n::t('Contact'),
'$tab_str' => $tab_str,
'$submit' => L10n::t('Submit'),
'$lbl_vis1' => $lbl_vis1,
@ -619,7 +606,7 @@ class Contact extends BaseModule
'$visit' => L10n::t('Visit %s\'s profile [%s]', $contact['name'], $contact['url']),
'$blockunblock' => L10n::t('Block/Unblock contact'),
'$ignorecont' => L10n::t('Ignore contact'),
'$lblcrepair' => L10n::t("Repair URL settings"),
'$lblcrepair' => L10n::t('Repair URL settings'),
'$lblrecent' => L10n::t('View conversations'),
'$lblsuggest' => $lblsuggest,
'$nettype' => $nettype,
@ -656,19 +643,19 @@ class Contact extends BaseModule
'$profileurllabel'=> L10n::t('Profile URL'),
'$profileurl' => $contact['url'],
'$account_type' => Model\Contact::getAccountType($contact),
'$location' => BBCode::convert($contact["location"]),
'$location_label' => L10n::t("Location:"),
'$xmpp' => BBCode::convert($contact["xmpp"]),
'$xmpp_label' => L10n::t("XMPP:"),
'$about' => BBCode::convert($contact["about"], false),
'$about_label' => L10n::t("About:"),
'$keywords' => $contact["keywords"],
'$keywords_label' => L10n::t("Tags:"),
'$contact_action_button' => L10n::t("Actions"),
'$location' => BBCode::convert($contact['location']),
'$location_label' => L10n::t('Location:'),
'$xmpp' => BBCode::convert($contact['xmpp']),
'$xmpp_label' => L10n::t('XMPP:'),
'$about' => BBCode::convert($contact['about'], false),
'$about_label' => L10n::t('About:'),
'$keywords' => $contact['keywords'],
'$keywords_label' => L10n::t('Tags:'),
'$contact_action_button' => L10n::t('Actions'),
'$contact_actions'=> $contact_actions,
'$contact_status' => L10n::t("Status"),
'$contact_status' => L10n::t('Status'),
'$contact_settings_label' => $contact_settings_label,
'$contact_profile_label' => L10n::t("Profile"),
'$contact_profile_label' => L10n::t('Profile'),
]);
$arr = ['contact' => $contact, 'output' => $o];
@ -705,8 +692,8 @@ class Contact extends BaseModule
$sql_extra .= sprintf(" AND `network` != '%s' ", Protocol::PHANTOM);
$search = defaults($_GET, 'search') ? notags(trim($_GET['search'])) : '';
$nets = defaults($_GET, 'nets' ) ? notags(trim($_GET['nets'])) : '';
$search = notags(trim(defaults($_GET, 'search', '')));
$nets = notags(trim(defaults($_GET, 'nets' , '')));
$tabs = [
[
@ -812,7 +799,7 @@ class Contact extends BaseModule
}
}
$tpl = get_markup_template("contacts-template.tpl");
$tpl = get_markup_template('contacts-template.tpl');
$o .= replace_macros($tpl, [
'$baseurl' => System::baseUrl(),
'$header' => L10n::t('Contacts') . (($nets) ? ' - ' . ContactSelector::networkToName($nets) : ''),
@ -820,7 +807,7 @@ class Contact extends BaseModule
'$total' => $total,
'$search' => $search_hdr,
'$desc' => L10n::t('Search your contacts'),
'$finding' => $searching ? L10n::t('Results for: %s', $search) : "",
'$finding' => $searching ? L10n::t('Results for: %s', $search) : '',
'$submit' => L10n::t('Find'),
'$cmd' => $a->cmd,
'$contacts' => $contacts,
@ -828,10 +815,10 @@ class Contact extends BaseModule
'multiselect' => 1,
'$batch_actions' => [
'contacts_batch_update' => L10n::t('Update'),
'contacts_batch_block' => L10n::t('Block') . "/" . L10n::t("Unblock"),
"contacts_batch_ignore" => L10n::t('Ignore') . "/" . L10n::t("Unignore"),
"contacts_batch_archive" => L10n::t('Archive') . "/" . L10n::t("Unarchive"),
"contacts_batch_drop" => L10n::t('Delete'),
'contacts_batch_block' => L10n::t('Block') . '/' . L10n::t('Unblock'),
'contacts_batch_ignore' => L10n::t('Ignore') . '/' . L10n::t('Unignore'),
'contacts_batch_archive' => L10n::t('Archive') . '/' . L10n::t('Unarchive'),
'contacts_batch_drop' => L10n::t('Delete'),
],
'$h_batch_actions' => L10n::t('Batch Actions'),
'$paginate' => paginate($a),
@ -950,18 +937,16 @@ class Contact extends BaseModule
}
if (DBA::isResult($contact)) {
$a->page['aside'] = "";
$a->page['aside'] = '';
$profiledata = Model\Contact::getDetailsByURL($contact["url"]);
$profiledata = Model\Contact::getDetailsByURL($contact['url']);
if (local_user()) {
if (in_array($profiledata["network"], [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS])) {
$profiledata["remoteconnect"] = System::baseUrl()."/follow?url=".urlencode($profiledata["url"]);
}
if (local_user() && in_array($profiledata['network'], [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS])) {
$profiledata['remoteconnect'] = System::baseUrl() . '/follow?url=' . urlencode($profiledata['url']);
}
Model\Profile::load($a, "", 0, $profiledata, true);
$o .= Model\Contact::getPostsFromUrl($contact["url"], true, $update);
Model\Profile::load($a, '', 0, $profiledata, true);
$o .= Model\Contact::getPostsFromUrl($contact['url'], true, $update);
}
return $o;
@ -974,18 +959,16 @@ class Contact extends BaseModule
$o = self::getTabsHTML($a, $contact, 2);
if (DBA::isResult($contact)) {
$a->page['aside'] = "";
$a->page['aside'] = '';
$profiledata = Model\Contact::getDetailsByURL($contact["url"]);
$profiledata = Model\Contact::getDetailsByURL($contact['url']);
if (local_user()) {
if (in_array($profiledata["network"], [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS])) {
$profiledata["remoteconnect"] = System::baseUrl()."/follow?url=".urlencode($profiledata["url"]);
}
if (local_user() && in_array($profiledata['network'], [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS])) {
$profiledata['remoteconnect'] = System::baseUrl() . '/follow?url=' . urlencode($profiledata['url']);
}
Model\Profile::load($a, "", 0, $profiledata, true);
$o .= Model\Contact::getPostsFromUrl($contact["url"]);
Model\Profile::load($a, '', 0, $profiledata, true);
$o .= Model\Contact::getPostsFromUrl($contact['url']);
}
return $o;
@ -1043,7 +1026,7 @@ class Contact extends BaseModule
'username' => htmlentities($rr['name']),
'account_type' => Model\Contact::getAccountType($rr),
'sparkle' => $sparkle,
'itemurl' => (($rr['addr'] != "") ? $rr['addr'] : $rr['url']),
'itemurl' => defaults($rr, 'addr', $rr['url']),
'url' => $url,
'network' => ContactSelector::networkToName($rr['network'], $rr['url']),
'nick' => htmlentities($rr['nick']),