powermaker450/simple-review-server
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Add skip parameter to /api/reviews

Browse source
This commit is contained in:
powermaker450 2024-09-20 12:38:23 -04:00
parent 1a967876b8
commit 65382936cf
2 changed files with 11 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/routes/reviews.ts
View file

@ -21,15 +21,15 @@ export class MessagesResponder extends ApiRoute {
}
// Same check as the above one, but for skip
// if (req.query.skip && !+req.query.skip) {
// res.writeHead(400, typeJson);
// res.write(Responder.requestError("skip parameter must be a number or undefined"));
// res.end();
// return;
// }
if (req.query.skip && !+req.query.skip || +req.query.skip < 1) {
res.writeHead(400, typeJson);
res.write(Responder.requestError("skip parameter must be a number or undefined"));
res.end();
return;
}
const receiver = req.headers["user-agent"];
const result = data.getReviews(+req.query.max);
const result = data.getReviews(+req.query.max, +req.query.skip);
res.writeHead(200, typeJson);
res.write(JSON.stringify(result));

7
src/utils/ReviewData.ts
View file

@ -32,17 +32,18 @@ export class ReviewData {
});
}
public getReviews(max?: number): ServerSideReview[] {
public getReviews(max?: number, skip?: number): ServerSideReview[] {
let final: ServerSideReview[] = [];
// If "max" isn't provided, is Infinity, or is greater than the amount of reviews
const sendAllReviews = !max || max === Infinity || max > this.data.length;
const useList = skip ? this.data.slice(skip, this.data.length) : this.data;
if (sendAllReviews) {
for (const review of this.data) {
for (const review of useList) {
final.push(review);
}
} else {
for (const review of this.data) {
for (const review of useList) {
while (final.length !== max) {
final.push(review);
}