Invert patch override check in POST routes

Simplify POST handlers for patch routes by inverting the rest.checkPatchOverrideSupport condition and moving the controller invocation out of the conditional in routes/patchSet.js, routes/patchUnset.js, and routes/patchUpdate.js. Behavior is preserved: if patch-override is not supported respond with 405, otherwise call the corresponding controller. This reduces nesting and improves readability.

Author: cubap

routes/patchSet.js

@@ -8,15 +8,13 @@ import rest from '../rest.js'
 router.route('/')
     .patch(auth.checkJwt, rest.verifyJsonContentType, controller.patchSet)
     .post(auth.checkJwt, rest.verifyJsonContentType, (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,15 +8,13 @@ import rest from '../rest.js'
 router.route('/')
     .patch(auth.checkJwt, rest.verifyJsonContentType, controller.patchUnset)
     .post(auth.checkJwt, rest.verifyJsonContentType, (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,15 +9,13 @@ import auth from '../auth/index.js'
 router.route('/')
     .patch(auth.checkJwt, rest.verifyJsonContentType, controller.patchUpdate) 
     .post(auth.checkJwt, rest.verifyJsonContentType, (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.'