Loadырь

Вот вам недалёкий пример платного компонента на конкурентной cms. Там и стоимость в 4-5 раз больше, чем самое дорогое дополнение тут и лицензии, как вы любите и договора с юр. лицами, и сроки поддержки в 3 (три (ТРИ)) рабочих дня  и прочее и т. п. Но это не мешает им ломаться при обновлении движка самой cms. Это не дает никакой уверенности, что это будет работать в будущем. Это просто есть в каталоге дополнений той cms и всё. Хотите, используете, не хотите, не используете. Успеете решить все вопросы за их 3 (три (ТРИ)) рабочих дня тех. поддержки — супер, не успеете — либо ваши проблемы, либо готовьте сумму на продление тех. поддержки.

Подобное можно встретить на любых других cms. Я к тому, что движок вообще никак не связан с каталогом дополнений. В каталоге дополнений предлагаются разного рода решения для расширения возможностей самой cms от пользователей этой системы. И чем больше каталог, тем лучше. Но их работоспособность зависит только от заинтересованности в этом их разработчиков. При этом оно не обязано работать на всех версиях движка, на всех шаблонах существующих и ещё не опубликованных и поддерживать все сторонние компоненты, и работать на протяжении существования всемирного интернета. Только у вас есть выбор — использовать это дополнение или искать другие варианты решения поставленной задачи на конкурентной cms.

Тянуть две версии движка (что есть и премиум) для Fuze это не вариант. Сегодня команда есть, а завтра она может матерясь разбежаться ))), оставив Fuze одного с кучей версий премиумов. Но если будет команда разработчиков, тогда вопрос: «Зачем им Fuze?». Ведь можно премиум разработку тянуть и без него. Ту же электронную коммерцию заваять они сами смогут, используя движок в качестве фреймворка и выдавать его за премиум компонент. Например, как интернет магазин на битрикс24 с crm системой по 99 000 р. в год.

Это была шутка, если что )))

В последнем ролике после ракеты не хватает космонавта, как на главной. Наш посыл не «лететь быстро непонятно куда», наш посыл — «болтаться среди звёзд», как lokanaft на главной (ИМХО).

Надо было через прямую черту ( | ) сделать в одном поле. Тогда бы точно всё было бы в порядке. Есть великий соблазн удалить класс из третьей строки первого поля и стили из второй строки второго поля и вот я уже бегу на форум с вопросом «Сайт сломали, но я ничего не делал, как быть?». 

Автор понимает, что встроенные стили  -это плохо и не современно

Главное, чтобы пользователи это понимали.

Тут если админ, то код с 96 по 117 строки можно пропустить не выполняя.

        // Если меняем под администратором, то причина всегда такая
        if ($this->cms_user->is_admin) {
            $reason = 'Статус изменен решением администрации сайта';
        } elseif ($reason_id) {
 
            // Поле мы получили ранее и делать это еще раз как минимум странно
            // Поэтому я убрал получение поля еще раз
            // Теперь получаем причины
            // Вот так было ранее
            // $reasons = $field['options']['reason'] ? preg_split('/\\r\\n?|\\n/', $field['options']['reason']) : '';
            // Но лучше это сделать иначе
            // Пустой массив по умолчанию
            $reasons = [];
            // Если опция есть
            if (!empty($field['options']['reason'])) {
 
                // Для распарсивания строк есть функция в движке
                $reasons = string_explode_list($field['options']['reason']);
            }
 
            // Теперь проверяем, что переданный id причины есть
            if (!empty($reasons[$reason_id])) {
                $reason = $reasons[$reason_id];
            }
        }
 

Оптимизация 😄

Если создать хук с названием render_page, то можно выводить шорткоды в любом html коде и не только в виджетах.

Я специально сделал не так, как предложил Fuze, чтобы послушать критику по этому поводу. Как говорится, век живи — век учись))

Ну так слушайте ))):

Ваш новый код:

if (!isset($this->item) || !$this->item['id'] || !$this->item['ctype_name'] || !$this->item['user_id']) {
 
    return '';
 
}

Не решает проблему с нотисами. $this->item может сушествовать, но может не содержать id и прочих элементов массива. Как говорил Fuze надо делать так

if (empty($this->item['id'])) {
 
    return '';
 
}

empty не выдает нотисов при отсутсвии элементов массива, но при этом вернет false если $this->item не существует. То есть вы одним условием проверяете сразу оба варианта.

$this->item['user_id'] можно не проверять, так как оно всегда идет в $this->item, если есть $this->item['id'].

$this->item['ctype_name'] в последних версиях движка тоже нет смысла проверять (оно есть, если есть $this->item['id'] даже в профилях пользователей), но вы его впоследствии отбросили.

То же самое тут

if ($reasons && isset($reasons[$reason_id])) {
 
    $reason = $reasons[$reason_id];
 
}

Достаточно так

if (!empty($reasons[$reason_id])) {
 
    $reason = $reasons[$reason_id];
 
}

Начало там такое же, поэтому сделаем всё то же самое:

Дублирующийся код желательно выносить в отдельный метод/файл

Надо переделать и не просто с комментариями, а с подробным объяснением почему именно так в данном случае, а не иначе. Это будет «весомый пендаль» в сторону саморазвития сторонних разработчиков.

👍 Наконец-то, хоть кто-то объяснил и показал, как это надо делать. Всё оказывается читаемо, легко и просто. Вообще не понимаю, за что тут люди дерут по 500 р. и выше 😂

Ахонь, 👍

А мы просто ждём. Молча так, тихо, ненавязчиво ЖДЁМ ))

Поднимите версию php до 7.3 или выше, если это возможно.

Можно же сделать «покультурнее». Есть переменная $filters

Есть же поле «Категории контента» в списке полей в 2.15.1 github.com/instantsoft/icms2/blob/master/system/fields/category.php

У автора в файлах профиля, но я почему-то сам его там не вижу. Видать не готово ещё обновление.

Прикручивать не надо. При обновлении форума до версии 2.4.8, автоматика всё сделает за вас. 

В требованиях к этому дополнению говорится следующее

Для добавления комментариев в избранное — InstantCMS 2.15.0 и выше (или текущая с GitHub).

Требуем данную плюшку для «народного» редактора TinyMCE 😁

А ещё лучше вынести данную фичу в виде жирной кнопки перед всеми ректорами через эту опцию github.com/instantsoft/icms2/blob/master/wysiwyg/redactor/wysiwyg.class.php#L82

Как-то так yadi.sk/d/p6MWX56SIe83CA

А зачем правка нужна в файле редактора, которая слетит минимум после следующего релиза? Можно же в popup_img.js указать селектор типа «content img». Или там не только класс добавляется к картинкам?