Fix ajax for challenge creation

diff --git a/client/src/utils/ajax.js b/client/src/utils/ajax.js
index 01ea849..5b46078 100644 (file)
--- a/client/src/utils/ajax.js
+++ b/client/src/utils/ajax.js
@@ -17,15 +17,16 @@ function toQueryString(data)
 export function ajax(url, method, data, success, error)
 {
        let xhr = new XMLHttpRequest();
-       if (typeof(data) === "function") //no data
+       if (data === undefined || typeof(data) === "function") //no data
        {
                error = success;
                success = data;
                data = {};
        }
+  if (!success)
+    success = () => {}; //by default, do nothing
        if (!error)
                error = errmsg => { alert(errmsg); };
-
        xhr.onreadystatechange = function() {
                if (this.readyState == 4 && this.status == 200)
                {

diff --git a/server/models/Challenge.js b/server/models/Challenge.js
index 82b252f..c988aba 100644 (file)
--- a/server/models/Challenge.js
+++ b/server/models/Challenge.js
@@ -35,7 +35,7 @@ const ChallengeModel =
         "(" + Date.now() + "," + c.uid + "," + (!!c.to ? c.to + "," : "") +
           c.vid + ",'" + c.fen + "','" + c.timeControl + "')";
       db.run(query, err => {
-        return cb(err);
+        return cb(err, {cid: this.lastID});
       });
     });
   },

diff --git a/server/routes/challenges.js b/server/routes/challenges.js
index c2e55c8..acec3c1 100644 (file)
--- a/server/routes/challenges.js
+++ b/server/routes/challenges.js
@@ -24,9 +24,8 @@ router.post("/challenges", access.logged, access.ajax, (req,res) => {
     to: req.body.chall.to, //string: user name (may be empty)
   };
   const insertChallenge = () => {
-    ChallengeModel.create(challenge, (err) => {
-      if (!!err)
-        return res.json(err);
+    ChallengeModel.create(challenge, (err,ret) => {
+      return res.json(err || {cid:ret.cid});
     });
   };
   if (!!req.body.chall.to)