CLEANUP: better debugging option for DB querries

[e-DoKo.git] / include / reminder.php

diff --git a/include/reminder.php b/include/reminder.php
index 8844e4801679ffa4f67b3627b6cf1e89058836d2..61bd37ba1322d9104213353ae23c0080a79e59e8 100644 (file)
--- a/include/reminder.php
+++ b/include/reminder.php
@@ -5,14 +5,10 @@
 if(!isset($HOST))
   exit;
 
-output_status();
-
 if(!myisset("me"))
   {
     echo "Hmm, you really shouldn't mess with the urls.<br />\n";
-    output_footer();
-    DB_close();
-    exit();
+    return;
   }
 
 $me = $_REQUEST["me"];
@@ -23,9 +19,7 @@ if(!$myid)
   {
     echo "Can't find you in the database, please check the url.<br />\n";
     echo "perhaps the game has been canceled, check by login in <a href=\"$INDEX\">here</a>.";
-    output_footer();
-    DB_close();
-    exit();
+    return;
   }
 
 DB_update_user_timestamp($myid);
@@ -39,7 +33,6 @@ $r = DB_query_array("SELECT mod_date,player,status from Game WHERE id='$gameid'
 if( (time()-strtotime($r[0]) > 60*60*24*7)  && ($r[2]!='gameover') ) /* = 1 week */
   {
     $name = DB_get_name('userid',$r[1]);
-    $To   = DB_get_email('userid',$r[1]);
     $userhash = DB_get_hash_from_gameid_and_userid($gameid,$r[1]);
 
     $message = "Hello $name, \n\n".
@@ -56,12 +49,13 @@ if( (time()-strtotime($r[0]) > 60*60*24*7)  && ($r[2]!='gameover') ) /* = 1 week
     else
       {
        DB_set_reminder($r[1],$gameid);
-       mymail($To,$EmailName."Reminder: game ".DB_format_gameid($gameid)." it's your turn",$message);
+       $subject ='Reminder: game '.DB_format_gameid($gameid)." it's your turn";
+       mymail($r[1],$subject,$message);
 
        echo "<p style=\"background-color:red\";>Game ".DB_format_gameid($gameid).
          ": an email has been sent out.<br /><br /></p>";
       }
   }
  else
-   echo "<p>You need to wait longer before you can send out a reminder...</p>\n";
+   echo '<p>You need to wait longer before you can send out a reminder...</p>\n';
 ?>
\ No newline at end of file