Question Threads 2.1.2

CMTV · 21 Апр 2019

Угу. Написали уже и на офф форуме. Сейчас исправлю.

TechSide Там ошибка возникла уже в после-установочном процессе. То есть сама установка должна была пройти нормально. В вашем случае далее проблем быть не должно.
Все работает корректно?

Captain · 21 Апр 2019

Аддон как и все переписать бы :). Доставляет удовольствие.

PHP:

namespace CMTV\QuestionThreads;

use CMTV\QuestionThreads\Constants as C;

class Constants
{
    const ADDON_ID = [
        'CMTV',
        'QuestionThreads'
    ];

    const ADDON_ID_SHORT = [
        'CMTV',
        'QT'
    ];

    public static function _(string $content = ''): string
    {
        $shortId = implode('_', self::ADDON_ID_SHORT);

        return $shortId . (empty($content) ? '' : '_' . $content);
    }

    public static function __(string $content): string
    {
        return implode('\\', self::ADDON_ID) . ':' . $content;
    }
}

PHP:

case C::_('questions_asked'):
                $db = \XF::db();
                $questions = $db->fetchOne(
                    "SELECT COUNT(*) FROM `xf_thread` WHERE `CMTV_QT_is_question` = 1 AND `user_id` = ?",
                    $user->user_id
                );

                $returnValue = $questions >= $data['questions'];
                break;
            case C::_('questions_maximum'):
                $db = \XF::db();
                $questions = $db->fetchOne(
                    "SELECT COUNT(*) FROM `xf_thread` WHERE `CMTV_QT_is_question` = 1 AND `user_id` = ?",
                    $user->user_id
                );

                $returnValue = $questions <= $data['questions'];
                break;

МММ а потом вот из-за таких авторов не подцепить что то другое минуя абстракцию и их прямыми запросами в бд. Привет Kolya groza morey, _Werewolf_, fara, AndyB, etc...
Ну раз бесплатный то и так сойдёт в принципе а не продаются... и вешать столько поддержки ну не найс =)...

CMTV · 21 Апр 2019

А что не так с Constants?

Прямые запросы к БД - да нехорошо. Но другого пути без создания отдельных счетчиков вопросов я не нашел.

Captain · 21 Апр 2019

Так никто и не говорит, что они плохие. Суть в том, что лучше их вынести было бы в какие то утилы, с нормальными именами и комментариями. И в чём смысл юз этого же класса.
Насчёт жобов, ничего не говорю и разработчики сами грешат этим, но они стараются с каждым релизом избавляться от них. Для критериев, что идёт обычный файндер который найдёт к тому же идёт 1 и тот же запрос и вынести его выше свитча в виде файндера и и потом уже сравнивать в самом запросе свитче $questions <= $data['questions'].

CMTV · 21 Апр 2019

Captain написал(а):
утилы, с нормальными именами и комментариями

Типо как тут?

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

Просто мне стало лень придумывать разные имена. Решил сделать покороче. На это, кстати и напоролся. Эта ошибка выше косвенно из-за этого и вылезла.

Captain написал(а):
И в чём смысл юз этого же класса.

Смысл в том, чтобы писать C::_('phrase'), а не 'CMTV_QT_phrase' и C::__('BestAnswer'), а не 'CMTV/QuestionThreads:BestAnswer'.

Captain написал(а):
Для критериев, что идёт обычный файндер который найдёт к тому же идёт 1 и тот же запрос и вынести его выше свитча в виде файндера и и потом уже сравнивать в самом запросе

Можно и так. Просто я вроде где-то слышал (от Майка вроде), что Finder работает медленно. Но, возможно, это было про другое... Посмотрю, что можно сделать.

Captain · 21 Апр 2019

Да почти, только ещё комментарии.

CMTV написал(а):
Можно и так. Просто я вроде где-то слышал (от Майка вроде), что Finder работает медленно. Но, возможно, это было про другое... Посмотрю, что можно сделать.

Эм 1 и тот же запрос будет, не минуя абстракцию каким образом оно будет медленнее?

CMTV · 21 Апр 2019

Captain так?

PHP:

$questions = \XF::finder('XF:Thread')
    ->where('CMTV_QT_is_question', true)
    ->where('user_id', $user->user_id)
    ->fetch()
    ->count();

Тут проблема в том, что он возвращает массив Entity. А что если вопросов 1000, 10000?

Captain написал(а):
каким образом оно будет медленнее

Да, наврал.

Вот та тема. Там про другое речь шла.

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

unconfirmed · 21 Апр 2019

Captain написал(а):
Эм 1 и тот же запрос будет, не минуя абстракцию каким образом оно будет медленнее?

.в чём смысл? Если один и тот же запрос. Разработчики сами спокойно используют такие запросы. Одно дело использовать finder для того, чтобы получить какие-то объекты из БД, но когда тебе нужно сделать select count это не имеет смысла. Лишнюю память только занимать.

Captain · 21 Апр 2019

