fix(options): add warning if provide limit or offset

fix #1
eclass · Sep 14, 2018 · 206a8ae · 206a8ae
1 parent 3c0abc7
commit 206a8ae
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -41,6 +41,8 @@ const options = {
 const { docs, pages, total } = await db.MyModel.paginate(options)
 ```
 
+**NOTE:** _If **options** include **limit** or **offset** are ignored._
+
 ## License
 
 [MIT](https://tldrlegal.com/license/mit-license)
diff --git a/src/index.js b/src/index.js
@@ -46,6 +46,14 @@ class SequelizePaginate {
       const pages = Math.ceil(total / params.paginate)
       options.limit = params.paginate
       options.offset = params.paginate * (params.page - 1)
+      /* eslint-disable no-console */
+      if (params.limit) {
+        console.warn(`(sequelize-pagination) Warning: limit option is ignored.`)
+      }
+      if (params.offset) {
+        console.warn(`(sequelize-pagination) Warning: offset option is ignored.`)
+      }
+      /* eslint-enable no-console */
       if (params.order) options.order = params.order
       const docs = await Model.findAll(options)
       return { docs, pages, total }