Skip to content

fix: preserve requested review filter and add ReviewsBatch.next_batch#41

Open
Flummy1 wants to merge 1 commit into
devfrom
fix/reviews-pagination
Open

fix: preserve requested review filter and add ReviewsBatch.next_batch#41
Flummy1 wants to merge 1 commit into
devfrom
fix/reviews-pagination

Conversation

@Flummy1

@Flummy1 Flummy1 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Что и зачем

В пагинации отзывов было два пробела по сравнению с транзакциями:

  1. Нет метода next_batch(). В ReviewsBatch уже лежали все данные для
    пагинации (next_review_id, filter, user_id), но не было способа
    запросить следующую страницу.

  2. Тот же баг с расширением фильтра, что и в транзакциях (fix: preserve requested transaction filter across pagination #40). FunPay в
    части ответов users/reviews не отдаёт скрытые инпуты filter и user_id,
    поэтому ReviewsParser считывает для них None. Любая пагинация, опирающаяся
    на спарсенное значение, молча теряла запрошенный фильтр по звёздам и id
    профиля.

Изменения

  • GetReviews.transform_result теперь штампует в батч запрошенные filter
    и user_id, перекрывая ненадёжные спарсенные значения — тот же подход, что и
    у GetTransactions в fix: preserve requested transaction filter across pagination #40.
  • Добавлен ReviewsBatch.next_batch():
    • бросает ValueError('Last batch.'), если next_review_id пуст;
    • бросает ValueError('Unknown user id.'), если user_id неизвестен (в
      отличие от транзакций, отзывам нужен явный user_id для пагинации);
    • передаёт сохранённый filter в следующий запрос.

Stamp the requested filter and user_id onto ReviewsBatch in
GetReviews.transform_result, since FunPay omits the hidden filter/user_id
inputs in some responses. Add ReviewsBatch.next_batch() to paginate while
preserving the filter, mirroring the transactions pagination fix.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Flummy1
Flummy1 requested a review from qvvonk as a code owner July 16, 2026 09:43
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

Successfully merging this pull request may close these issues.

1 participant