Refactor route files to use guard clauses instead of if/else

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>

Author: Copilot

routes/patchSet.js

@@ -8,14 +8,13 @@ import rest from '../rest.js'
 router.route('/')
     .patch(auth.checkJwt, controller.patchSet)
     .post(auth.checkJwt, (req, res, next) => {
-        if (rest.checkPatchOverrideSupport(req, res)) {
-            controller.patchSet(req, res, next)
-        }
-        else {
+        if (!rest.checkPatchOverrideSupport(req, res)) {
             res.statusMessage = 'Improper request method for updating, please use PATCH to add new keys to this object.'
             res.status(405)
             next(res)
+            return
         }
+        controller.patchSet(req, res, next)
     }) 
     .all((req, res, next) => {
         res.statusMessage = 'Improper request method for updating, please use PATCH to add new keys to this object.'

routes/patchUnset.js

@@ -8,14 +8,13 @@ import rest from '../rest.js'
 router.route('/')
     .patch(auth.checkJwt, controller.patchUnset)
     .post(auth.checkJwt, (req, res, next) => {
-        if (rest.checkPatchOverrideSupport(req, res)) {
-            controller.patchUnset(req, res, next)
-        }
-        else {
+        if (!rest.checkPatchOverrideSupport(req, res)) {
             res.statusMessage = 'Improper request method for updating, please use PATCH to remove keys from this object.'
             res.status(405)
             next(res)
+            return
         }
+        controller.patchUnset(req, res, next)
     }) 
     .all((req, res, next) => {
         res.statusMessage = 'Improper request method for updating, please use PATCH to remove keys from this object.'

routes/patchUpdate.js

@@ -9,14 +9,13 @@ import auth from '../auth/index.js'
 router.route('/')
     .patch(auth.checkJwt, controller.patchUpdate) 
     .post(auth.checkJwt, (req, res, next) => {
-        if (rest.checkPatchOverrideSupport(req, res)) {
-            controller.patchUpdate(req, res, next)
-        }
-        else {
+        if (!rest.checkPatchOverrideSupport(req, res)) {
             res.statusMessage = 'Improper request method for updating, please use PATCH to alter the existing keys this object.'
             res.status(405)
             next(res)
+            return
         }
+        controller.patchUpdate(req, res, next)
     }) 
     .all((req, res, next) => {
         res.statusMessage = 'Improper request method for updating, please use PATCH to alter existing keys on this object.'