More usage of dbm::is_result($r) instead of count($r):

[friendica.git] / include / redir.php

diff --git a/include/redir.php b/include/redir.php
index ba7d02993f86526a9cd33cf180f719aca3025209..8d8a035f1db5fa83c07e388111526c5855416b29 100644 (file)
--- a/include/redir.php
+++ b/include/redir.php
@@ -4,7 +4,7 @@ function auto_redir(&$a, $contact_nick) {
 
        // prevent looping
 
-       if(intval($_REQUEST,'redir'))
+       if(x($_REQUEST,'redir') && intval($_REQUEST['redir']))
                return;
 
        if((! $contact_nick) || ($contact_nick === $a->user['nickname']))
@@ -36,7 +36,7 @@ function auto_redir(&$a, $contact_nick) {
                dbesc($nurl)
                );
 
-               if((!$r) || (! count($r)) || $r[0]['id'] == remote_user())
+               if((! dbm::is_result($r)) || $r[0]['id'] == remote_user())
                        return;
 
 
@@ -48,7 +48,7 @@ function auto_redir(&$a, $contact_nick) {
                       dbesc($baseurl)
                );
 
-               if(! ($r && count($r)))
+               if(! dbm::is_result($r))
                        return;
 
                $cid = $r[0]['id'];
@@ -64,6 +64,12 @@ function auto_redir(&$a, $contact_nick) {
                        $dfrn_id = '0:' . $orig_id;
                }
 
+               // ensure that we've got a valid ID. There may be some edge cases with forums and non-duplex mode
+               // that may have triggered some of the "went to {profile/intro} and got an RSS feed" issues
+
+               if(strlen($dfrn_id) < 3)
+                       return;
+                       
                $sec = random_string();
 
                q("INSERT INTO `profile_check` ( `uid`, `cid`, `dfrn_id`, `sec`, `expire`)