]> git.mxchange.org Git - friendica.git/commitdiff
Fix formatting in dfrn_confirm
authorHypolite Petovan <mrpetovan@gmail.com>
Sat, 13 Jan 2018 04:23:51 +0000 (23:23 -0500)
committerHypolite Petovan <mrpetovan@gmail.com>
Sun, 14 Jan 2018 04:46:58 +0000 (23:46 -0500)
- Add missing braces
- Use dba::selectFirst whenever possible
- Initialize missing variables
- Remove unused variables
- Use defaults instead of ternary operator
- Simplify sprintf(t()) structure

mod/dfrn_confirm.php

index a5f5f1bd34b6cc9bea72aad5ea22381c5f08cf7b..8baacf0931e5a54da053e1377039bbb93c0a5359 100644 (file)
@@ -4,15 +4,15 @@
  * @file mod/dfrn_confirm.php
  * @brief Module: dfrn_confirm
  * Purpose: Friendship acceptance for DFRN contacts
- *.
+ *
  * There are two possible entry points and three scenarios.
- *.
+ *
  *   1. A form was submitted by our user approving a friendship that originated elsewhere.
  *      This may also be called from dfrn_request to automatically approve a friendship.
  *
  *   2. We may be the target or other side of the conversation to scenario 1, and will
  *      interact with that process on our own user's behalf.
- *.
+ *
  *  @see PDF with dfrn specs: https://github.com/friendica/friendica/blob/master/spec/dfrn2.pdf
  *    You also find a graphic which describes the confirmation process at
  *    https://github.com/friendica/friendica/blob/master/spec/dfrn2_contact_confirmation.png
@@ -32,110 +32,100 @@ use Friendica\Protocol\Diaspora;
 use Friendica\Util\Crypto;
 
 require_once 'include/enotify.php';
+require_once 'include/items.php';
 
