Skip to content

Commit a915589

Browse files
committed
Fixing defaults.. Again.
1 parent 0eb9ab6 commit a915589

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

src/services/TermsOfUseService.js

+7-4
Original file line numberDiff line numberDiff line change
@@ -409,16 +409,16 @@ deleteTermsOfUse.schema = {
409409
* @returns {Object} the search result, contain total/page/perPage and result array
410410
*/
411411
async function searchTermsOfUses (criteria) {
412-
const page = criteria.page || 1
413-
const perPage = criteria.perPage || 20
412+
const page = criteria.page > 0 ? criteria.page : 1
413+
const perPage = criteria.perPage > 0 ? criteria.perPage : 20
414414

415415
const countResult = await TermsOfUse.findOne({
416416
attributes: [[models.Sequelize.fn('COUNT', models.Sequelize.col('id')), 'total']],
417417
where: _.assign({ deletedAt: null }, _.omit(criteria, ['perPage', 'page'])),
418418
raw: true
419419
})
420420

421-
const result = await TermsOfUse.findAll({
421+
const query = {
422422
order: [['id', 'ASC']],
423423
attributes: ['id', 'legacyId', 'title', 'url', 'agreeabilityTypeId'],
424424
include: [
@@ -435,7 +435,10 @@ async function searchTermsOfUses (criteria) {
435435
limit: perPage,
436436
offset: (page - 1) * perPage,
437437
raw: true
438-
})
438+
}
439+
const result = await TermsOfUse.findAll(query)
440+
441+
logger.debug(`Query: ${JSON.stringify(query)}`)
439442

440443
for (const element of result) {
441444
convertRawData(element, false)

0 commit comments

Comments
 (0)