Fixed querying a specific article
This commit is contained in:
@@ -9,6 +9,7 @@ Run the SQL script `sql/create_database.sql`. This will set up an empty database
|
||||
# API
|
||||
|
||||
```
|
||||
//NOTE: GET will return null if a specific article can't be found
|
||||
//NOTE: you can add a "limit" query parameter to change the default limit
|
||||
GET /news?limit=10
|
||||
|
||||
|
||||
@@ -1,3 +1,4 @@
|
||||
const { Op } = require('sequelize');
|
||||
const { articles } = require('../database/models');
|
||||
|
||||
//the query function that can be reused
|
||||
@@ -8,13 +9,14 @@ const query = (ascending, titlesOnly) => async (req, res) => {
|
||||
attributes: [
|
||||
'index', 'title', 'author', ...(!titlesOnly ? ['body', 'edits'] : [])
|
||||
],
|
||||
order: [
|
||||
['index', ascending ? 'ASC' : 'DESC']
|
||||
],
|
||||
offset: parseInt(req.query.id) || 0,
|
||||
limit: 1
|
||||
where: {
|
||||
index: {
|
||||
[Op.eq]: ascending ? parseInt(req.params.id) : (await articles.max('index')) - parseInt(req.params.id) + 1
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
//returns null if failed to find
|
||||
return res.status(200).json(result);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user