fixed comment and explained one

Signed-off-by: Roland Haeder <roland@mxchange.org>

diff --git a/include/dfrn.php b/include/dfrn.php
index 34cd33c339bf58b55a9766349a9a8b9b4ed576b9..9413b4d3af2e567fab962f460a531230c721ec2b 100644 (file)
--- a/include/dfrn.php
+++ b/include/dfrn.php
@@ -1637,7 +1637,13 @@ class dfrn {
                                intval($fid)
                        );
 
-                       /// @TODO Really abort on valid result??? Maybe missed ! here?
+                       /*
+                        * The valid result means the friend we're about to send a friend
+                        * suggestion already has them in their contact, which means no further
+                        * action is required.
+                        *
+                        * @see https://github.com/friendica/friendica/pull/3254#discussion_r107315246
+                        */
                        if (dbm::is_result($r)) {
                                return false;
                        }
@@ -1656,7 +1662,10 @@ class dfrn {
                        dbesc($suggest["request"])
                );
 
-               // This way a useless else-block is being spared, resulting in easier code.
+               /*
+                * If no record in fcontact is found, below INSERT statement will not
+                * link an introduction to it.
+                */
                if (!dbm::is_result($r)) {
                        // database record did not get created. Quietly give up.
                        killme();