From 85f107e8bd1e581ca3711cf09f37cfa7d727d94d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mike=20Schw=C3=B6rer?= <pubgit@mikescher.com>
Date: Fri, 17 Jan 2020 00:55:41 +0100
Subject: [PATCH] Highscores

---
 .idea/inspectionProfiles/Project_Default.xml |   6 +-
 www/internals/highscores.php                 | 102 ------------------
 www/internals/modules.php                    |   6 ++
 www/internals/modules/highscores.php         | 108 +++++++++++++++++++
 www/pages/highscores_insert.php              |  40 +++----
 www/pages/highscores_listentries.php         |  43 ++++----
 www/pages/highscores_listgames.php           |  16 +--
 www/pages/highscores_newid.php               |  22 ++--
 www/pages/highscores_top50.php               |  16 +--
 www/pages/highscores_update.php              |  62 ++++++-----
 10 files changed, 230 insertions(+), 191 deletions(-)
 delete mode 100644 www/internals/highscores.php
 create mode 100644 www/internals/modules/highscores.php

diff --git a/.idea/inspectionProfiles/Project_Default.xml b/.idea/inspectionProfiles/Project_Default.xml
index c66df00..e3c2bd0 100644
--- a/.idea/inspectionProfiles/Project_Default.xml
+++ b/.idea/inspectionProfiles/Project_Default.xml
@@ -1,7 +1,11 @@
 <component name="InspectionProjectProfileManager">
   <profile version="1.0" is_locked="false">
     <option name="myName" value="Project Default" />
-    <option name="myLocal" value="false" />
+    <inspection_tool class="DuplicatedCode" enabled="true" level="WEAK WARNING" enabled_by_default="true">
+      <Languages>
+        <language minSize="100" name="PHP" />
+      </Languages>
+    </inspection_tool>
     <inspection_tool class="SpellCheckingInspection" enabled="false" level="TYPO" enabled_by_default="false">
       <option name="processCode" value="true" />
       <option name="processLiterals" value="true" />
diff --git a/www/internals/highscores.php b/www/internals/highscores.php
deleted file mode 100644
index 999da89..0000000
--- a/www/internals/highscores.php
+++ /dev/null
@@ -1,102 +0,0 @@
-<?php
-
-require_once (__DIR__ . '/../internals/database.php');
-
-class Highscores
-{
-	public static function generateChecksum($rand, $player, $playerid, $points, $gamesalt)
-	{
-		if ($playerid >= 0)
-			return md5($rand . $player . $playerid . $points . $gamesalt);
-		else
-			return md5($rand . $player . $points . $gamesalt);
-	}
-
-	public static function insert($gameid, $points, $name, $playerid, $check, $time, $ip)
-	{
-		return Database::sql_exec_prep('INSERT INTO highscoreentries (GAME_ID, POINTS, PLAYER, PLAYERID, CHECKSUM, TIMESTAMP, IP) VALUES (:gid, :p, :pn, :pid, :cs, :ts, :ip)',
-			[
-				[':gid', $gameid,   PDO::PARAM_INT],
-				[':p',   $points,   PDO::PARAM_INT],
-				[':pn',  $name,     PDO::PARAM_STR],
-				[':pid', $playerid, PDO::PARAM_INT],
-				[':cs',  $check,    PDO::PARAM_STR],
-				[':ts',  $time,     PDO::PARAM_STR],
-				[':ip',  $ip,       PDO::PARAM_STR],
-			]);
-	}
-
-	public static function update($gameid, $points, $name, $playerid, $check, $time, $ip)
-	{
-		return Database::sql_exec_prep('UPDATE highscoreentries SET POINTS = :p, PLAYER = :pn, CHECKSUM = :cs, IP = :ip, TIMESTAMP = :ts WHERE GAME_ID = :gid AND PLAYERID = :pid',
-			[
-				[':gid', $gameid,   PDO::PARAM_INT],
-				[':p',   $points,   PDO::PARAM_INT],
-				[':pn',  $name,     PDO::PARAM_STR],
-				[':pid', $playerid, PDO::PARAM_INT],
-				[':cs',  $check,    PDO::PARAM_STR],
-				[':ts',  $time,     PDO::PARAM_STR],
-				[':ip',  $ip,       PDO::PARAM_STR],
-			]);
-	}
-
-	public static function getGameByID($gameid)
-	{
-		return Database::sql_query_single_prep('SELECT * FROM highscoregames WHERE ID = :id',
-			[
-				[ ':id', $gameid, PDO::PARAM_INT ],
-			]);
-	}
-
-	public static function getOrderedEntriesFromGame($gameid, $limit = null)
-	{
-		$sql = 'SELECT * FROM highscoreentries WHERE GAME_ID = :id ORDER BY POINTS DESC';
-		if ($limit !== null) $sql .= " LIMIT $limit";
-
-		return Database::sql_query_assoc_prep($sql,
-			[
-				[ ':id', $gameid, PDO::PARAM_INT ]
-			]);
-	}
-
-	public static function getNewestEntriesFromGame($gameid, $limit = null)
-	{
-		$sql = 'SELECT * FROM highscoreentries WHERE GAME_ID = :id ORDER BY TIMESTAMP DESC';
-		if ($limit !== null) $sql .= " LIMIT $limit";
-
-		return Database::sql_query_assoc_prep($sql,
-			[
-				[ ':id', $gameid, PDO::PARAM_INT ]
-			]);
-	}
-
-	public static function getEntryCountFromGame($gameid)
-	{
-		return Database::sql_query_num_prep('SELECT COUNT(*) FROM highscoreentries WHERE GAME_ID = :id',
-			[
-				[ ':id', $gameid, PDO::PARAM_INT ]
-			]);
-	}
-
-	public static function getAllGames()
-	{
-		return Database::sql_query_assoc('SELECT * FROM highscoregames');
-	}
-
-	public static function getNextPlayerID($gameid)
-	{
-		return Database::sql_query_num_prep('SELECT MAX(PLAYERID)+1 AS NID FROM highscoreentries WHERE GAME_ID = :gid',
-			[
-				[ ':id', $gameid, PDO::PARAM_INT ]
-			]);
-	}
-
-	public static function getSpecificScore($gameid, $playerid)
-	{
-		return Database::sql_query_single_prep('SELECT * FROM highscoreentries WHERE GAME_ID = :gid AND PLAYERID = :pid',
-			[
-				[ ':gid', $gameid,   PDO::PARAM_INT ],
-				[ ':pid', $playerid, PDO::PARAM_INT ],
-			]);
-	}
-}
\ No newline at end of file
diff --git a/www/internals/modules.php b/www/internals/modules.php
index ac5288c..08821ef 100644
--- a/www/internals/modules.php
+++ b/www/internals/modules.php
@@ -14,6 +14,7 @@ class Modules
 	/** @var UpdatesLog|null */          private $updateslog = null;
 	/** @var WebApps|null */             private $webapps = null;
 	/** @var MikescherGitGraph|null */   private $extendedgitgraph = null;
+	/** @var Highscores|null */          private $highscores = null;
 
 	/** @var Website */
 	private $site;
@@ -83,4 +84,9 @@ class Modules
 		return $this->extendedgitgraph;
 	}
 
+	public function Highscores(): Highscores
+	{
+		if ($this->highscores === null) { require_once 'modules/highscores.php'; $this->highscores = new Highscores($this->site); }
+		return $this->highscores;
+	}
 }
