Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

La paginación de episodios no funciona al filtrar por show #1

Closed
juandjara opened this issue Apr 17, 2018 · 2 comments
Closed

La paginación de episodios no funciona al filtrar por show #1

juandjara opened this issue Apr 17, 2018 · 2 comments

Comments

@juandjara
Copy link
Owner

juandjara commented Apr 17, 2018

Cuando se piden episodios para un show, solo la primera página devuelve episodios. La propiedad episodes devuelta en el json es siempre un array vacio cuando se pide una página mayor que 0.
Esto ocurre aunque exista mas contenido que traerse de nyaa. Solo se recuperan los últimos 20 episodios mas recientes.

Ejemplo 1: https://nyapi.fuken.xyz/show/boruto-naruto-next-generations?page=1&meta=&source=hs
Ejemplo 2: https://nyapi.fuken.xyz/show/shingeki-no-bahamut-virgin-soul?page=1&meta=&source=hs

@juandjara juandjara changed the title Episode pagination is not working when filtering by show La paginación de episodios no funciona cuando filtras por show Apr 17, 2018
@juandjara juandjara changed the title La paginación de episodios no funciona cuando filtras por show La paginación de episodios no funciona al filtrar por show Apr 17, 2018
@ibaca
Copy link
Contributor

ibaca commented Apr 23, 2018

El problema es que (req.query.page || 0) + 1 termina traduciendose como 11 porque page es un string. Cambia la expresión por +(req.query.page || 0) + 1, seguramente mejor hacer lo mismo para rpp sea buena idea, o alguna otra forma mas JS friendly si existe (no suelo trabajar en js).
image

@juandjara
Copy link
Owner Author

Perfecto. Esta tiene toda pinta de ser la fuente del error. Se podría hacer lo que sugieres. Mañana si saco un rato lo pruebo.

juandjara added a commit that referenced this issue Apr 23, 2018
Fix #1 paginación falla por culpa del type coercion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants