1

Тема: Возможно баг: неправильно отображаются списки пользователей

Заметил, что этот скрипт /admin/users.php
неправильно отображает длинные списки пользователей. Версия форума 1.4.4.
По умолчанию у меня на форуме стоит отображение 25 элементов списка.
Первая страница показывает 25, но последующие (до середины) увеличивают количество отображаемых элементов.
После середины количество снова уменьшается.

Ошибка кроется в файле (Please log in or register to see this URL):

$forum_page['finish_at'] = min(($forum_page['start_from'] + $forum_user['disp_topics']), ($forum_page['num_users']));

SQL запрос ведь формируется так:

            if (!empty($query['LIMIT']))
                $sql .= ' LIMIT '.$query['LIMIT'];

А оператор LIMIT в качестве второго аргумента принимает не номер максимальной запписи, а КОЛИЧЕСТВО записей.
(Please log in or register to see this URL)

Если исправить ошибку:

$forum_page['finish_at'] = min(($/*forum_page['start_from'] +*/ $forum_user['disp_topics']), ($forum_page['num_users']));

То всё работает идеально.

В файле \userlist.php очевидно тоже [url=$forum_page['finish_at'] = min(($forum_page['start_from'] + 50), ($forum_page['num_users']));]допущена такая же ошибка[/url], но она потом (Please log in or register to see this URL), и всё работает.

Добавлено спустя 53 секунды:

Ошибка в файле Userlist: (Please log in or register to see this URL)

Post's attachments

1.PNG 245.86 Кб, файл не был скачан. 

2.PNG 254.35 Кб, файл не был скачан. 

You don't have the permssions to download the attachments of this post.

Поделиться

2

Re: Возможно баг: неправильно отображаются списки пользователей

(Please log in or register to see this URL)

Поделиться

3

Re: Возможно баг: неправильно отображаются списки пользователей

synthetic пишет:

(Please log in or register to see this URL)

И кто-то там скобок до фига лишних наставил smile

Моя сборка FluxBB 1.5 - fluxbb.qb7.ru, парсер сообщений с бб-кодами "Parserus" и анализатор useragent "UserAgentAnalyzer" - github.com/MioVisman/

Сайт Visman

Поделиться

4

Re: Возможно баг: неправильно отображаются списки пользователей

Неужели никто не будет исправлять?
Есть ли какой-нибудь багтрекер куда бы я мог написать об этой ошибке?
Подскажите, пожалуйста.

Поделиться

5

Re: Возможно баг: неправильно отображаются списки пользователей

(Please log in or register to see this URL)

Сайт sempai

Поделиться

6

Re: Возможно баг: неправильно отображаются списки пользователей

вот именно этим багрепортом пользоваться не надо, он для особо критичных в план безопасности проблем. сделайте патч, отправьте запрос на слиянеи в гитхабе,  на крайний случай создайте issue : (Please log in or register to see this URL)

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

7

Re: Возможно баг: неправильно отображаются списки пользователей

Создал ISSUE (Please log in or register to see this URL). Посмотрим, за какое время исправится.

Поделиться

8

Re: Возможно баг: неправильно отображаются списки пользователей

Сделал даже Pull Request: (Please log in or register to see this URL)
Авось это поможет

Поделиться

9

Re: Возможно баг: неправильно отображаются списки пользователей

Ура, объединили smile

Поделиться