\ No newline at end of file
diff --git a/www/internals/modules/highscores.php b/www/internals/modules/highscores.php
new file mode 100644
index 0000000..88a2a42
--- /dev/null
+++ b/www/internals/modules/highscores.php
@@ -0,0 +1,108 @@
+<?php
+
+class Highscores
+{
+	/** @var Website */
+	private $site;
+
+	public function __construct(Website $site)
+	{
+		$this->site = $site;
+	}
+
+	public function generateChecksum($rand, $player, $playerid, $points, $gamesalt)
+	{
+		if ($playerid >= 0)
+			return md5($rand . $player . $playerid . $points . $gamesalt);
+		else
+			return md5($rand . $player . $points . $gamesalt);
+	}
+
+	public function insert($gameid, $points, $name, $playerid, $check, $time, $ip)
+	{
+		return $this->site->modules->Database()->sql_exec_prep('INSERT INTO highscoreentries (GAME_ID, POINTS, PLAYER, PLAYERID, CHECKSUM, TIMESTAMP, IP) VALUES (:gid, :p, :pn, :pid, :cs, :ts, :ip)',
+			[
+				[':gid', $gameid,   PDO::PARAM_INT],
+				[':p',   $points,   PDO::PARAM_INT],
+				[':pn',  $name,     PDO::PARAM_STR],
+				[':pid', $playerid, PDO::PARAM_INT],
+				[':cs',  $check,    PDO::PARAM_STR],
+				[':ts',  $time,     PDO::PARAM_STR],
+				[':ip',  $ip,       PDO::PARAM_STR],
+			]);
+	}
+
+	public function update($gameid, $points, $name, $playerid, $check, $time, $ip)
+	{
+		return $this->site->modules->Database()->sql_exec_prep('UPDATE highscoreentries SET POINTS = :p, PLAYER = :pn, CHECKSUM = :cs, IP = :ip, TIMESTAMP = :ts WHERE GAME_ID = :gid AND PLAYERID = :pid',
+			[
+				[':gid', $gameid,   PDO::PARAM_INT],
+				[':p',   $points,   PDO::PARAM_INT],
+				[':pn',  $name,     PDO::PARAM_STR],
+				[':pid', $playerid, PDO::PARAM_INT],
+				[':cs',  $check,    PDO::PARAM_STR],
+				[':ts',  $time,     PDO::PARAM_STR],
+				[':ip',  $ip,       PDO::PARAM_STR],
+			]);
+	}
+
+	public function getGameByID($gameid)
+	{
+		return $this->site->modules->Database()->sql_query_single_prep('SELECT * FROM highscoregames WHERE ID = :id',
+			[
+				[ ':id', $gameid, PDO::PARAM_INT ],
+			]);
+	}
+
+	public function getOrderedEntriesFromGame($gameid, $limit = null)
+	{
+		$sql = 'SELECT * FROM highscoreentries WHERE GAME_ID = :id ORDER BY POINTS DESC';
+		if ($limit !== null) $sql .= " LIMIT $limit";
+
+		return $this->site->modules->Database()->sql_query_assoc_prep($sql,
+			[
+				[ ':id', $gameid, PDO::PARAM_INT ]
+			]);
+	}
+
+	public function getNewestEntriesFromGame($gameid, $limit = null)
+	{
+		$sql = 'SELECT * FROM highscoreentries WHERE GAME_ID = :id ORDER BY TIMESTAMP DESC';
+		if ($limit !== null) $sql .= " LIMIT $limit";
+
+		return $this->site->modules->Database()->sql_query_assoc_prep($sql,
+			[
+				[ ':id', $gameid, PDO::PARAM_INT ]
+			]);
+	}
+
+	public function getEntryCountFromGame($gameid)
+	{
+		return $this->site->modules->Database()->sql_query_num_prep('SELECT COUNT(*) FROM highscoreentries WHERE GAME_ID = :id',
+			[
+				[ ':id', $gameid, PDO::PARAM_INT ]
+			]);
+	}
+
+	public function getAllGames()
+	{
+		return $this->site->modules->Database()->sql_query_assoc('SELECT * FROM highscoregames');
+	}
+
+	public function getNextPlayerID($gameid)
+	{
+		return $this->site->modules->Database()->sql_query_num_prep('SELECT MAX(PLAYERID)+1 AS NID FROM highscoreentries WHERE GAME_ID = :gid',
+			[
+				[ ':id', $gameid, PDO::PARAM_INT ]
+			]);
+	}
+
+	public function getSpecificScore($gameid, $playerid)
+	{
+		return $this->site->modules->Database()->sql_query_single_prep('SELECT * FROM highscoreentries WHERE GAME_ID = :gid AND PLAYERID = :pid',
+			[
+				[ ':gid', $gameid,   PDO::PARAM_INT ],
+				[ ':pid', $playerid, PDO::PARAM_INT ],
+			]);
+	}
+}
\ No newline at end of file
diff --git a/www/pages/highscores_insert.php b/www/pages/highscores_insert.php
index a0eb4ad..66a785d 100644
--- a/www/pages/highscores_insert.php
+++ b/www/pages/highscores_insert.php
@@ -1,26 +1,30 @@
 <?php
-	global $OPTIONS;
+require_once (__DIR__ . '/../internals/website.php');
 
