Commit 2fb7e8e1 authored by Dennis Willers's avatar Dennis Willers 🏀

Merge branch 'fix-setVote' into 'master'

Fix Missing vote if vote is 0

See merge request !2
parents 532a5ce1 6e90d3f0
Pipeline #468 passed with stages
in 2 minutes and 17 seconds
...@@ -10,11 +10,12 @@ const path = require('path'), ...@@ -10,11 +10,12 @@ const path = require('path'),
module.exports = function(app, sql, errorHandler) { module.exports = function(app, sql, errorHandler) {
app.post('/'+scriptName, function(req, res) { app.post('/'+scriptName, function(req, res) {
const mapId = req.query.map || req.body.map, const mapId = req.query.map || req.body.map,
player = req.query.player || req.body.player, player = req.query.player || req.body.player;
vote = Number(req.query.vote || req.body.vote); let vote = req.query.vote || req.body.vote;
if (!mapId) return errorHandler(res, 400, 'Missing map id'); if (!mapId) return errorHandler(res, 400, 'Missing map id');
if (!player) return errorHandler(res, 400, 'Missing player login'); if (!player) return errorHandler(res, 400, 'Missing player login');
if (!vote) return errorHandler(res, 400, 'Missing vote'); if (!vote) return errorHandler(res, 400, 'Missing vote');
vote = Number(vote);
if (isNaN(vote)) return errorHandler(res, 400, 'Invalid vote'); if (isNaN(vote)) return errorHandler(res, 400, 'Invalid vote');
if (vote < 0 || vote > 100) return errorHandler(res, 400, 'Vote must be between 0 and 100'); if (vote < 0 || vote > 100) return errorHandler(res, 400, 'Vote must be between 0 and 100');
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment