changes during review and testing

Author: thehabes

routes/delete.js

@@ -10,6 +10,7 @@ router.delete('/', checkAccessToken, async (req, res, next) => {
     // check for @id in body.  Any value is valid.  Lack of value is a bad request.
     if (!req?.body || !(req.body['@id'] ?? req.body.id)) {
       res.status(400).send("No record id to delete! (https://store.rerum.io/v1/API.html#delete)")
+      return
     }
     const body = JSON.stringify(req.body)
     const deleteOptions = {

routes/overwrite.js

@@ -1,17 +1,17 @@
 import express from "express"
 import checkAccessToken from "../tokens.js"
 const router = express.Router()
-import rerumPropertiesWasher from "../preprocessor.js"
 
 /* PUT an overwrite to the thing. */
-router.put('/', checkAccessToken, rerumPropertiesWasher, async (req, res, next) => {
+router.put('/', checkAccessToken, async (req, res, next) => {
 
   try {
 
     const overwriteBody = req.body
     // check for @id; any value is valid
     if (!(overwriteBody['@id'] ?? overwriteBody.id)) {
-      throw Error("No record id to overwrite! (https://store.rerum.io/API.html#overwrite)")
+      res.status(400).send("No record id to overwrite! (https://store.rerum.io/API.html#overwrite)")
+      return
     }
 
     const overwriteOptions = {

routes/update.js

@@ -1,15 +1,15 @@
 import express from "express"
 import checkAccessToken from "../tokens.js"
 const router = express.Router()
-import rerumPropertiesWasher from "../preprocessor.js"
 
 /* PUT an update to the thing. */
-router.put('/', checkAccessToken, rerumPropertiesWasher, async (req, res, next) => {
+router.put('/', checkAccessToken, async (req, res, next) => {
 
   try {
     // check for @id in body.  Any value is valid.  Lack of value is a bad request.
     if (!req?.body || !(req.body['@id'] ?? req.body.id)) {
       res.status(400).send("No record id to update! (https://store.rerum.io/v1/API.html#update)")
+      return
     }
     // check body for JSON
     const body = JSON.stringify(req.body)