-	require_once (__DIR__ . '/../internals/base.php');
-	require_once (__DIR__ . '/../internals/database.php');
-	require_once (__DIR__ . '/../internals/highscores.php');
+/** @var PageFrameOptions $FRAME_OPTIONS */ global $FRAME_OPTIONS;
+/** @var URLRoute $ROUTE */ global $ROUTE;
+/** @var Website $SITE */ global $SITE;
 
-	Database::connect();
+$FRAME_OPTIONS->title = null;
+$FRAME_OPTIONS->canonical_url = null;
+$FRAME_OPTIONS->activeHeader = null;
+$FRAME_OPTIONS->frame = 'api_frame.php';
 
-	$gameid = $OPTIONS['gameid'];
-	$check  = $OPTIONS['check'];
-	$name   = $OPTIONS['name'];
-	$rand   = $OPTIONS['rand'];
-	$points = $OPTIONS['points'];
 
-	if (! is_numeric($gameid)) httpError(400, 'Invalid Request');
-	if (! is_numeric($points)) httpError(400, 'Invalid Request');
+$gameid = $ROUTE->parameter['gameid'];
+$check  = $ROUTE->parameter['check'];
+$name   = $ROUTE->parameter['name'];
+$rand   = $ROUTE->parameter['rand'];
+$points = $ROUTE->parameter['points'];
 
-	$game = Highscores::getGameByID($gameid);
-	if ($game == NULL) httpError(400, 'Invalid Request');
+if (! is_numeric($gameid)) { $FRAME_OPTIONS->forceResult(400, 'Invalid Request'); return; }
+if (! is_numeric($points)) { $FRAME_OPTIONS->forceResult(400, 'Invalid Request'); return; }
 
-	$checksum_generated = Highscores::generateChecksum($rand, $name, -1, $points, $game['SALT']);
-	if ($checksum_generated != $check) die('Nice try !');
+$game = $SITE->modules->Highscores()->getGameByID($gameid);
+if ($game == NULL) { $FRAME_OPTIONS->forceResult(400, 'Invalid Request'); return; }
 
-	Highscores::insert($gameid, $points, $name, -1, $check, date("Y-m-d H:m:s", time()), $_SERVER['REMOTE_ADDR']);
-	echo 'ok.';
\ No newline at end of file
+$checksum_generated = $SITE->modules->Highscores()->generateChecksum($rand, $name, -1, $points, $game['SALT']);
+if ($checksum_generated != $check) die('Nice try !');
+
+$SITE->modules->Highscores()->insert($gameid, $points, $name, -1, $check, date("Y-m-d H:m:s", time()), $_SERVER['REMOTE_ADDR']);
+echo 'ok.';
\ No newline at end of file
diff --git a/www/pages/highscores_listentries.php b/www/pages/highscores_listentries.php
index 51adbed..01e0f25 100644
--- a/www/pages/highscores_listentries.php
+++ b/www/pages/highscores_listentries.php
@@ -1,30 +1,35 @@
 <?php
-	global $OPTIONS;
+require_once (__DIR__ . '/../internals/website.php');
 
-	require_once (__DIR__ . '/../internals/base.php');
-	require_once (__DIR__ . '/../internals/database.php');
-	require_once (__DIR__ . '/../internals/highscores.php');
+/** @var PageFrameOptions $FRAME_OPTIONS */ global $FRAME_OPTIONS;
+/** @var URLRoute $ROUTE */ global $ROUTE;
+/** @var Website $SITE */ global $SITE;
 
-	Database::connect();
+$FRAME_OPTIONS->title = null;
+$FRAME_OPTIONS->canonical_url = null;
+$FRAME_OPTIONS->activeHeader = null;
+$FRAME_OPTIONS->frame = 'api_frame.php';
 
-	$pagesize = 20;
-	$start = 0;
-	$highlight = 0;
 
-	if (isset($_GET["start"]))
-	{
-		$start = intval(htmlspecialchars($_GET["start"])) - 1;
-		if ($start < 0) $start = 0;
-	}
 
-	if (isset($_GET["highlight"]))
-	{
-		$highlight= intval(htmlspecialchars($_GET["highlight"]));
-	}
+$pagesize = 20;
+$start = 0;
+$highlight = 0;
 
-    $game = Highscores::getGameByID($OPTIONS['gameid']);
+if (isset($_GET["start"]))
+{
+    $start = intval(htmlspecialchars($_GET["start"])) - 1;
+    if ($start < 0) $start = 0;
+}
 
-    $entries = Highscores::getOrderedEntriesFromGame($OPTIONS['gameid']);
+if (isset($_GET["highlight"]))
+{
+    $highlight= intval(htmlspecialchars($_GET["highlight"]));
+}
+
+$game = $SITE->modules->Highscores()->getGameByID($ROUTE->parameter['gameid']);
+
+$entries = $SITE->modules->Highscores()->getOrderedEntriesFromGame($ROUTE->parameter['gameid']);
 ?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
 	"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
diff --git a/www/pages/highscores_listgames.php b/www/pages/highscores_listgames.php
index 959b38a..b5b2f59 100644
--- a/www/pages/highscores_listgames.php
+++ b/www/pages/highscores_listgames.php
@@ -1,14 +1,16 @@
 <?php
-	global $OPTIONS;
-	
-	require_once (__DIR__ . '/../internals/base.php');
-	require_once (__DIR__ . '/../internals/database.php');
-	require_once (__DIR__ . '/../internals/highscores.php');
+require_once (__DIR__ . '/../internals/website.php');
 
-	Database::connect();
+/** @var PageFrameOptions $FRAME_OPTIONS */ global $FRAME_OPTIONS;
+/** @var URLRoute $ROUTE */ global $ROUTE;
+/** @var Website $SITE */ global $SITE;
 
-	$games = Highscores::getAllGames();
+$FRAME_OPTIONS->title = null;
+$FRAME_OPTIONS->canonical_url = null;
+$FRAME_OPTIONS->activeHeader = null;
+$FRAME_OPTIONS->frame = 'api_frame.php';
 
+$games = $SITE->modules->Highscores()->getAllGames();
 ?>
 <html>
 <head>
diff --git a/www/pages/highscores_newid.php b/www/pages/highscores_newid.php
index 4e903c4..e0a4c64 100644
--- a/www/pages/highscores_newid.php
+++ b/www/pages/highscores_newid.php
@@ -1,14 +1,18 @@
 <?php
-	global $OPTIONS;
+require_once (__DIR__ . '/../internals/website.php');
 
-	require_once (__DIR__ . '/../internals/base.php');
-	require_once (__DIR__ . '/../internals/database.php');
-	require_once (__DIR__ . '/../internals/highscores.php');
+/** @var PageFrameOptions $FRAME_OPTIONS */ global $FRAME_OPTIONS;
+/** @var URLRoute $ROUTE */ global $ROUTE;
+/** @var Website $SITE */ global $SITE;
 
-	Database::connect();
+$FRAME_OPTIONS->title = null;
+$FRAME_OPTIONS->canonical_url = null;
+$FRAME_OPTIONS->activeHeader = null;
+$FRAME_OPTIONS->frame = 'api_frame.php';
 
-	$newid = Highscores::getNextPlayerID($OPTIONS['gameid']);
-	
-	if ($newid < 1024) $newid = 1024;
 
-	print $newid;
\ No newline at end of file
+$newid = $SITE->modules->Highscores()->getNextPlayerID($ROUTE->parameter['gameid']);
+
+if ($newid < 1024) $newid = 1024;
+
+print $newid;
\ No newline at end of file
diff --git a/www/pages/highscores_top50.php b/www/pages/highscores_top50.php
index ec2aa6c..6510352 100644
--- a/www/pages/highscores_top50.php
+++ b/www/pages/highscores_top50.php
@@ -1,13 +1,17 @@
 <?php
-	global $OPTIONS;
+require_once (__DIR__ . '/../internals/website.php');
 
-	require_once (__DIR__ . '/../internals/base.php');
-	require_once (__DIR__ . '/../internals/database.php');
-	require_once (__DIR__ . '/../internals/highscores.php');
+/** @var PageFrameOptions $FRAME_OPTIONS */ global $FRAME_OPTIONS;
+/** @var URLRoute $ROUTE */ global $ROUTE;
+/** @var Website $SITE */ global $SITE;
 
-	Database::connect();
+$FRAME_OPTIONS->title = null;
+$FRAME_OPTIONS->canonical_url = null;
+$FRAME_OPTIONS->activeHeader = null;
+$FRAME_OPTIONS->frame = 'api_frame.php';
 
-	$entries = Highscores::getOrderedEntriesFromGame($OPTIONS['gameid'], 50);
+
+$entries = $SITE->modules->Highscores()->getOrderedEntriesFromGame($ROUTE->parameter['gameid'], 50);
 
 for ($i = 0; $i < count($entries); $i++)
 {
diff --git a/www/pages/highscores_update.php b/www/pages/highscores_update.php
index 62b4275..fd5121e 100644
--- a/www/pages/highscores_update.php
+++ b/www/pages/highscores_update.php
@@ -1,38 +1,42 @@
 <?php
-	global $OPTIONS;
+require_once (__DIR__ . '/../internals/website.php');
 
-	require_once (__DIR__ . '/../internals/base.php');
-	require_once (__DIR__ . '/../internals/database.php');
-	require_once (__DIR__ . '/../internals/highscores.php');
+/** @var PageFrameOptions $FRAME_OPTIONS */ global $FRAME_OPTIONS;
+/** @var URLRoute $ROUTE */ global $ROUTE;
+/** @var Website $SITE */ global $SITE;
 
-	Database::connect();
+$FRAME_OPTIONS->title = null;
+$FRAME_OPTIONS->canonical_url = null;
+$FRAME_OPTIONS->activeHeader = null;
+$FRAME_OPTIONS->frame = 'api_frame.php';
 
-	$gameid = $OPTIONS['gameid'];
-	$check  = $OPTIONS['check'];
-	$name   = $OPTIONS['name'];
-	$nameid = $OPTIONS['nameid'];
-	$rand   = $OPTIONS['rand'];
-	$points = $OPTIONS['points'];
 
-	if (! is_numeric($gameid)) httpError(400, 'Invalid Request');
-	if (! is_numeric($nameid)) httpError(400, 'Invalid Request');
-	if (! is_numeric($points)) httpError(400, 'Invalid Request');
+$gameid = $ROUTE->parameter['gameid'];
+$check  = $ROUTE->parameter['check'];
+$name   = $ROUTE->parameter['name'];
+$nameid = $ROUTE->parameter['nameid'];
+$rand   = $ROUTE->parameter['rand'];
+$points = $ROUTE->parameter['points'];
 
-	$game = Highscores::getGameByID($OPTIONS['gameid']);
-	if ($game == NULL) httpError(400, 'Invalid Request');
+if (! is_numeric($gameid)) { $FRAME_OPTIONS->forceResult(400, 'Invalid Request'); return; }
+if (! is_numeric($nameid)) { $FRAME_OPTIONS->forceResult(400, 'Invalid Request'); return; }
+if (! is_numeric($points)) { $FRAME_OPTIONS->forceResult(400, 'Invalid Request'); return; }
 
-	$checksum_generated = Highscores::generateChecksum($rand, $name, $nameid, $points, $game['SALT']);
-	if ($checksum_generated != $check) die('Nice try !');
+$game = $SITE->modules->Highscores()->getGameByID($ROUTE->parameter['gameid']);
+if ($game == NULL) { $FRAME_OPTIONS->forceResult(400, 'Invalid Request'); return; }
 
-	$old = Highscores::getSpecificScore($gameid, $nameid);
+$checksum_generated = $SITE->modules->Highscores()->generateChecksum($rand, $name, $nameid, $points, $game['SALT']);
+if ($checksum_generated != $check) die('Nice try !');
 
-	if ($old == null)
-	{
-		Highscores::insert($gameid, $points, $name, $nameid, $check, date("Y-m-d H:m:s", time()), $_SERVER['REMOTE_ADDR']);
-		echo 'ok.';
-	}
-	else 
-	{
-		Highscores::update($gameid, $points, $name, $nameid, $check, date("Y-m-d H:m:s", time()), $_SERVER['REMOTE_ADDR']);
-		echo 'ok.';
-	}
+$old = $SITE->modules->Highscores()->getSpecificScore($gameid, $nameid);
+
+if ($old == null)
+{
+	$SITE->modules->Highscores()->insert($gameid, $points, $name, $nameid, $check, date("Y-m-d H:m:s", time()), $_SERVER['REMOTE_ADDR']);
+	echo 'ok.';
+}
+else
+{
+	$SITE->modules->Highscores()->update($gameid, $points, $name, $nameid, $check, date("Y-m-d H:m:s", time()), $_SERVER['REMOTE_ADDR']);
+	echo 'ok.';
+}