aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorfriendica <info@friendica.com>2013-01-27 17:05:32 -0800
committerfriendica <info@friendica.com>2013-01-27 17:05:32 -0800
commit357e71cb44651b364ef5a94e66bd8f7c3125b21a (patch)
tree192aaa0f5725a10ddf2a3d262f659f35562ae9c2
parentbec484237a626c8c0ae930277b17cc1341380a47 (diff)
downloadvolse-hubzilla-357e71cb44651b364ef5a94e66bd8f7c3125b21a.tar.gz
volse-hubzilla-357e71cb44651b364ef5a94e66bd8f7c3125b21a.tar.bz2
volse-hubzilla-357e71cb44651b364ef5a94e66bd8f7c3125b21a.zip
simplify friendship approval - merge with connection editing
-rw-r--r--include/nav.php2
-rw-r--r--mod/connections.php60
-rw-r--r--view/theme/redbasic/css/style.css3
-rw-r--r--view/tpl/abook_edit.tpl12
-rw-r--r--view/tpl/smarty3/abook_edit.tpl12
5 files changed, 80 insertions, 9 deletions
diff --git a/include/nav.php b/include/nav.php
index 73421beab..bbf803390 100644
--- a/include/nav.php
+++ b/include/nav.php
@@ -148,7 +148,7 @@ EOT;
$nav['home']['mark'] = array('', t('Mark all channel notifications seen'), '','');
- $nav['intros'] = array('intro', t('Introductions'), "", t('New Connections'));
+ $nav['intros'] = array('connections/pending', t('Introductions'), "", t('New Connections'));
$nav['intros']['all']=array('intro', t('See all channel introductions'), "", "");
diff --git a/mod/connections.php b/mod/connections.php
index 90e74999e..1e23ace9b 100644
--- a/mod/connections.php
+++ b/mod/connections.php
@@ -93,11 +93,18 @@ function connections_post(&$a) {
}
}
- $r = q("UPDATE abook SET abook_profile = %d, abook_my_perms = %d , abook_closeness = %d
+ $abook_flags = $orig_record[0]['abook_flags'];
+
+ if(($_REQUEST['pending']) && ($abook_flags & ABOOK_FLAG_PENDING)) {
+ $abook_flags = ( $abook_flags ^ ABOOK_FLAG_PENDING );
+ }
+
+ $r = q("UPDATE abook SET abook_profile = %d, abook_my_perms = %d , abook_closeness = %d, abook_flags = %d
where abook_id = %d AND abook_channel = %d LIMIT 1",
intval($profile_id),
intval($abook_my_perms),
intval($closeness),
+ intval($abook_flags),
intval($contact_id),
intval(local_user())
);
@@ -216,6 +223,20 @@ function connections_content(&$a) {
goaway($a->get_baseurl(true) . '/connections/' . $contact_id);
}
+ // We'll prevent somebody from unapproving a contact.
+
+ if($cmd === 'approve') {
+ if($orig_record[0]['abook_flags'] & ABOOK_FLAG_PENDING) {
+ if(abook_toggle_flag($orig_record[0],ABOOK_FLAG_PENDING))
+ info((($orig_record[0]['abook_flags'] & ABOOK_FLAG_PENDING)
+ ? t('Channel has been approved')
+ : t('Channel has been unapproved')) . EOL );
+ else
+ notice(t('Unable to set address book parameters.') . EOL);
+ }
+ goaway($a->get_baseurl(true) . '/connections/' . $contact_id);
+ }
+
if($cmd === 'drop') {
@@ -327,6 +348,7 @@ function connections_content(&$a) {
$global_perms = get_perms();
$existing = get_all_perms(local_user(),$contact);
+ $unapproved = array('pending', t('Approve this connection'), '', t('Accept connection to allow communication'));
foreach($global_perms as $k => $v) {
$perms[] = array('perms_' . $k, $v[3], (($contact['abook_their_perms'] & $v[1]) ? "1" : ""),((($contact['abook_my_perms'] & $v[1]) || $existing[$k]) ? "1" : ""), $v[1], (($channel[$v[0]] == PERMS_SPECIFIC) ? '' : '1'), $v[4]);
@@ -344,6 +366,10 @@ function connections_content(&$a) {
'$slide' => $slide,
'$tabs' => $t,
'$tab_str' => $tab_str,
+ '$is_pending' => (($contact['abook_flags'] & ABOOK_FLAG_PENDING) ? 1 : ''),
+ '$unapproved' => $unapproved,
+ '$approve' => t('Approve this connection'),
+ '$noperms' => (((! $self) && (! $contact['abook_myperms'])) ? t('Connection has no permissions!') : ''),
'$submit' => t('Submit'),
'$lbl_vis1' => t('Profile Visibility'),
'$lbl_vis2' => sprintf( t('Please choose the profile you would like to display to %s when viewing your profile securely.'), $contact['name']),
@@ -384,6 +410,7 @@ function connections_content(&$a) {
'$blocked' => (($contact['blocked']) ? t('Currently blocked') : ''),
'$ignored' => (($contact['readonly']) ? t('Currently ignored') : ''),
'$archived' => (($contact['archive']) ? t('Currently archived') : ''),
+ '$pending' => (($contact['archive']) ? t('Currently pending') : ''),
'$hidden' => array('hidden', t('Hide this contact from others'), ($contact['hidden'] == 1), t('Replies/likes to your public posts <strong>may</strong> still be visible')),
'$photo' => $contact['photo'],
'$name' => $contact['name'],
@@ -407,31 +434,45 @@ function connections_content(&$a) {
$ignored = false;
$archived = false;
$unblocked = false;
+ $pending = false;
+
$all = false;
$_SESSION['return_url'] = $a->query_string;
$search_flags = 0;
+ $head = '';
if(argc() == 2) {
switch(argv(1)) {
case 'blocked':
$search_flags = ABOOK_FLAG_BLOCKED;
+ $head = t('Blocked');
$blocked = true;
break;
case 'ignored':
$search_flags = ABOOK_FLAG_IGNORED;
+ $head = t('Ignored');
$ignored = true;
break;
case 'hidden':
$search_flags = ABOOK_FLAG_HIDDEN;
+ $head = t('Hidden');
$hidden = true;
break;
case 'archived':
$search_flags = ABOOK_FLAG_ARCHIVED;
+ $head = t('Archived');
$archived = true;
break;
+ case 'pending':
+ $search_flags = ABOOK_FLAG_PENDING;
+ $head = t('New');
+ $pending = true;
+ break;
+
case 'all':
+ $head = t('All');
default:
$search_flags = 0;
$all = true;
@@ -458,6 +499,12 @@ function connections_content(&$a) {
'title' => t('Suggest new connections'),
),
array(
+ 'label' => t('New Connections'),
+ 'url' => $a->get_baseurl(true) . '/connections/pending',
+ 'sel' => ($pending) ? 'active' : '',
+ 'title' => t('Show pending (new) connections'),
+ ),
+ array(
'label' => t('All Connections'),
'url' => $a->get_baseurl(true) . '/connections/all',
'sel' => ($all) ? 'active' : '',
@@ -513,10 +560,9 @@ function connections_content(&$a) {
$r = q("SELECT COUNT(abook.abook_id) AS total FROM abook left join xchan on abook.abook_xchan = xchan.xchan_hash
- where abook_channel = %d and not (abook_flags & %d) and not (abook_flags & %d) $sql_extra $sql_extra2 ",
+ where abook_channel = %d and not (abook_flags & %d) $sql_extra $sql_extra2 ",
intval(local_user()),
- intval(ABOOK_FLAG_SELF),
- intval(ABOOK_FLAG_PENDING)
+ intval(ABOOK_FLAG_SELF)
);
if(count($r)) {
$a->set_pager_total($r[0]['total']);
@@ -524,10 +570,9 @@ function connections_content(&$a) {
}
$r = q("SELECT abook.*, xchan.* FROM abook left join xchan on abook.abook_xchan = xchan.xchan_hash
- WHERE abook_channel = %d and not (abook_flags & %d) and not (abook_flags & %d) $sql_extra $sql_extra2 ORDER BY xchan_name LIMIT %d , %d ",
+ WHERE abook_channel = %d and not (abook_flags & %d) $sql_extra $sql_extra2 ORDER BY xchan_name LIMIT %d , %d ",
intval(local_user()),
intval(ABOOK_FLAG_SELF),
- intval(ABOOK_FLAG_PENDING),
intval($a->pager['start']),
intval($a->pager['itemspage'])
);
@@ -559,9 +604,10 @@ function connections_content(&$a) {
}
+
$tpl = get_markup_template("contacts-template.tpl");
$o .= replace_macros($tpl,array(
- '$header' => t('Connnections') . (($nets) ? ' - ' . network_to_name($nets) : ''),
+ '$header' => t('Connnections') . (($head) ? ' - ' . $head : ''),
'$tabs' => $t,
'$total' => $total,
'$search' => $search_hdr,
diff --git a/view/theme/redbasic/css/style.css b/view/theme/redbasic/css/style.css
index 91aa38618..5cf3dc06f 100644
--- a/view/theme/redbasic/css/style.css
+++ b/view/theme/redbasic/css/style.css
@@ -315,7 +315,8 @@ nav #nav-link-wrapper .nav-link {
}
.tab.active {
- font-weight: bold;
+/* font-weight: bold; */
+ color: #444444;
}
ul.tabs {
diff --git a/view/tpl/abook_edit.tpl b/view/tpl/abook_edit.tpl
index 4deb84823..e77458478 100644
--- a/view/tpl/abook_edit.tpl
+++ b/view/tpl/abook_edit.tpl
@@ -13,6 +13,7 @@ $tabs
<div id="autoperm-desc" class="descriptive-paragraph">$autolbl</div>
{{ endif }}
+
<div id="contact-edit-wrapper">
{{ if $notself }}
@@ -24,12 +25,23 @@ $slide
{{ endif }}
{{ endif }}
+
+
<h3>$permlbl</h3>
<form action="connections/$contact_id" method="post" >
<input type="hidden" name="contact_id" value="$contact_id">
<input id="contact-closeness-mirror" type="hidden" name="closeness" value="$close" />
+{{ if $noperms }}
+<div id="noperm-desc" class="descriptive-paragraph">$noperms</div>
+{{ endif }}
+
+
+{{ if $is_pending }}
+{{inc field_checkbox.tpl with $field=$unapproved }}{{endinc}}
+{{ endif }}
+
<br />
<b>$quick</b>
<ul>
diff --git a/view/tpl/smarty3/abook_edit.tpl b/view/tpl/smarty3/abook_edit.tpl
index b540ba531..6331aa46c 100644
--- a/view/tpl/smarty3/abook_edit.tpl
+++ b/view/tpl/smarty3/abook_edit.tpl
@@ -18,6 +18,7 @@
<div id="autoperm-desc" class="descriptive-paragraph">{{$autolbl}}</div>
{{/if}}
+
<div id="contact-edit-wrapper">
{{if $notself}}
@@ -29,12 +30,23 @@
{{/if}}
{{/if}}
+
+
<h3>{{$permlbl}}</h3>
<form action="connections/{{$contact_id}}" method="post" >
<input type="hidden" name="contact_id" value="{{$contact_id}}">
<input id="contact-closeness-mirror" type="hidden" name="closeness" value="{{$close}}" />
+{{if $noperms}}
+<div id="noperm-desc" class="descriptive-paragraph">{{$noperms}}</div>
+{{/if}}
+
+
+{{if $is_pending}}
+{{include file="field_checkbox.tpl" field=$unapproved}}
+{{/if}}
+
<br />
<b>{{$quick}}</b>
<ul>