From f2f613b35b4052a05d64628082c08a4ce780fcee Mon Sep 17 00:00:00 2001 From: Joey Baker Date: Sat, 3 Sep 2016 18:48:03 -0700 Subject: [PATCH] Fix: boom error on invalid `include` Sends a 501 `notImplemented` error when `parseInclude` can't find models to include. --- src/utils.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/utils.js b/src/utils.js index ac752e5..880cf8b 100644 --- a/src/utils.js +++ b/src/utils.js @@ -1,4 +1,5 @@ import { omit, identity } from 'lodash'; +import { notImplemented } from 'boom'; export const parseInclude = request => { const include = Array.isArray(request.query.include) ? request.query.include @@ -8,7 +9,7 @@ export const parseInclude = request => { const noRequestModels = !request.models; if (noGetDb && noRequestModels) { - return new Error('`request.getDb` or `request.models` are not defined.' + return notImplemented('`request.getDb` or `request.models` are not defined.' + 'Be sure to load hapi-sequelize before hapi-sequelize-crud.'); }