Avoid a contact lookup for empty URL

diff --git a/src/Module/Magic.php b/src/Module/Magic.php
index 4510d8d934f63d6f351e091d263e8bfec772b631..7cbc4794b32bf841d07e3fc72526b21b0b37ce89 100644 (file)
--- a/src/Module/Magic.php
+++ b/src/Module/Magic.php
@@ -32,15 +32,16 @@ class Magic extends BaseModule
                $dest = defaults($_REQUEST, 'dest', '');
                $test = (!empty($_REQUEST['test']) ? intval($_REQUEST['test']) : 0);
                $owa  = (!empty($_REQUEST['owa'])  ? intval($_REQUEST['owa'])  : 0);
+               $cid  = 0;
 
                if (!empty($addr)) {
                        $cid = Contact::getIdForURL($addr);
-               } else {
+               } elseif (!empty($dest)) {
                        $cid = Contact::getIdForURL($dest);
                }
 
                if (!$cid) {
-                       Logger::log('No contact record found: ' . json_encode($_REQUEST), Logger::DEBUG);
+                       Logger::info('No contact record found', $_REQUEST);
                        // @TODO Finding a more elegant possibility to redirect to either internal or external URL
                        $a->redirect($dest);
                }