// routes/characters/index.js
router.post("/new", bodyParser.json(), (req, res) => {
    // Check if the body is formed correctly
    if (
        !req.body || 
        !req.body.data || 
        // I check for longname because it's the only field
        // where I didn't define an alternative assignment.
        !req.body.data.longName
    ) {
        res.statusCode = 400,
        res.json({error: "Bad Request"})
        return false
    }
    let data = req.body.data
    let abilityScoreDefault = 8
    req.context.db.models.Character.create({
        shortName: data.shortName || data.longName,
        longName: data.longName,
        // I changed some of these fields to be nullable for testing purposes
        // race and class will remain nullable while player won't be.
        // I'll have to set up authentication to grab the right user for the player field.
        race: data.raceId || null,
        class: data.classId || null,
        player: data.playerId || null,
        description: data.description || "",
        age: data.age || "",
        urlSlug: data.urlSlug || "",
        stats: {
            abilities: {
                // I should ensure that the 'abilities' key exists.
                strength: data.stats.abilities.strength || abilityScoreDefault,
                dexterity: data.stats.abilities.dexterity || abilityScoreDefault,
                constitution: data.stats.abilities.constitution || abilityScoreDefault,
                intelligence: data.stats.abilities.intelligence || abilityScoreDefault,
                wisdom: data.stats.abilities.wisdom || abilityScoreDefault,
                charisma: data.stats.abilities.charisma || abilityScoreDefault
            }
        }
    })
    .then(result => res.json(result))
});