LICENSE

@@ -1,21 +0,0 @@
-The MIT License (MIT)
-
-Copyright (c) 2021 The hapi-sequelize-crud Authors
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "hapi-sequelize-crud",
-  "version": "2.9.3",
+  "version": "2.9.0",
   "description": "Hapi plugin that automatically generates RESTful API for CRUD",
   "main": "build/index.js",
   "config": {

src/crud-list-limit-and-offset.integration.test.js

@@ -1,94 +0,0 @@
-import test from 'ava';
-import 'sinon-bluebird';
-import setup from '../test/integration-setup.js';
-
-const STATUS_OK = 200;
-const STATUS_NOT_FOUND = 404;
-
-setup(test);
-
-test('/players?limit=2', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players?limit=${limit}`;
-  const method = 'GET';
-
-  const { result, statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_OK);
-  t.is(result.length, limit);
-});
-
-test('/players?limit=2&offset=1', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players?limit=${limit}&offset=1`;
-  const method = 'GET';
-
-  const { result, statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_OK);
-  t.is(result.length, limit);
-});
-
-test('/players?limit=2&offset=2', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players?limit=${limit}&offset=2`;
-  const method = 'GET';
-
-  const { result, statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_OK);
-  t.is(result.length, 1, 'with only 3 players, only get 1 back with an offset of 2');
-});
-
-test('/players?limit=2&offset=20', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players?limit=${limit}&offset=20`;
-  const method = 'GET';
-
-  const { statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_NOT_FOUND, 'with a offset/limit greater than the data, returns a 404');
-});
-
-test('scope /players/returnsAll?limit=2', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players/returnsAll?limit=${limit}`;
-  const method = 'GET';
-
-  const { result, statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_OK);
-  t.is(result.length, limit);
-});
-
-test('scope /players/returnsAll?limit=2&offset=1', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players/returnsAll?limit=${limit}&offset=1`;
-  const method = 'GET';
-
-  const { result, statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_OK);
-  t.is(result.length, limit);
-});
-
-test('scope /players/returnsAll?limit=2&offset=2', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players/returnsAll?limit=${limit}&offset=2`;
-  const method = 'GET';
-
-  const { result, statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_OK);
-  t.is(result.length, 1, 'with only 3 players, only get 1 back with an offset of 2');
-});
-
-test('scope /players/returnsAll?limit=2&offset=20', async (t) => {
-  const { server } = t.context;
-  const limit = 2;
-  const url = `/players/returnsAll?limit=${limit}&offset=20`;
-  const method = 'GET';
-
-  const { statusCode } = await server.inject({ url, method });
-  t.is(statusCode, STATUS_NOT_FOUND, 'with a offset/limit greater than the data, returns a 404');
-});

src/crud.js

@@ -165,8 +165,6 @@ export const list = ({ server, model, prefix = '/', config }) => {
         where, include, limit, offset, order,
       });
 
-      if (!list.length) return void reply(notFound('Nothing found.'));
-
       reply(list.map((item) => item.toJSON()));
     },
 

src/index.js

@@ -38,6 +38,7 @@ const register = (server, options = {}, next) => {
     // Join tables
     if (model.options.name.singular !== model.name) continue;
 
+
     for (const key of Object.keys(model.associations)) {
       const association = model.associations[key];
       const { source, target } = association;
@@ -94,15 +95,11 @@ const register = (server, options = {}, next) => {
 
   // build the methods for each model now that we've defined all the
   // associations
-  Object.keys(models).filter((modelName) => {
-    const model = models[modelName];
-    return model.options.name.singular === model.name;
-  }).forEach((modelName) => {
+  Object.keys(models).forEach((modelName) => {
     const model = models[modelName];
     crud(server, model, options);
   });
 
-
   next();
 };
 

test/fixtures/models/player.js

@@ -27,13 +27,6 @@ export default (sequelize, DataTypes) => {
           name: 'notaname',
         },
       },
-      returnsAll: {
-        where: {
-          name: {
-            $ne: 'notaname',
-          },
-        },
-      },
     },
   });
 };