Они от них избавляются потихоньку с каждым релизом и переделывают и это прекрасно видно. Прикрутить свой любой адаптер и использовать тот же psql будет проблемно от таких прямых запросов.

West14 · 21 Апр 2019

CMTV, вместо fetch()->count() используйте total().

CMTV · 21 Апр 2019

CMTV обновил(а) ресурс Question Threads новой записью:

Исправления 2.1.1

Исправления

Исправлена ошибка во время установки

Узнать больше об этом обновлении...

Captain · 21 Апр 2019

CMTV написал(а):
Captain так?

PHP:

$questions = \XF::finder('XF:Thread') ->where('CMTV_QT_is_question', true) ->where('user_id', $user->user_id) ->fetch() ->count();

Тут проблема в том, что он возвращает массив Entity. А что если вопросов 1000, 10000?

Да, наврал.

Вот та тема. Там про другое речь шла.

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

Не совсем так, а вот так:

PHP:

$questions = \XF::finder('XF:Thread')
    ->where([
                    'CMTV_QT_is_question' => true,
                    'user_id' => $user->user_id
])->total();

Мы можем использовать массив

unconfirmed · 21 Апр 2019

Captain написал(а):
Они от них избавляются потихоньку с каждым релизом и переделывают и это прекрасно видно.

.чего? :D В каком это месте?

Captain написал(а):
Прикрутить свой любой адаптер и использовать тот же psql будет проблемно от таких прямых запросов.

Так запрос пропускается через адаптер как бы в любом случае.

CMTV написал(а):
@Captain так?

Всё нормально у тебя. Если ты будешь делать это через Finder, то ты будешь только тратить лишние ресурсы, а это уже не оптимизация.

Captain · 21 Апр 2019

И я могу кучу привести примеров и из релиза в релиз переделывается это всё. Писать прямые запросы в бд, минуя абстракцию - моветон. И что если он пропускается? Если ты не прикрутишь не psql, mssql, mongodb и будет криво работать с этими запросами. в XF 2 не просто так используется ORM и нужно работать правильно с ним и если ты хочешь использовать любую другую субд помимо mysql ты это должен получить результат. Прямые запросы в бд это лишь подводит черту к тому, что ты будешь жёстко привязан лишь к 1 субд. Каким образом оно будет больше тратить памяти? Ну расскажи мне как устроена память и как она работает на низком уровне уж очень интересно. Раз будет больше требовать ресурсов это очень интересно :D.

unconfirmed · 21 Апр 2019

Captain написал(а):
И я могу кучу привести примеров и из релиза в релиз переделывается это всё.

.так ты посмотри, в каких случаях они используют total, в большинстве он используется только когда они с помощью него получают объекты.

Captain написал(а):
Каким образом оно будет больше тратить памяти? Ну расскажи мне как устроена память и как она работает на низком уровне уж очень интересно. Раз будет больше требовать ресурсов это очень интересно :D.

memory_get_usage при db()->fetchOne('select count(*) from xf_user')
До: 4590344
После: 4592336

При $app->finder('XF:User')->total()
До: 4590320
После: 5428696

CMTV · 21 Апр 2019

CMTV обновил(а) ресурс Question Threads новой записью:

Исправления 2.1.2

Исправлена проблема с обновлением с версии 2.0.2

Узнать больше об этом обновлении...

TechSide · 22 Апр 2019

CMTV написал(а):
Угу. Написали уже и на офф форуме. Сейчас исправлю.

TechSide Там ошибка возникла уже в после-установочном процессе. То есть сама установка должна была пройти нормально. В вашем случае далее проблем быть не должно.
Все работает корректно?

Переустановил обновленную версию - всё стало без проблем и ошибок! Спасибо! Но нет иконки и фразы "пометить как лучший ответ"!

Sidd · 22 Апр 2019

TechSide написал(а):
Но нет иконки и фразы "пометить как лучший ответ"!

А права у группы выставлены?

У меня все установилось нормально и все иконки/опции работают корректно

Sidd · 1 Май 2019

Когда создаешь тему из сообщения и переносишь в форум с ответами и вопросами, в которой только вопросы, то тема остается темой и конвертировать ее в вопрос можно только через админку.
Предложение: сделать конвертацию автоматической или через опцию в меню редактирования темы.

Bams · 5 Июн 2019

CMTV, здравствуйте, помогите разобраться с некоторыми проблемами пожалуйста.
Внутри категории где расположены списки вопросов на форуме, не отображаются иконки вообще никакие, а вот внутри самой темы все на месте

Не применились модификации я так понял

Стиль U/I 2

Question Threads 2.1.2

CMTV

Captain

Какие бекапы? О_о

CMTV

Captain

Какие бекапы? О_о

CMTV

Captain

Какие бекапы? О_о

CMTV

unconfirmed

Captain

Какие бекапы? О_о

West14

meow

CMTV

Captain

Какие бекапы? О_о

unconfirmed

Captain

Какие бекапы? О_о

unconfirmed

CMTV

TechSide

Sidd

Sidd

Bams

Похожие темы

Мы ценим вашу конфиденциальность