-function dfrn_confirm_post(App $a, $handsfree = null) {
-
-       if(is_array($handsfree)) {
-
+function dfrn_confirm_post(App $a, $handsfree = null)
+{
+       $node = null;
+       if (is_array($handsfree)) {
                /*
                 * We were called directly from dfrn_request due to automatic friend acceptance.
                 * Any $_POST parameters we may require are supplied in the $handsfree array.
                 *
                 */
-
                $node = $handsfree['node'];
                $a->interactive = false; // notice() becomes a no-op since nobody is there to see it
-
-       }
-       else {
-               if($a->argc > 1)
-                       $node = $a->argv[1];
+       } elseif ($a->argc > 1) {
+               $node = $a->argv[1];
        }
 
-               /*
-                *
-                * Main entry point. Scenario 1. Our user received a friend request notification (perhaps
-                * from another site) and clicked 'Approve'.
-                * $POST['source_url'] is not set. If it is, it indicates Scenario 2.
-                *
-                * We may also have been called directly from dfrn_request ($handsfree != null) due to
-                * this being a page type which supports automatic friend acceptance. That is also Scenario 1
-                * since we are operating on behalf of our registered user to approve a friendship.
-                *
-                */
-
-       if(! x($_POST,'source_url')) {
-
-               $uid = ((is_array($handsfree)) ? $handsfree['uid'] : local_user());
-
-               if(! $uid) {
-                       notice( t('Permission denied.') . EOL );
+       /*
+        * Main entry point. Scenario 1. Our user received a friend request notification (perhaps
+        * from another site) and clicked 'Approve'.
+        * $POST['source_url'] is not set. If it is, it indicates Scenario 2.
+        *
+        * We may also have been called directly from dfrn_request ($handsfree != null) due to
+        * this being a page type which supports automatic friend acceptance. That is also Scenario 1
+        * since we are operating on behalf of our registered user to approve a friendship.
+        */
+       if (!x($_POST, 'source_url')) {
+               $uid = defaults($handsfree, 'uid', local_user());
+               if (!$uid) {
+                       notice(t('Permission denied.') . EOL);
                        return;
                }
 
-               $user = q("SELECT * FROM `user` WHERE `uid` = %d LIMIT 1",
-                       intval($uid)
-               );
-
-               if(! $user) {
-                       notice( t('Profile not found.') . EOL );
+               $user = dba::selectFirst('user', [], ['uid' => $uid]);
+               if (!DBM::is_result($user)) {
+                       notice(t('Profile not found.') . EOL);
                        return;
                }
 
-
                // These data elements may come from either the friend request notification form or $handsfree array.
-
-               if(is_array($handsfree)) {
+               if (is_array($handsfree)) {
                        logger('Confirm in handsfree mode');
-                       $dfrn_id   = $handsfree['dfrn_id'];
-                       $intro_id  = $handsfree['intro_id'];
-                       $duplex    = $handsfree['duplex'];
-                       $hidden    = ((array_key_exists('hidden',$handsfree)) ? intval($handsfree['hidden']) : 0 );
-                       $activity  = ((array_key_exists('activity',$handsfree)) ? intval($handsfree['activity']) : 0 );
-               }
-               else {
-                       $dfrn_id  = ((x($_POST,'dfrn_id'))    ? notags(trim($_POST['dfrn_id'])) : "");
-                       $intro_id = ((x($_POST,'intro_id'))   ? intval($_POST['intro_id'])      : 0 );
-                       $duplex   = ((x($_POST,'duplex'))     ? intval($_POST['duplex'])        : 0 );
-                       $cid      = ((x($_POST,'contact_id')) ? intval($_POST['contact_id'])    : 0 );
-                       $hidden   = ((x($_POST,'hidden'))     ? intval($_POST['hidden'])        : 0 );
-                       $activity = ((x($_POST,'activity'))   ? intval($_POST['activity'])      : 0 );
+                       $dfrn_id  = $handsfree['dfrn_id'];
+                       $intro_id = $handsfree['intro_id'];
+                       $duplex   = $handsfree['duplex'];
+                       $cid      = 0;
+                       $hidden   = intval(defaults($handsfree, 'hidden'  , 0));
+                       $activity = intval(defaults($handsfree, 'activity', 0));
+               else {
+                       $dfrn_id  = notags(trim(defaults($_POST, 'dfrn_id'   , '')));
+                       $intro_id =      intval(defaults($_POST, 'intro_id'  , 0));
+                       $duplex   =      intval(defaults($_POST, 'duplex'    , 0));
+                       $cid      =      intval(defaults($_POST, 'contact_id', 0));
+                       $hidden   =      intval(defaults($_POST, 'hidden'    , 0));
+                       $activity =      intval(defaults($_POST, 'activity'  , 0));
                }
 
                /*
-                *
                 * Ensure that dfrn_id has precedence when we go to find the contact record.
                 * We only want to search based on contact id if there is no dfrn_id,
                 * e.g. for OStatus network followers.
-                *
                 */
-
-               if(strlen($dfrn_id))
+               if (strlen($dfrn_id)) {
                        $cid = 0;
+               }
 
                logger('Confirming request for dfrn_id (issued) ' . $dfrn_id);
-               if($cid)
+               if ($cid) {
                        logger('Confirming follower with contact_id: ' . $cid);
-
+               }
 
                /*
-                *
                 * The other person will have been issued an ID when they first requested friendship.
                 * Locate their record. At this time, their record will have both pending and blocked set to 1.
                 * There won't be any dfrn_id if this is a network follower, so use the contact_id instead.
-                *
                 */
-
-               $r = q("SELECT * FROM `contact` WHERE ( ( `issued-id` != '' AND `issued-id` = '%s' ) OR ( `id` = %d AND `id` != 0 ) ) AND `uid` = %d AND `duplex` = 0 LIMIT 1",
+               $r = q("SELECT *
+                       FROM `contact`
+                       WHERE (
+                               (`issued-id` != '' AND `issued-id` = '%s')
+                               OR
+                               (`id` = %d AND `id` != 0)
+                       )
+                       AND `uid` = %d
+                       AND `duplex` = 0
+                       LIMIT 1",
                        dbesc($dfrn_id),
                        intval($cid),
                        intval($uid)
                );
-
-               if (! DBM::is_result($r)) {
+               if (!DBM::is_result($r)) {
                        logger('Contact not found in DB.');
-                       notice( t('Contact not found.') . EOL );
-                       notice( t('This may occasionally happen if contact was requested by both persons and it has already been approved.') . EOL );
+                       notice(t('Contact not found.') . EOL);
+                       notice(t('This may occasionally happen if contact was requested by both persons and it has already been approved.') . EOL);
                        return;
                }
 
@@ -149,38 +139,33 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
 
                $network = ((strlen($contact['issued-id'])) ? NETWORK_DFRN : NETWORK_OSTATUS);
 
-               if($contact['network'])
+               if ($contact['network']) {
                        $network = $contact['network'];
+               }
 
-               if($network === NETWORK_DFRN) {
-
+               if ($network === NETWORK_DFRN) {
                        /*
-                        *
                         * Generate a key pair for all further communications with this person.
                         * We have a keypair for every contact, and a site key for unknown people.
                         * This provides a means to carry on relationships with other people if
                         * any single key is compromised. It is a robust key. We're much more
                         * worried about key leakage than anybody cracking it.
-                        *
                         */
                        $res = Crypto::newKeypair(4096);
 
-
                        $private_key = $res['prvkey'];
                        $public_key  = $res['pubkey'];
 
                        // Save the private key. Send them the public key.
-
-                       $r = q("UPDATE `contact` SET `prvkey` = '%s' WHERE `id` = %d AND `uid` = %d",
+                       q("UPDATE `contact` SET `prvkey` = '%s' WHERE `id` = %d AND `uid` = %d",
                                dbesc($private_key),
                                intval($contact_id),
                                intval($uid)
                        );
 
-                       $params = array();
+                       $params = [];
 
                        /*
-                        *
                         * Per the DFRN protocol, we will verify both ends by encrypting the dfrn_id with our
                         * site private key (person on the other end can decrypt it with our site public key).
                         * Then encrypt our profile URL with the other person's site public key. They can decrypt
@@ -191,39 +176,41 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
                         * If other site is willing to accept full encryption, (aes_allow is 1 AND we have php5.3
                         * or later) then we encrypt the personal public key we send them using AES-256-CBC and a
                         * random key which is encrypted with their site public key.
-                        *
                         */
 
                        $src_aes_key = openssl_random_pseudo_bytes(64);
 
                        $result = '';
-                       openssl_private_encrypt($dfrn_id, $result, $user[0]['prvkey']);
+                       openssl_private_encrypt($dfrn_id, $result, $user['prvkey']);
 
                        $params['dfrn_id'] = bin2hex($result);
                        $params['public_key'] = $public_key;
 
-
-                       $my_url = System::baseUrl() . '/profile/' . $user[0]['nickname'];
+                       $my_url = System::baseUrl() . '/profile/' . $user['nickname'];
 
                        openssl_public_encrypt($my_url, $params['source_url'], $site_pubkey);
                        $params['source_url'] = bin2hex($params['source_url']);
 
-                       if($aes_allow && function_exists('openssl_encrypt')) {
+                       if ($aes_allow && function_exists('openssl_encrypt')) {
                                openssl_public_encrypt($src_aes_key, $params['aes_key'], $site_pubkey);
                                $params['aes_key'] = bin2hex($params['aes_key']);
-                               $params['public_key'] = bin2hex(openssl_encrypt($public_key,'AES-256-CBC',$src_aes_key));
+                               $params['public_key'] = bin2hex(openssl_encrypt($public_key, 'AES-256-CBC', $src_aes_key));
                        }
 
-                       $params['dfrn_version'] = DFRN_PROTOCOL_VERSION ;
-                       if($duplex == 1)
+                       $params['dfrn_version'] = DFRN_PROTOCOL_VERSION;
+                       if ($duplex == 1) {
                                $params['duplex'] = 1;
+                       }
 
-                       if($user[0]['page-flags'] == PAGE_COMMUNITY)
+                       if ($user['page-flags'] == PAGE_COMMUNITY) {
                                $params['page'] = 1;
-                       if($user[0]['page-flags'] == PAGE_PRVGROUP)
+                       }
+
+                       if ($user['page-flags'] == PAGE_PRVGROUP) {
                                $params['page'] = 2;
+                       }
 
-                       logger('Confirm: posting data to ' . $dfrn_confirm . ': ' . print_r($params,true), LOGGER_DATA);
+                       logger('Confirm: posting data to ' . $dfrn_confirm . ': ' . print_r($params, true), LOGGER_DATA);
 
                        /*
                         *
@@ -240,104 +227,91 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
                        // Now figure out what they responded. Try to be robust if the remote site is
                        // having difficulty and throwing up errors of some kind.
 
-                       $leading_junk = substr($res,0,strpos($res,'<?xml'));
+                       $leading_junk = substr($res, 0, strpos($res, '<?xml'));
 
-                       $res = substr($res,strpos($res,'<?xml'));
-                       if(! strlen($res)) {
-
-                                       // No XML at all, this exchange is messed up really bad.
-                                       // We shouldn't proceed, because the xml parser might choke,
-                                       // and $status is going to be zero, which indicates success.
-                                       // We can hardly call this a success.
-
-                               notice( t('Response from remote site was not understood.') . EOL);
+                       $res = substr($res, strpos($res, '<?xml'));
+                       if (!strlen($res)) {
+                               // No XML at all, this exchange is messed up really bad.
+                               // We shouldn't proceed, because the xml parser might choke,
+                               // and $status is going to be zero, which indicates success.
+                               // We can hardly call this a success.
+                               notice(t('Response from remote site was not understood.') . EOL);
                                return;
                        }
 
-                       if(strlen($leading_junk) && Config::get('system','debugging')) {
-
-                                       // This might be more common. Mixed error text and some XML.
-                                       // If we're configured for debugging, show the text. Proceed in either case.
-
-                               notice( t('Unexpected response from remote site: ') . EOL . $leading_junk . EOL );
+                       if (strlen($leading_junk) && Config::get('system', 'debugging')) {
+                               // This might be more common. Mixed error text and some XML.
+                               // If we're configured for debugging, show the text. Proceed in either case.
+                               notice(t('Unexpected response from remote site: ') . EOL . $leading_junk . EOL);
                        }
 
-                       if(stristr($res, "<status")===false) {
+                       if (stristr($res, "<status") === false) {
                                // wrong xml! stop here!
-                               notice( t('Unexpected response from remote site: ') . EOL . htmlspecialchars($res) . EOL );
+                               notice(t('Unexpected response from remote site: ') . EOL . htmlspecialchars($res) . EOL);
                                return;
                        }
 
                        $xml = parse_xml_string($res);
                        $status = (int) $xml->status;
                        $message = unxmlify($xml->message);   // human readable text of what may have gone wrong.
-                       switch($status) {
+                       switch ($status) {
                                case 0:
-                                       info( t("Confirmation completed successfully.") . EOL);
-                                       if(strlen($message))
-                                               notice( t('Remote site reported: ') . $message . EOL);
+                                       info(t("Confirmation completed successfully.") . EOL);
                                        break;
                                case 1:
                                        // birthday paradox - generate new dfrn-id and fall through.
                                        $new_dfrn_id = random_string();
-                                       $r = q("UPDATE contact SET `issued-id` = '%s' WHERE `id` = %d AND `uid` = %d",
+                                       q("UPDATE contact SET `issued-id` = '%s' WHERE `id` = %d AND `uid` = %d",
                                                dbesc($new_dfrn_id),
                                                intval($contact_id),
                                                intval($uid)
                                        );
 
                                case 2:
-                                       notice( t("Temporary failure. Please wait and try again.") . EOL);
-                                       if(strlen($message))
-                                               notice( t('Remote site reported: ') . $message . EOL);
+                                       notice(t("Temporary failure. Please wait and try again.") . EOL);
                                        break;
-
-
                                case 3:
-                                       notice( t("Introduction failed or was revoked.") . EOL);
-                                       if(strlen($message))
-                                               notice( t('Remote site reported: ') . $message . EOL);
+                                       notice(t("Introduction failed or was revoked.") . EOL);
                                        break;
-                               }
+                       }
 
-                       if(($status == 0) && ($intro_id)) {
+                       if (strlen($message)) {
+                               notice(t('Remote site reported: ') . $message . EOL);
+                       }
 
+                       if (($status == 0) && ($intro_id)) {
                                // Success. Delete the notification.
-
-                               $r = q("DELETE FROM `intro` WHERE `id` = %d AND `uid` = %d",
+                               q("DELETE FROM `intro` WHERE `id` = %d AND `uid` = %d",
                                        intval($intro_id),
                                        intval($uid)
                                );
-
                        }
 
-                       if($status != 0)
+                       if ($status != 0) {
                                return;
+                       }
                }
 
-
                /*
-                *
                 * We have now established a relationship with the other site.
                 * Let's make our own personal copy of their profile photo so we don't have
                 * to always load it from their site.
                 *
                 * We will also update the contact record with the nature and scope of the relationship.
-                *
                 */
-
                Contact::updateAvatar($contact['photo'], $uid, $contact_id);
 
                logger('dfrn_confirm: confirm - imported photos');
 
-               if($network === NETWORK_DFRN) {
-
+               if ($network === NETWORK_DFRN) {
                        $new_relation = CONTACT_IS_FOLLOWER;
-                       if(($relation == CONTACT_IS_SHARING) || ($duplex))
+                       if (($relation == CONTACT_IS_SHARING) || ($duplex)) {
                                $new_relation = CONTACT_IS_FRIEND;
+                       }
 
-                       if(($relation == CONTACT_IS_SHARING) && ($duplex))
+                       if (($relation == CONTACT_IS_SHARING) && ($duplex)) {
                                $duplex = 0;
+                       }
 
                        $r = q("UPDATE `contact` SET `rel` = %d,
                                `name-date` = '%s',
@@ -357,42 +331,36 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
                                intval($contact_id)
                        );
                } else {
-
                        // $network !== NETWORK_DFRN
-
-                       $network = (($contact['network']) ? $contact['network'] : NETWORK_OSTATUS);
-                       $notify = (($contact['notify']) ? $contact['notify'] : '');
-                       $poll   = (($contact['poll']) ? $contact['poll'] : '');
+                       $network = defaults($contact, 'network', NETWORK_OSTATUS);
 
                        $arr = Probe::uri($contact['url']);
-                       if (empty($contact['notify'])) {
-                               $notify = $arr['notify'];
-                       }
-                       if (empty($contact['poll'])) {
-                               $poll = $arr['poll'];
-                       }
+
+                       $notify  = defaults($contact, 'notify' , $arr['notify']);
+                       $poll    = defaults($contact, 'poll'   , $arr['poll']);
 
                        $addr = $arr['addr'];
 
                        $new_relation = $contact['rel'];
                        $writable = $contact['writable'];
 
-                       if($network === NETWORK_DIASPORA) {
-                               if($duplex)
+                       if ($network === NETWORK_DIASPORA) {
+                               if ($duplex) {
                                        $new_relation = CONTACT_IS_FRIEND;
-                               else
+                               } else {
                                        $new_relation = CONTACT_IS_FOLLOWER;
+                               }
 
-                               if($new_relation != CONTACT_IS_FOLLOWER)
+                               if ($new_relation != CONTACT_IS_FOLLOWER) {
                                        $writable = 1;
+                               }
                        }
 
-                       $r = q("DELETE FROM `intro` WHERE `id` = %d AND `uid` = %d",
+                       q("DELETE FROM `intro` WHERE `id` = %d AND `uid` = %d",
                                intval($intro_id),
                                intval($uid)
                        );
 
-
                        $r = q("UPDATE `contact` SET `name-date` = '%s',
                                `uri-date` = '%s',
                                `addr` = '%s',
@@ -420,67 +388,43 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
                }
 
                /// @TODO is DBM::is_result() working here?
-               if ($r === false) {
-                       notice( t('Unable to set contact photo.') . EOL);
+               if (!DBM::is_result($r)) {
+                       notice(t('Unable to set contact photo.') . EOL);
                }
 
                // reload contact info
-
-               $r = q("SELECT * FROM `contact` WHERE `id` = %d LIMIT 1",
-                       intval($contact_id)
-               );
-               if (DBM::is_result($r)) {
-                       $contact = $r[0];
-               } else {
-                       $contact = null;
-               }
-
-
+               $contact = dba::selectFirst('contact', [], ['id' => $contact_id]);
                if ((isset($new_relation) && $new_relation == CONTACT_IS_FRIEND)) {
-
-                       if (($contact) && ($contact['network'] === NETWORK_DIASPORA)) {
-                               $ret = Diaspora::sendShare($user[0],$r[0]);
+                       if (DBM::is_result($contact) && ($contact['network'] === NETWORK_DIASPORA)) {
+                               $ret = Diaspora::sendShare($user, $contact);
                                logger('share returns: ' . $ret);
                        }
 
                        // Send a new friend post if we are allowed to...
-
-                       $r = q("SELECT `hide-friends` FROM `profile` WHERE `uid` = %d AND `is-default` = 1 LIMIT 1",
-                               intval($uid)
-                       );
-
-                       if((DBM::is_result($r)) && ($r[0]['hide-friends'] == 0) && ($activity) && (! $hidden)) {
-
-                               require_once 'include/items.php';
-
-                               $self = q("SELECT * FROM `contact` WHERE `self` = 1 AND `uid` = %d LIMIT 1",
-                                       intval($uid)
-                               );
-
-                               if(count($self)) {
-
-                                       $arr = array();
+                       $profile = dba::selectFirst('profile', ['hide-friends'], ['is-default' => true, 'uid' => $uid]);
+                       if (x($profile, 'hide-friends') === 0 && $activity && !$hidden) {
+                               $self = dba::selectFirst('contact', [], ['self' => true, 'uid' => $uid]);
+                               if (DBM::is_result($self)) {
+                                       $arr = [];
                                        $arr['guid'] = get_guid(32);
                                        $arr['uri'] = $arr['parent-uri'] = item_new_uri($a->get_hostname(), $uid);
                                        $arr['uid'] = $uid;
-                                       $arr['contact-id'] = $self[0]['id'];
+                                       $arr['contact-id'] = $self['id'];
                                        $arr['wall'] = 1;
                                        $arr['type'] = 'wall';
                                        $arr['gravity'] = 0;
                                        $arr['origin'] = 1;
-                                       $arr['author-name'] = $arr['owner-name'] = $self[0]['name'];
-                                       $arr['author-link'] = $arr['owner-link'] = $self[0]['url'];
-                                       $arr['author-avatar'] = $arr['owner-avatar'] = $self[0]['thumb'];
-
-                                       $A = '[url=' . $self[0]['url'] . ']' . $self[0]['name'] . '[/url]';
-                                       $APhoto = '[url=' . $self[0]['url'] . ']' . '[img]' . $self[0]['thumb'] . '[/img][/url]';
+                                       $arr['author-name']   = $arr['owner-name']   = $self['name'];
+                                       $arr['author-link']   = $arr['owner-link']   = $self['url'];
+                                       $arr['author-avatar'] = $arr['owner-avatar'] = $self['thumb'];
 
+                                       $A = '[url=' . $self['url'] . ']' . $self['name'] . '[/url]';
                                        $B = '[url=' . $contact['url'] . ']' . $contact['name'] . '[/url]';
                                        $BPhoto = '[url=' . $contact['url'] . ']' . '[img]' . $contact['thumb'] . '[/img][/url]';
 
                                        $arr['verb'] = ACTIVITY_FRIEND;
                                        $arr['object-type'] = ACTIVITY_OBJ_PERSON;
-                                       $arr['body'] =  sprintf( t('%1$s is now friends with %2$s'), $A, $B)."\n\n\n".$BPhoto;
+                                       $arr['body'] = t('%1$s is now friends with %2$s', $A, $B) . "\n\n\n" . $BPhoto;
 
                                        $arr['object'] = '<object><type>' . ACTIVITY_OBJ_PERSON . '</type><title>' . $contact['name'] . '</title>'
                                                . '<id>' . $contact['url'] . '/' . $contact['name'] . '</id>';
@@ -490,14 +434,15 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
 
                                        $arr['last-child'] = 1;
 
-                                       $arr['allow_cid'] = $user[0]['allow_cid'];
-                                       $arr['allow_gid'] = $user[0]['allow_gid'];
-                                       $arr['deny_cid']  = $user[0]['deny_cid'];
-                                       $arr['deny_gid']  = $user[0]['deny_gid'];
+                                       $arr['allow_cid'] = $user['allow_cid'];
+                                       $arr['allow_gid'] = $user['allow_gid'];
+                                       $arr['deny_cid']  = $user['deny_cid'];
+                                       $arr['deny_gid']  = $user['deny_gid'];
 
                                        $i = item_store($arr);
-                                       if($i)
+                                       if ($i) {
                                                Worker::add(PRIORITY_HIGH, "Notifier", "activity", $i);
+                                       }
                                }
                        }
                }
@@ -515,35 +460,28 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
        }
 
        /*
-        *
-        *
         * End of Scenario 1. [Local confirmation of remote friend request].
         *
         * Begin Scenario 2. This is the remote response to the above scenario.
         * This will take place on the site that originally initiated the friend request.
         * In the section above where the confirming party makes a POST and
         * retrieves xml status information, they are communicating with the following code.
-        *
         */
-
-       if (x($_POST,'source_url')) {
-
+       if (x($_POST, 'source_url')) {
                // We are processing an external confirmation to an introduction created by our user.
-
-               $public_key = ((x($_POST,'public_key'))   ? $_POST['public_key']           : '');
-               $dfrn_id    = ((x($_POST,'dfrn_id'))      ? hex2bin($_POST['dfrn_id'])     : '');
-               $source_url = ((x($_POST,'source_url'))   ? hex2bin($_POST['source_url'])  : '');
-               $aes_key    = ((x($_POST,'aes_key'))      ? $_POST['aes_key']              : '');
-               $duplex     = ((x($_POST,'duplex'))       ? intval($_POST['duplex'])       : 0 );
-               $page       = ((x($_POST,'page'))         ? intval($_POST['page'])         : 0 );
-               $version_id = ((x($_POST,'dfrn_version')) ? (float) $_POST['dfrn_version'] : 2.0);
+               $public_key =         defaults($_POST, 'public_key', '');
+               $dfrn_id    = hex2bin(defaults($_POST, 'dfrn_id'   , ''));
+               $source_url = hex2bin(defaults($_POST, 'source_url', ''));
+               $aes_key    =         defaults($_POST, 'aes_key'   , '');
+               $duplex     =  intval(defaults($_POST, 'duplex'    , 0));
+               $page       =  intval(defaults($_POST, 'page'      , 0));
 
                $forum = (($page == 1) ? 1 : 0);
                $prv   = (($page == 2) ? 1 : 0);
 
                logger('dfrn_confirm: requestee contacted: ' . $node);
 
-               logger('dfrn_confirm: request: POST=' . print_r($_POST,true), LOGGER_DATA);
+               logger('dfrn_confirm: request: POST=' . print_r($_POST, true), LOGGER_DATA);
 
                // If $aes_key is set, both of these items require unpacking from the hex transport encoding.
 
@@ -553,90 +491,77 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
                }
 
                // Find our user's account
-
-               $r = q("SELECT * FROM `user` WHERE `nickname` = '%s' LIMIT 1",
-                       dbesc($node));
-
-               if (! DBM::is_result($r)) {
-                       $message = sprintf(t('No user record found for \'%s\' '), $node);
-                       xml_status(3,$message); // failure
+               $user = dba::selectFirst('user', [], ['nickname' => $node]);
+               if (!DBM::is_result($user)) {
+                       $message = t('No user record found for \'%s\' ', $node);
+                       xml_status(3, $message); // failure
                        // NOTREACHED
                }
 
-               $my_prvkey = $r[0]['prvkey'];
-               $local_uid = $r[0]['uid'];
+               $my_prvkey = $user['prvkey'];
+               $local_uid = $user['uid'];
 
 
-               if(! strstr($my_prvkey,'PRIVATE KEY')) {
+               if (!strstr($my_prvkey, 'PRIVATE KEY')) {
                        $message = t('Our site encryption key is apparently messed up.');
-                       xml_status(3,$message);
+                       xml_status(3, $message);
                }
 
                // verify everything
 
                $decrypted_source_url = "";
-               openssl_private_decrypt($source_url,$decrypted_source_url,$my_prvkey);
+               openssl_private_decrypt($source_url, $decrypted_source_url, $my_prvkey);
 
 
-               if(! strlen($decrypted_source_url)) {
+               if (!strlen($decrypted_source_url)) {
                        $message = t('Empty site URL was provided or URL could not be decrypted by us.');
-                       xml_status(3,$message);
+                       xml_status(3, $message);
                        // NOTREACHED
                }
 
-               $ret = q("SELECT * FROM `contact` WHERE `url` = '%s' AND `uid` = %d LIMIT 1",
-                       dbesc($decrypted_source_url),
-                       intval($local_uid)
-               );
-               if (!DBM::is_result($ret)) {
-                       if (strstr($decrypted_source_url,'http:')) {
-                               $newurl = str_replace('http:','https:',$decrypted_source_url);
+               $contact = dba::selectFirst('contact', [], ['url' => $decrypted_source_url, 'uid' => $local_uid]);
+               if (!DBM::is_result($contact)) {
+                       if (strstr($decrypted_source_url, 'http:')) {
+                               $newurl = str_replace('http:', 'https:', $decrypted_source_url);
                        } else {
-                               $newurl = str_replace('https:','http:',$decrypted_source_url);
+                               $newurl = str_replace('https:', 'http:', $decrypted_source_url);
                        }
 
-                       $ret = q("SELECT * FROM `contact` WHERE `url` = '%s' AND `uid` = %d LIMIT 1",
-                               dbesc($newurl),
-                               intval($local_uid)
-                       );
-                       if (!DBM::is_result($ret)) {
+                       $contact = dba::selectFirst('contact', [], ['url' => $newurl, 'uid' => $local_uid]);
+                       if (!DBM::is_result($contact)) {
                                // this is either a bogus confirmation (?) or we deleted the original introduction.
                                $message = t('Contact record was not found for you on our site.');
-                               xml_status(3,$message);
+                               xml_status(3, $message);
                                return; // NOTREACHED
                        }
                }
 
-               $relation = $ret[0]['rel'];
+               $relation = $contact['rel'];
 
                // Decrypt all this stuff we just received
 
-               $foreign_pubkey = $ret[0]['site-pubkey'];
-               $dfrn_record    = $ret[0]['id'];
+               $foreign_pubkey = $contact['site-pubkey'];
+               $dfrn_record = $contact['id'];
 
-               if (! $foreign_pubkey) {
-                       $message = sprintf( t('Site public key not available in contact record for URL %s.'), $newurl);
-                       xml_status(3,$message);
+               if (!$foreign_pubkey) {
+                       $message = t('Site public key not available in contact record for URL %s.', $decrypted_source_url);
+                       xml_status(3, $message);
                }
 
                $decrypted_dfrn_id = "";
-               openssl_public_decrypt($dfrn_id,$decrypted_dfrn_id,$foreign_pubkey);
+               openssl_public_decrypt($dfrn_id, $decrypted_dfrn_id, $foreign_pubkey);
 
                if (strlen($aes_key)) {
                        $decrypted_aes_key = "";
-                       openssl_private_decrypt($aes_key,$decrypted_aes_key,$my_prvkey);
-                       $dfrn_pubkey = openssl_decrypt($public_key,'AES-256-CBC',$decrypted_aes_key);
-               }
-               else {
+                       openssl_private_decrypt($aes_key, $decrypted_aes_key, $my_prvkey);
+                       $dfrn_pubkey = openssl_decrypt($public_key, 'AES-256-CBC', $decrypted_aes_key);
+               } else {
                        $dfrn_pubkey = $public_key;
                }
 
-               $r = q("SELECT * FROM `contact` WHERE `dfrn-id` = '%s' LIMIT 1",
-                       dbesc($decrypted_dfrn_id)
-               );
-               if (DBM::is_result($r)) {
+               if (dba::exists('contact', ['dfrn-id' => $decrypted_dfrn_id])) {
                        $message = t('The ID provided by your system is a duplicate on our system. It should work if you try again.');
-                       xml_status(1,$message); // Birthday paradox - duplicate dfrn-id
+                       xml_status(1, $message); // Birthday paradox - duplicate dfrn-id
                        // NOTREACHED
                }
 
@@ -645,34 +570,29 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
                        dbesc($dfrn_pubkey),
                        intval($dfrn_record)
                );
-               if (! DBM::is_result($r)) {
+               if (!DBM::is_result($r)) {
                        $message = t('Unable to set your contact credentials on our system.');
-                       xml_status(3,$message);
+                       xml_status(3, $message);
                }
 
                // It's possible that the other person also requested friendship.
                // If it is a duplex relationship, ditch the issued-id if one exists.
 
-               if($duplex) {
-                       $r = q("UPDATE `contact` SET `issued-id` = '' WHERE `id` = %d",
+               if ($duplex) {
+                       q("UPDATE `contact` SET `issued-id` = '' WHERE `id` = %d",
                                intval($dfrn_record)
                        );
                }
 
                // We're good but now we have to scrape the profile photo and send notifications.
-
-
-
-               $r = q("SELECT `photo` FROM `contact` WHERE `id` = %d LIMIT 1",
-                       intval($dfrn_record));
-
-               if (DBM::is_result($r)) {
-                       $photo = $r[0]['photo'];
+               $contact = dba::selectFirst('contact', ['photo'], ['id' => $dfrn_record]);
+               if (DBM::is_result($contact)) {
+                       $photo = $contact['photo'];
                } else {
                        $photo = System::baseUrl() . '/images/person-175.jpg';
                }
 
-               Contact::updateAvatar($photo,$local_uid,$dfrn_record);
+               Contact::updateAvatar($photo, $local_uid, $dfrn_record);
 
                logger('dfrn_confirm: request - photos imported');
 
@@ -705,9 +625,9 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
                        dbesc(NETWORK_DFRN),
                        intval($dfrn_record)
                );
-               if ($r === false) {    // indicates schema is messed up or total db failure
+               if (!DBM::is_result($r)) {      // indicates schema is messed up or total db failure
                        $message = t('Unable to update your contact profile details on our system');
-                       xml_status(3,$message);
+                       xml_status(3, $message);
                }
 
                // Otherwise everything seems to have worked and we are almost done. Yay!
@@ -715,70 +635,61 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
 
                logger('dfrn_confirm: request: info updated');
 
-               $r = q("SELECT `contact`.*, `user`.* FROM `contact` LEFT JOIN `user` ON `contact`.`uid` = `user`.`uid`
-                       WHERE `contact`.`id` = %d LIMIT 1",
+               $combined = null;
+               $r = q("SELECT `contact`.*, `user`.*
+                       FROM `contact`
+                       LEFT JOIN `user` ON `contact`.`uid` = `user`.`uid`
+                       WHERE `contact`.`id` = %d
+                       LIMIT 1",
                        intval($dfrn_record)
                );
-
-               if (DBM::is_result($r))
+               if (DBM::is_result($r)) {
                        $combined = $r[0];
 
-               if((DBM::is_result($r)) && ($r[0]['notify-flags'] & NOTIFY_CONFIRM)) {
-                       $mutual = ($new_relation == CONTACT_IS_FRIEND);
-                       notification(array(
-                               'type'         => NOTIFY_CONFIRM,
-                               'notify_flags' => $r[0]['notify-flags'],
-                               'language'     => $r[0]['language'],
-                               'to_name'      => $r[0]['username'],
-                               'to_email'     => $r[0]['email'],
-                               'uid'          => $r[0]['uid'],
-                               'link'             => System::baseUrl() . '/contacts/' . $dfrn_record,
-                               'source_name'  => ((strlen(stripslashes($r[0]['name']))) ? stripslashes($r[0]['name']) : t('[Name Withheld]')),
-                               'source_link'  => $r[0]['url'],
-                               'source_photo' => $r[0]['photo'],
-                               'verb'         => ($mutual?ACTIVITY_FRIEND:ACTIVITY_FOLLOW),
-                               'otype'        => 'intro'
-                       ));
+                       if ($combined['notify-flags'] & NOTIFY_CONFIRM) {
+                               $mutual = ($new_relation == CONTACT_IS_FRIEND);
+                               notification([
+                                       'type'         => NOTIFY_CONFIRM,
+                                       'notify_flags' => $combined['notify-flags'],
+                                       'language'     => $combined['language'],
+                                       'to_name'      => $combined['username'],
+                                       'to_email'     => $combined['email'],
+                                       'uid'          => $combined['uid'],
+                                       'link'         => System::baseUrl() . '/contacts/' . $dfrn_record,
+                                       'source_name'  => ((strlen(stripslashes($combined['name']))) ? stripslashes($combined['name']) : t('[Name Withheld]')),
+                                       'source_link'  => $combined['url'],
+                                       'source_photo' => $combined['photo'],
+                                       'verb'         => ($mutual?ACTIVITY_FRIEND:ACTIVITY_FOLLOW),
+                                       'otype'        => 'intro'
+                               ]);
+                       }
                }
 
                // Send a new friend post if we are allowed to...
-
-               if($page && intval(PConfig::get($local_uid,'system','post_joingroup'))) {
-                       $r = q("SELECT `hide-friends` FROM `profile` WHERE `uid` = %d AND `is-default` = 1 LIMIT 1",
-                               intval($local_uid)
-                       );
-
-                       if((DBM::is_result($r)) && ($r[0]['hide-friends'] == 0)) {
-
-                               require_once 'include/items.php';
-
-                               $self = q("SELECT * FROM `contact` WHERE `self` = 1 AND `uid` = %d LIMIT 1",
-                                       intval($local_uid)
-                               );
-
-                               if(count($self)) {
-
-                                       $arr = array();
+               if ($page && intval(PConfig::get($local_uid, 'system', 'post_joingroup'))) {
+                       $profile = dba::selectFirst('profile', ['hide-friends'], ['is-default' => true, 'uid' => $local_uid]);
+                       if (x($profile, 'hide-friends') === 0) {
+                               $self = dba::selectFirst('contact', [], ['self' => true, 'uid' => $local_uid]);
+                               if (DBM::is_result($self)) {
+                                       $arr = [];
                                        $arr['uri'] = $arr['parent-uri'] = item_new_uri($a->get_hostname(), $local_uid);
                                        $arr['uid'] = $local_uid;
-                                       $arr['contact-id'] = $self[0]['id'];
+                                       $arr['contact-id'] = $self['id'];
                                        $arr['wall'] = 1;
                                        $arr['type'] = 'wall';
                                        $arr['gravity'] = 0;
                                        $arr['origin'] = 1;
-                                       $arr['author-name'] = $arr['owner-name'] = $self[0]['name'];
-                                       $arr['author-link'] = $arr['owner-link'] = $self[0]['url'];
-                                       $arr['author-avatar'] = $arr['owner-avatar'] = $self[0]['thumb'];
-
-                                       $A = '[url=' . $self[0]['url'] . ']' . $self[0]['name'] . '[/url]';
-                                       $APhoto = '[url=' . $self[0]['url'] . ']' . '[img]' . $self[0]['thumb'] . '[/img][/url]';
+                                       $arr['author-name']   = $arr['owner-name']   = $self['name'];
+                                       $arr['author-link']   = $arr['owner-link']   = $self['url'];
+                                       $arr['author-avatar'] = $arr['owner-avatar'] = $self['thumb'];
 
+                                       $A = '[url=' . $self['url'] . ']' . $self['name'] . '[/url]';
                                        $B = '[url=' . $combined['url'] . ']' . $combined['name'] . '[/url]';
                                        $BPhoto = '[url=' . $combined['url'] . ']' . '[img]' . $combined['thumb'] . '[/img][/url]';
 
                                        $arr['verb'] = ACTIVITY_JOIN;
                                        $arr['object-type'] = ACTIVITY_OBJ_GROUP;
-                                       $arr['body'] =  sprintf( t('%1$s has joined %2$s'), $A, $B)."\n\n\n" .$BPhoto;
+                                       $arr['body'] = t('%1$s has joined %2$s', $A, $B) . "\n\n\n" . $BPhoto;
                                        $arr['object'] = '<object><type>' . ACTIVITY_OBJ_GROUP . '</type><title>' . $combined['name'] . '</title>'
                                                . '<id>' . $combined['url'] . '/' . $combined['name'] . '</id>';
                                        $arr['object'] .= '<link>' . xmlify('<link rel="alternate" type="text/html" href="' . $combined['url'] . '" />' . "\n");
@@ -787,27 +698,24 @@ function dfrn_confirm_post(App $a, $handsfree = null) {
 
                                        $arr['last-child'] = 1;
 
-                                       $arr['allow_cid'] = $user[0]['allow_cid'];
-                                       $arr['allow_gid'] = $user[0]['allow_gid'];
-                                       $arr['deny_cid']  = $user[0]['deny_cid'];
-                                       $arr['deny_gid']  = $user[0]['deny_gid'];
+                                       $arr['allow_cid'] = $user['allow_cid'];
+                                       $arr['allow_gid'] = $user['allow_gid'];
+                                       $arr['deny_cid']  = $user['deny_cid'];
+                                       $arr['deny_gid']  = $user['deny_gid'];
 
                                        $i = item_store($arr);
-                                       if($i)
+                                       if ($i) {
                                                Worker::add(PRIORITY_HIGH, "Notifier", "activity", $i);
-
+                                       }
                                }
                        }
                }
                xml_status(0); // Success
                return; // NOTREACHED
-
-                       ////////////////////// End of this scenario ///////////////////////////////////////////////
+               ////////////////////// End of this scenario ///////////////////////////////////////////////
        }
 
        // somebody arrived here by mistake or they are fishing. Send them to the homepage.
-
        goaway(System::baseUrl());
        // NOTREACHED
-
 }