Loadырь

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

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

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

Тут если админ, то код с 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». Или там не только класс добавляется к картинкам?

Всё верно. Ведь вы же назвали файл ваштипконтента_list.tpl.php, а не default_list_ваштипконтента.tpl.php. Когда есть файл ваштипконтента_list.tpl.php, всегда будет работать только он, поэтому сам факт выбора стиля не нужен.

Обновите файлы шаблона админки admincoreui

Смотреть надо настройки темы во вкладке «Интерфейс».