Redesign of the contact pages, new tab for contact statuses

2015-11-29 23:22:05 +01:00 · 2015-11-29 23:22:05 +01:00 · 42d903dd68
parent ca47c1c4df
commit 42d903dd68
7 changed files with 150 additions and 53 deletions
--- a/include/Contact.php
+++ b/include/Contact.php
@ -322,7 +322,7 @@ function contact_photo_menu($contact) {
 		$poke_link = $a->get_baseurl() . '/poke/?f=&c=' . $contact['id'];

 	$contact_url = $a->get_baseurl() . '/contacts/' . $contact['id'];
-	$posts_link = $a->get_baseurl() . '/network/0?nets=all&cid=' . $contact['id'];
+	$posts_link = $a->get_baseurl() . "/contacts/" . $contact['id'] . '/posts';
 	$contact_drop_link = $a->get_baseurl() . "/contacts/" . $contact['id'] . '/drop?confirm=1';


--- a/include/conversation.php
+++ b/include/conversation.php
@ -891,7 +891,7 @@ function item_photo_menu($item){
 	if(($cid) && (! $item['self'])) {
 		$poke_link = $a->get_baseurl($ssl_state) . '/poke/?f=&c=' . $cid;
 		$contact_url = $a->get_baseurl($ssl_state) . '/contacts/' . $cid;
-		$posts_link = $a->get_baseurl($ssl_state) . '/network/0?nets=all&cid=' . $cid;
+		$posts_link = $a->get_baseurl($ssl_state) . '/contacts/' . $cid . '/posts';

 		$clean_url = normalise_link($item['author-link']);

--- a/mod/contacts.php
+++ b/mod/contacts.php
@ -13,7 +13,7 @@ function contacts_init(&$a) {

 	$contact_id = 0;

-	if(($a->argc == 2) && intval($a->argv[1])) {
+	if((($a->argc == 2) && intval($a->argv[1])) OR (($a->argc == 3) && intval($a->argv[1]) && ($a->argv[2] == "posts"))) {
 		$contact_id = intval($a->argv[1]);
 		$r = q("SELECT * FROM `contact` WHERE `uid` = %d and `id` = %d LIMIT 1",
 			intval(local_user()),
@ -55,6 +55,7 @@ function contacts_init(&$a) {
 		$findpeople_widget .= findpeople_widget();
 	}

+	if ($a->argv[2] != "posts")
 		$groups_widget .= group_side('contacts','group','full',0,$contact_id);

 	$a->page['aside'] .= replace_macros(get_markup_template("contacts-widget-sidebar.tpl"),array(
@ -464,6 +465,9 @@ function contacts_content(&$a) {
 				goaway($a->get_baseurl(true) . '/contacts');
 			return; // NOTREACHED
 		}
+		if($cmd === 'posts') {
+			return contact_posts($a, $contact_id);
+		}
 	}


@ -548,51 +552,7 @@ function contacts_content(&$a) {
 		$all_friends = (($x) ? t('View all contacts') : '');

 		// tabs
-		$tabs = array(
-                        array(
-                                'label'=>t('Network Posts'),
-                                'url' => "network/0?nets=all&cid=".$contact["id"],
-                                'sel' => ((!isset($tab)&&$a->argv[0]=='profile')?'active':''),
-                                'title' => t('Status Messages and Posts'),
-                                'id' => 'status-tab',
-                                'accesskey' => 'm',
-                        ),
-			array(
-				'label' => (($contact['blocked']) ? t('Unblock') : t('Block') ),
-				'url'   => $a->get_baseurl(true) . '/contacts/' . $contact_id . '/block',
-				'sel'   => '',
-				'title' => t('Toggle Blocked status'),
-				'id'	=> 'toggle-block-tab',
-				'accesskey' => 'b',
-			),
-			array(
-				'label' => (($contact['readonly']) ? t('Unignore') : t('Ignore') ),
-				'url'   => $a->get_baseurl(true) . '/contacts/' . $contact_id . '/ignore',
-				'sel'   => '',
-				'title' => t('Toggle Ignored status'),
-				'id'	=> 'toggle-ignore-tab',
-				'accesskey' => 'i',
-			),
-
-			array(
-				'label' => (($contact['archive']) ? t('Unarchive') : t('Archive') ),
-				'url'   => $a->get_baseurl(true) . '/contacts/' . $contact_id . '/archive',
-				'sel'   => '',
-				'title' => t('Toggle Archive status'),
-				'id'	=> 'toggle-archive-tab',
-				'accesskey' => 'v',
-			),
-			array(
-				'label' => t('Repair'),
-				'url'   => $a->get_baseurl(true) . '/crepair/' . $contact_id,
-				'sel'   => '',
-				'title' => t('Advanced Contact Settings'),
-				'id'	=> 'repair-tab',
-				'accesskey' => 'r',
-			)
-		);
-		$tab_tpl = get_markup_template('common_tabs.tpl');
-		$tab_str = replace_macros($tab_tpl, array('$tabs' => $tabs));
+		$tab_str = contact_tabs($a, $contact_id, 2);

 		$lost_contact = (($contact['archive'] && $contact['term-date'] != '0000-00-00 00:00:00' && $contact['term-date'] < datetime_convert('','','now')) ? t('Communications lost with this contact!') : '');

@ -853,6 +813,116 @@ function contacts_content(&$a) {
 	return $o;
 }

+function contact_tabs($a, $contact_id, $active_tab) {
+	// tabs
+	$tabs = array(
+		array(
+			'label'=>t('Status'),
+			'url' => "contacts/".$contact_id."/posts",
+			'sel' => (($active_tab == 1)?'active':''),
+			'title' => t('Status Messages and Posts'),
+			'id' => 'status-tab',
+			'accesskey' => 'm',
+		),
+		array(
+			'label'=>t('Profile'),
+			'url' => "contacts/".$contact_id,
+			'sel' => (($active_tab == 2)?'active':''),
+			'title' => t('Profile Details'),
+			'id' => 'status-tab',
+			'accesskey' => 'r',
+		),
+		array(
+			'label' => t('Repair'),
+			'url'   => $a->get_baseurl(true) . '/crepair/' . $contact_id,
+			'sel' => (($active_tab == 3)?'active':''),
+			'title' => t('Advanced Contact Settings'),
+			'id'	=> 'repair-tab',
+			'accesskey' => 'r',
+		),
+		array(
+			'label' => (($contact['blocked']) ? t('Unblock') : t('Block') ),
+			'url'   => $a->get_baseurl(true) . '/contacts/' . $contact_id . '/block',
+			'sel'   => '',
+			'title' => t('Toggle Blocked status'),
+			'id'	=> 'toggle-block-tab',
+			'accesskey' => 'b',
+		),
+		array(
+			'label' => (($contact['readonly']) ? t('Unignore') : t('Ignore') ),
+			'url'   => $a->get_baseurl(true) . '/contacts/' . $contact_id . '/ignore',
+			'sel'   => '',
+			'title' => t('Toggle Ignored status'),
+			'id'	=> 'toggle-ignore-tab',
+			'accesskey' => 'i',
+		),
+		array(
+			'label' => (($contact['archive']) ? t('Unarchive') : t('Archive') ),
+			'url'   => $a->get_baseurl(true) . '/contacts/' . $contact_id . '/archive',
+			'sel'   => '',
+			'title' => t('Toggle Archive status'),
+			'id'	=> 'toggle-archive-tab',
+			'accesskey' => 'v',
+		)
+	);
+	$tab_tpl = get_markup_template('common_tabs.tpl');
+	$tab_str = replace_macros($tab_tpl, array('$tabs' => $tabs));
+
+	return $tab_str;
+}
+
+function contact_posts($a, $contact_id) {
+
+	require_once('include/conversation.php');
+
+	$r = q("SELECT * FROM `contact` WHERE `id` = %d", intval($contact_id));
+	if ($r) {
+		$contact = $r[0];
+		$a->page['aside'] = "";
+		profile_load($a, "", 0, get_contact_details_by_url($contact["url"]));
+	}
+
+	$r = q("SELECT COUNT(*) AS `total` FROM `item`
+		WHERE `item`.`uid` = %d AND `contact-id` = %d AND `item`.`id` = `item`.`parent`",
+		intval(local_user()), intval($contact_id));
+
+	$a->set_pager_total($r[0]['total']);
+
+	$r = q("SELECT `item`.`uri`, `item`.*, `item`.`id` AS `item_id`,
+			`author-name` AS `name`, `owner-avatar` AS `photo`,
+			`owner-link` AS `url`, `owner-avatar` AS `thumb`
+		FROM `item` WHERE `item`.`uid` = %d AND `contact-id` = %d AND `item`.`id` = `item`.`parent`
+		ORDER BY `item`.`created` DESC LIMIT %d, %d",
+		intval(local_user()),
+		intval($contact_id),
+		intval($a->pager['start']),
+		intval($a->pager['itemspage'])
+	);
+
+	$tab_str = contact_tabs($a, $contact_id, 1);
+
+	$header = $contact["name"];
+
+	if ($contact["addr"] != "")
+		$header .= " <".$contact["addr"].">";
+
+	$header .= " (".network_to_name($contact['network'], $contact['url']).")";
+
+//{{include file="section_title.tpl"}}
+
+	$o = "<h2>".htmlentities($header)."</h2>".$tab_str;
+
+	$o .= conversation($a,$r,'community',false);
+
+	if(!get_config('system', 'old_pager')) {
+		$o .= alt_pager($a,count($r));
+	} else {
+		$o .= paginate($a);
+	}
+
+	return $o;
+}
+
 function _contact_detail_for_template($rr){
 	switch($rr['rel']) {
 		case CONTACT_IS_FRIEND:
--- a/mod/crepair.php
+++ b/mod/crepair.php
@ -1,4 +1,6 @@
 <?php
+require_once("include/contact_selectors.php");
+require_once("mod/contacts.php");

 function crepair_init(&$a) {
 	if(! local_user())
@ -157,9 +159,20 @@ function crepair_content(&$a) {

 	$update_profile = in_array($contact['network'], array(NETWORK_DFRN, NETWORK_DSPR, NETWORK_OSTATUS));

+	$tab_str = contact_tabs($a, $contact['id'], 3);
+
+	$header = $contact["name"];
+
+       if ($contact["addr"] != "")
+                $header .= " <".$contact["addr"].">";
+
+        $header .= " (".network_to_name($contact['network'], $contact['url']).")";
+
 	$tpl = get_markup_template('crepair.tpl');
 	$o .= replace_macros($tpl, array(
-		'$title'	=> t('Repair Contact Settings'),
+		//'$title'	=> t('Repair Contact Settings'),
+		'$title'	=> htmlentities($header),
+		'$tab_str'	=> $tab_str,
 		'$warning'	=> $warning,
 		'$info'		=> $info,
 		'$returnaddr'	=> $returnaddr,
--- a/view/global.css
+++ b/view/global.css
@ -301,3 +301,15 @@ ul.credits li {
 #forum-widget-collapse:hover {
  opacity: 1.0;
 }
+
+.crepair-label {
+        margin-top: 10px;
+        float: left;
+        width: 250px;
+}
+
+.crepair-input {
+        margin-top: 10px;
+        float: left;
+        width: 200px;
+}
--- a/view/templates/contact_edit.tpl
+++ b/view/templates/contact_edit.tpl
@ -62,6 +62,7 @@
 	</div>
 	<div id="contact-edit-nav-end"></div>

+<hr />

 <form action="contacts/{{$contact_id}}" method="post" >
 <input type="hidden" name="contact_id" value="{{$contact_id}}">
--- a/view/templates/crepair.tpl
+++ b/view/templates/crepair.tpl
@ -1,16 +1,17 @@
-
 {{include file="section_title.tpl"}}

+{{$tab_str}}
+
 <div class="crepair-error-message">{{$warning}}</div><br>
 <div class="crepair-return">
 	{{$info}}<br>
-	<a href="{{$returnaddr}}">{{$return}}</a>
+	<!-- <a href="{{$returnaddr}}">{{$return}}</a> -->
 </div>
 <br />

 <form id="crepair-form" action="crepair/{{$contact_id}}" method="post" >

-<h4>{{$contact_name}}</h4>
+<!-- <h4>{{$contact_name}}</h4> -->

 <div id="contact-update-profile-wrapper">
 {{if $update_profile}}