From f6ea87298643a1fb1b948ed78b059c39d1c070f2 Mon Sep 17 00:00:00 2001 From: Michael Vogel Date: Thu, 9 Apr 2015 00:10:21 +0200 Subject: [PATCH 1/4] The behaviour when following a new contact is now unified. --- include/follow.php | 4 +- mod/dfrn_request.php | 15 ++++++ mod/follow.php | 84 +++++++++++++++++++++++++++++++-- view/templates/auto_request.tpl | 13 ++++- view/templates/dfrn_request.tpl | 13 ++++- view/templates/follow.tpl | 2 +- 6 files changed, 121 insertions(+), 10 deletions(-) mode change 100644 => 100755 mod/follow.php diff --git a/include/follow.php b/include/follow.php index ba036cd48..287a38b6b 100644 --- a/include/follow.php +++ b/include/follow.php @@ -15,7 +15,7 @@ function new_contact($uid,$url,$interactive = false) { - $result = array('success' => false,'message' => ''); + $result = array('cid' => -1, 'success' => false,'message' => ''); $a = get_app(); @@ -208,7 +208,7 @@ function new_contact($uid,$url,$interactive = false) { $contact = $r[0]; $contact_id = $r[0]['id']; - + $result['cid'] = $contact_id; $g = q("select def_gid from user where uid = %d limit 1", intval($uid) diff --git a/mod/dfrn_request.php b/mod/dfrn_request.php index aee8c0306..62b6f4bd7 100644 --- a/mod/dfrn_request.php +++ b/mod/dfrn_request.php @@ -664,6 +664,21 @@ function dfrn_request_content(&$a) { $dfrn_url = notags(trim(hex2bin($_GET['dfrn_url']))); $aes_allow = (((x($_GET,'aes_allow')) && ($_GET['aes_allow'] == 1)) ? 1 : 0); $confirm_key = (x($_GET,'confirm_key') ? $_GET['confirm_key'] : ""); + + // Checking fastlane for validity + if (x($_SESSION, "fastlane") AND (normalise_link($_SESSION["fastlane"]) == normalise_link($dfrn_url))) { + $_POST["dfrn_url"] = $dfrn_url; + $_POST["confirm_key"] = $confirm_key; + $_POST["localconfirm"] = 1; + $_POST["hidden-contact"] = 0; + $_POST["submit"] = t('Confirm'); + + dfrn_request_post($a); + + killme(); + return; // NOTREACHED + } + $tpl = get_markup_template("dfrn_req_confirm.tpl"); $o = replace_macros($tpl,array( '$dfrn_url' => $dfrn_url, diff --git a/mod/follow.php b/mod/follow.php old mode 100644 new mode 100755 index 5c2c764c7..b9401ee82 --- a/mod/follow.php +++ b/mod/follow.php @@ -3,7 +3,7 @@ require_once('include/Scrape.php'); require_once('include/follow.php'); -function follow_init(&$a) { +function follow_content(&$a) { if(! local_user()) { notice( t('Permission denied.') . EOL); @@ -11,10 +11,85 @@ function follow_init(&$a) { // NOTREACHED } + $uid = local_user(); + $url = notags(trim($_REQUEST['url'])); + + $r = q("SELECT `url` FROM `contact` WHERE `uid` = %d AND (`nurl` = '%s' OR `alias` = '%s' OR `alias` = '%s') LIMIT 1", + intval(local_user()), dbesc(normalise_link($url)), dbesc(normalise_link($url)), dbesc($url)); + + if ($r) { + notice(t('You already added this contact.').EOL); + goaway($_SESSION['return_url']); + // NOTREACHED + } + + $ret = probe_url($url); + + if($ret['network'] === NETWORK_DFRN) { + $request = $ret["request"]; + $tpl = get_markup_template('dfrn_request.tpl'); + } else { + $request = $a->get_baseurl()."/follow"; + $tpl = get_markup_template('auto_request.tpl'); + } + + $r = q("SELECT `url` FROM `contact` WHERE `uid` = %d AND `self` LIMIT 1", intval($uid)); + + if (!$r) { + notice( t('Permission denied.') . EOL); + goaway($_SESSION['return_url']); + // NOTREACHED + } + + $myaddr = $r[0]["url"]; + + // Makes the connection request for friendica contacts easier + $_SESSION["fastlane"] = $ret["url"]; + + $o = replace_macros($tpl,array( + '$header' => $ret["name"]." (".$ret["addr"].")", + '$photo' => $ret["photo"], + '$desc' => "", + '$pls_answer' => t('Please answer the following:'), + '$does_know_you' => array('knowyou', sprintf(t('Does %s know you?'),$ret["name"]), false, '', array(t('No'),t('Yes'))), + '$add_note' => t('Add a personal note:'), + '$page_desc' => "", + '$friendica' => "", + '$statusnet' => "", + '$diaspora' => "", + '$diasnote' => "", + '$your_address' => t('Your Identity Address:'), + '$invite_desc' => "", + '$emailnet' => "", + '$submit' => t('Submit Request'), + '$cancel' => t('Cancel'), + '$nickname' => "", + '$name' => $ret["name"], + '$url' => $ret["url"], + '$myaddr' => $myaddr, + '$request' => $request + )); + return $o; +} + +function follow_post(&$a) { + + if(! local_user()) { + notice( t('Permission denied.') . EOL); + goaway($_SESSION['return_url']); + // NOTREACHED + } + + if ($_REQUEST['cancel']) + goaway($_SESSION['return_url']); + $uid = local_user(); $url = notags(trim($_REQUEST['url'])); $return_url = $_SESSION['return_url']; + // Makes the connection request for friendica contacts easier + // This is just a precaution if maybe this page is called somewhere directly via POST + $_SESSION["fastlane"] = $url; $result = new_contact($uid,$url,true); @@ -22,12 +97,13 @@ function follow_init(&$a) { if($result['message']) notice($result['message']); goaway($return_url); - } + } elseif ($result['cid']) + goaway($a->get_baseurl().'/contacts/'.$result['cid']); - info( t('Contact added') . EOL); + info( t('Contact added').EOL); if(strstr($return_url,'contacts')) - goaway($a->get_baseurl() . '/contacts/' . $contact_id); + goaway($a->get_baseurl().'/contacts/'.$contact_id); goaway($return_url); // NOTREACHED diff --git a/view/templates/auto_request.tpl b/view/templates/auto_request.tpl index b987b7849..56653c655 100644 --- a/view/templates/auto_request.tpl +++ b/view/templates/auto_request.tpl @@ -20,16 +20,27 @@

{{/if}} +{{if $request}} +
+{{else}} +{{/if}} + +{{if $photo}} + +{{/if}}
{{if $myaddr}} {{$myaddr}} - + {{else}} {{/if}} + {{if $url}} + + {{/if}}
diff --git a/view/templates/dfrn_request.tpl b/view/templates/dfrn_request.tpl index 44c8ef1e6..6c63c5331 100644 --- a/view/templates/dfrn_request.tpl +++ b/view/templates/dfrn_request.tpl @@ -1,5 +1,3 @@ - -

{{$header}}

{{if $myaddr}} @@ -19,7 +17,15 @@

{{/if}} +{{if $request}} + +{{else}} +{{/if}} + +{{if $photo}} + +{{/if}}
@@ -29,6 +35,9 @@ {{else}} {{/if}} + {{if $url}} + + {{/if}}
diff --git a/view/templates/follow.tpl b/view/templates/follow.tpl index c4d188765..ece5ed17b 100644 --- a/view/templates/follow.tpl +++ b/view/templates/follow.tpl @@ -2,7 +2,7 @@

{{$connect}}

{{$desc}}
- +
From 015ebd7d1bef1dfbda1cc62a8e846b567a0ecbb3 Mon Sep 17 00:00:00 2001 From: Michael Vogel Date: Thu, 9 Apr 2015 00:33:45 +0200 Subject: [PATCH 2/4] "fastlane" should now work with remote adding as well. --- boot.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/boot.php b/boot.php index 6da885117..eb5c7e5cb 100644 --- a/boot.php +++ b/boot.php @@ -1695,6 +1695,8 @@ if(! function_exists('profile_sidebar')) { if($a->theme['template_engine'] === 'internal') $location = template_escape($location); + if ($connect) + $_SESSION["fastlane"] = $p["url"]; $tpl = get_markup_template('profile_vcard.tpl'); $o .= replace_macros($tpl, array( From ccbac6874baa8b82c2d0cede825c7f3572d3ea92 Mon Sep 17 00:00:00 2001 From: Michael Vogel Date: Thu, 9 Apr 2015 07:45:48 +0200 Subject: [PATCH 3/4] The "connect" link ar the display page now directs to the new "follow" page as well. --- mod/display.php | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/mod/display.php b/mod/display.php index f483977cd..02ff37bf8 100644 --- a/mod/display.php +++ b/mod/display.php @@ -193,11 +193,13 @@ function display_fetchauthor($a, $item) { } if (local_user()) { - if ($profiledata["network"] == NETWORK_DFRN) { - $connect = str_replace("/profile/", "/dfrn_request/", $profiledata["url"])."&addr=".bin2hex($a->get_baseurl()."/profile/".$a->user["nickname"]); - $profiledata["remoteconnect"] = $connect; - } elseif ($profiledata["network"] == NETWORK_DIASPORA) - $profiledata["remoteconnect"] = $a->get_baseurl()."/contacts?add=".GetProfileUsername($profiledata["url"], "", true); + if (in_array($profiledata["network"], array(NETWORK_DFRN, NETWORK_DIASPORA, NETWORK_OSTATUS))) + $profiledata["remoteconnect"] = $a->get_baseurl()."/follow?url=".urlencode($profiledata["url"]); + //if ($profiledata["network"] == NETWORK_DFRN) { + // $connect = str_replace("/profile/", "/dfrn_request/", $profiledata["url"])."&addr=".bin2hex($a->get_baseurl()."/profile/".$a->user["nickname"]); + // $profiledata["remoteconnect"] = $connect; + //} elseif ($profiledata["network"] == NETWORK_DIASPORA) + // $profiledata["remoteconnect"] = $a->get_baseurl()."/contacts?add=".GetProfileUsername($profiledata["url"], "", true); } elseif ($profiledata["network"] == NETWORK_DFRN) { $connect = str_replace("/profile/", "/dfrn_request/", $profiledata["url"]); $profiledata["remoteconnect"] = $connect; From 98bf8d85fa81ada5096104a97ed206f86081f534 Mon Sep 17 00:00:00 2001 From: Michael Vogel Date: Thu, 9 Apr 2015 19:35:17 +0200 Subject: [PATCH 4/4] The "fastlane" can be removed there. --- boot.php | 3 --- 1 file changed, 3 deletions(-) diff --git a/boot.php b/boot.php index eb5c7e5cb..cb380ed99 100644 --- a/boot.php +++ b/boot.php @@ -1695,9 +1695,6 @@ if(! function_exists('profile_sidebar')) { if($a->theme['template_engine'] === 'internal') $location = template_escape($location); - if ($connect) - $_SESSION["fastlane"] = $p["url"]; - $tpl = get_markup_template('profile_vcard.tpl'); $o .= replace_macros($tpl, array( '$profile' => $p,