Загрузка...

API Исправлено Неправильная пагинация в Lolzteam API

Тема в разделе Рассмотренные недочеты создана пользователем its_niks 22 апр 2025. 245 просмотров

  1. its_niks
    its_niks Автор темы 22 апр 2025 make testers great again! 16 934 29 янв 2021
    Скриншот/видео недочета: [IMG]
    Полная ссылка на страницу, где возникает проблема: https://prod-api.zelenka.guru/posts?thread_id=3938630&page=12
    Как воспроизвести недочет: Как видите в пагинации показывает 3 страницы, но по факту их 12, я так понимаю это из-за того что начали недавно отображаться свои удаленные сообщения. Считаю можно добавить параметр, который отключает возврат своих удаленных сообщений. Ну или правильно отображать пагинацию с учетом их.
     
    1. Посмотреть предыдущие комментарии (1)
    2. AS7RID
      Если в настройках стоит галка показывать свои сообщения, то апи будет парсить посты пока не насчитает 20, а потом вернет. Собственно оно считает удаленные посты валидными и возвращает их сбивая всю пагинацию. Без галки норм
      Можно пример с веба взять и считать "валидными" только не удаленные посты, возвращая строго до 20 валидных постов и inf удаленных, но в теории можно так засрать страницу, что апи не будет успевать вернуть ответ :emm:

      uncpfiae Фича, что можно reply'ить на удаленные посты?
    3. its_niks Автор темы
      AS7RID, думаю можно параметр include_deleted_posts добавить. Если с ним запрос делать,, то в пагинации в данном случае будет 12 страниц, если без - 3. Все же думаю не хорошо, что ответ апи напрямую зависит от настроек на сайте
      22 апр 2025 Изменено
    4. xellan
      AS7RID, пользователи не должны знать о удаленных сообщениях и пагинация не должна быть сбитой. Если пропускать удаленные смс, то на странице окажется меньше смс, такой логике нет на сайте. К примеру на первой странице 2 сообщения валидные и 18 удаленных, а на второй странице 20 сообщений как положено.

      Если считать в апишке удаленные смс и нет, страниц будет отображено больше чем на сам деле, если удаленных смс так много.
      Достаточно сортировать данные без удаленных смс чтобы страницы и сообщения соответствовали с данными сайта.
      Если по какой то причине в апи нужны удаленные смс, для модерации и т.п можно выдавать роль чтобы к этому был доступ и данные не сортировались.
    5. Посмотреть следующие комментарии (1)
  2. uncpfiae
    uncpfiae 25 май 2025 ку 10 931 29 сен 2023
    удаленные сообщения теперь всегда отображаются в апи, независимо от настройки
     
Top
Загрузка...