X-Git-Url: https://git.auder.net/?a=blobdiff_plain;f=server%2Fmodels%2FChallenge.js;h=6d8f5c1e865d4f1d7d14f7d5af88ac8a4d41a28a;hb=052d17ea6e199533cefb11f1ef51020b55cb1382;hp=96db0a2b431ad5237d8b08b4bcfceca3c4155212;hpb=625022fdcf750f0aff8fcd699f7e9b89730e1d10;p=vchess.git

diff --git a/server/models/Challenge.js b/server/models/Challenge.js
index 96db0a2b..6d8f5c1e 100644
--- a/server/models/Challenge.js
+++ b/server/models/Challenge.js
@@ -8,8 +8,7 @@ var db = require("../utils/database");
  *   vid: variant id (int)
  *   nbPlayers: integer
  *   fen: varchar (optional)
- *   mainTime: integer
- *   addTime: integer
+ *   timeControl: string (3m+2s, 7d+1d ...)
  *
  * Structure table WillPlay:
  *   cid: ref challenge id
@@ -18,22 +17,41 @@ var db = require("../utils/database");
 
 const ChallengeModel =
 {
-	// fen cannot be undefined; TODO: generate fen on server instead
+	checkChallenge: function(c)
+	{
+    if (!c.vid.match(/^[0-9]+$/))
+			return "Wrong variant ID";
+
+    if (!c.timeControl.match(/^[0-9dhms +]+$/))
+      return "Wrong characters in time control";
+
+    if (!c.nbPlayers.match(/^[0-9]+$/))
+			return "Wrong number of players";
+
+		if (!c.fen.match(/^[a-zA-Z0-9, /-]+$/))
+			return "Bad FEN string";
+	},
+
+	// fen cannot be undefined
 	create: function(c, cb)
 	{
 		db.serialize(function() {
 			let query =
 				"INSERT INTO Challenges " +
-				"(added, uid, vid, nbPlayers, fen, mainTime, addTime) VALUES " +
+				"(added, uid, vid, nbPlayers, fen, timeControl) VALUES " +
 				"(" + Date.now() + "," + c.uid + "," + c.vid + "," + c.nbPlayers +
-					",'" + c.fen + "'," + c.mainTime + "," + c.increment + ")";
+					",'" + c.fen + "'," + c.timeControl + ")";
 			db.run(query, err => {
 				if (!!err)
 					return cb(err);
 				db.get("SELECT last_insert_rowid() AS rowid", (err2,lastId) => {
-					query =
+					
+          // TODO: also insert "will play" "no" for other players ?
+          // willplay = "maybe" by default ?
+          
+          query =
 						"INSERT INTO WillPlay VALUES " +
-						"(" + lastId["rowid"] + "," + c.uid + ")";
+						"(true," + lastId["rowid"] + "," + c.uid + ")";
 						db.run(query, (err,ret) => {
 							cb(err, lastId); //all we need is the challenge ID
 						});
@@ -71,8 +89,7 @@ const ChallengeModel =
 						nbPlayers: challengeInfo.nbPlayers,
 						players: players, //currently in
 						fen: challengeInfo.fen,
-						mainTime: challengeInfo.mainTime,
-						increment: challengeInfo.addTime,
+						timeControl: challengeInfo.timeControl,
 					};
 					return cb(null, challenge);
 				});