Вопрос 2.x Сегодняшние дни рождения

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#1
Вопрос по этому виджету. Подготавливаю форум к переходу на 2.х
Но вот с этим виджетом разобраться не могу. Он не показывает всех
1538736273327.png
Вот настройки
1538736302637.png
У меня же на форуме сегодня около 40 пользователей с днюхами.
 

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#4
Перестроение не помогло. :confused:
 

SOUL

АдминкА
Администратор
Сообщения
7.440
Симпатии
10.645
Баллы
266
#5
MEGAVOLT, в настройках конфиденциальности у пользователей, которые не отображаются в блоке, снята галочка в пункте Показывать день и месяц рождения?
 

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#6
SOUL, Нет. Галка стоит.
Может из за того, что это на локалке и на винде?
Попробую на линуксе поднять. Не знаю поможет ли...
 

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#8
А оно и именнинников то не всех показывает - глюк видимо движка...
1538993567970.png
В общем сам спросил, сам отвечаю.
Наводку на ответ нашел на офф сайте:

Пожалуйста, Войдите или Зарегистрируйтесь для просмотра скрытого текста.


Тут советовали в src/XF/Widget/Birthdays.php внести правку:
Вот это
PHP:
$userFinder = $this->finder('XF:User')
   ->isBirthday()
   ->isValidUser(true)
   ->order('username');
заменить вот этим
PHP:
$userFinder = $this->finder('XF:User')
   ->isBirthday()
   ->isRecentlyActive(365)
   ->isValidUser()
   ->order('username');
у меня же ->isRecentlyActive(365) уже был и я его закомментировал.
Дни рождения в виджете начало показывать
1538993721181.png
 

Skaiman

[SVG]ADD-ONS ™
Разработчик
Сообщения
6.094
Симпатии
5.870
Баллы
266
#9
И получим ошибку файла :)
--- Сообщение автоматически объединено: ---

Yes it will. Won't be too long until the next release.
Не надо торопиться поправят.
 

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#10
Самое что интересное - нет. Тоже думал ошибку файла получу - но ведь молчит...
--- Сообщение автоматически объединено: ---

Не надо торопиться поправят.
Дата 4е мая. А новая сборка как раз после вышла.
--- Сообщение автоматически объединено: ---

При этом если ->isRecentlyActive(365) увеличивать, в 2-3 раза, то он и отображать начинает пользователей в 2-3 раза больше чем изначально.
 

Skaiman

[SVG]ADD-ONS ™
Разработчик
Сообщения
6.094
Симпатии
5.870
Баллы
266
#11
Ну вероятно ты отключил ее :)
 

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#12
не))
Все настройки по дефолту стоят. Когда правлю другие файлы - ругается. А тут молчит.
Ладно, бум ждать новый релиз.
 

Skaiman

[SVG]ADD-ONS ™
Разработчик
Сообщения
6.094
Симпатии
5.870
Баллы
266
#13
Когда правлю другие файлы - ругается. А тут молчит.
Походу устал ругаться, плюнул и замолчал
--- Сообщение автоматически объединено: ---

Дата 4е мая. А новая сборка как раз после вышла.
Вот содержимое этого файла в последней версии 2.0.10
PHP:
<?php

namespace XF\Widget;

class Birthdays extends AbstractWidget
{
    protected $defaultOptions = [
        'limit' => 12
    ];

    public function render()
    {
        if (!\XF::visitor()->canViewMemberList())
        {
            return '';
        }

        $userFinder = $this->finder('XF:User')
            ->isBirthday()
            ->isRecentlyActive(365)
            ->isValidUser()
            ->order('username');

        if ($this->options['limit'])
        {
            $userFinder->limit($this->options['limit']);
        }

        $viewParams = [
            'users' => $userFinder->fetch()
        ];
        return $this->renderer('widget_birthdays', $viewParams);
    }

    public function verifyOptions(\XF\Http\Request $request, array &$options, &$error = null)
    {
        $options = $request->filter([
            'limit' => 'uint'
        ]);
        return true;
    }
}
И как видно все там исправлено, а вот твои вмешательства в файлы, ни чего хорошего не принесут
 
Последнее редактирование:

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#14
Вот содержимое этого файла в последней версии 2.0.10
Да, я про это говорил.
На оффе написали, что "про виджет правку забыли, и внесем ее в следующий релиз"
Релиз вышел, правка внесена, но изменений именно у меня нет. У меня же с точностью до на оборот. Без этой правки все нормально показывает.
Запустил проверку файлов - ругается на этот файл.
 

Skaiman

[SVG]ADD-ONS ™
Разработчик
Сообщения
6.094
Симпатии
5.870
Баллы
266
#15

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#16
меняю все как было, на файлы не ругается. юзеров из 40 показывает 6
Что еще может быть?
 

sonic

[KST]
Премиум
Сообщения
1.543
Симпатии
2.796
Баллы
153
#18
MEGAVOLT, блин ну нужно иметь понятия чего добавляешь в файлы движка..

ты закоментировал
->isRecentlyActive(365)
соответственно он будет показывать всех.. если не комментировать то покажет в виджете только тех кто был активен последний год.. если чел больше года не был активен на форуме он не попадет в виджет.


->isValidUser()
во второй ветке есть понятия валидный пользователь.. если пользователь зарегался, он еще не пользователь пока он не подтвердит регистрацию.. соответственно тот кто не подтвердил не попадают в статистику пользователей так как он не активирован.. соответственно он не будет попадать и в виджет..
 

MEGAVOLT

Проверенный
Сообщения
34
Симпатии
15
Баллы
3
#19
Вот теперь понял.
Я просто не понял к чему были эти дни здесь сделаны.
Спасибо!
 
Вверх Снизу