Форумы / National / Russian / The Release v0.9.22

мы по релизу и говорим.  Попросили  скин.  Я попросил посмореть - подойдет ли вам такой.  

Зачем темы плодить.  мне всё равно возьмёте вы его или нет.  ( мне наплевать на оценки скина )  Я просто преддложил два готовых варианта.  светлый по умолчанию  такой же , но тёмный доступен = близнецы, только цвет отличается.

#46198 webitproff:

хм... странно.

дисковый кеш вычищен, в структуре синхронизацию делал, "Оставить ссылку в старом разделе " чекбокс не ставил

Добавлено 3 минуты спустя:

соответственно и подсчет количества тем и сообщений в них здесь не корректный

Исправлено этим коммитом

Прошу потестировать. Нужно скачать обновленную версию модуля форумов с мастер ветки, заменить ей то что установлена и обновить в админке. Далее можно в структуре "Синхронизировать все". Должно пересчитать. Надесь что в правках ничего не упустил. Их много.

Об ошибках лучше писать в issue на гитхабе #1689, чтобы не потерялись и заново открыть сам Issue.

#46223 Alex300:

что тут понять? по урл в новую тему, будь модератором, - удалил бы сразу

А хочешь быть? У нас тут как раз вакансия и толковые парни нам не помешают.

Серьёзно? Каждоё второе сообщение содержит завуалированый мат. От того что он завуалированный матом он не перестаёт быть.

Кстати почему завуалирован? Что б не нарушать правила или нормы этикета? Ну тогда уже следует заметить модератор априори их сам не может и не должен нарушать. Так же как и бросатся угрозами "Исправьте или удалю". И вообще что б кому то говорить что вы что то нарушили должны быть чётко обозначенные правила что можно, а чего нельзя. До тех пор пока их нет уже каждый для себя сам решает что можно, а что нельзя. Но как то за все года необходимости в составлении правил как и в модераторах не было потому как всегда всё более менее всё саморегулировалось - никто не ругался и не нарушал, единственное что было проблемой за всё это время это редкие залётные спамеры которые в целом оперативно вычищались.

Потому давайте для начала если хотим что б движёк хоть чуть-чуть ожил, сперва умерим пыл, уберём ейфорию и спокойно, без выражений и целенаправлено трудится не поря гарячку. Трудится по мере собтвенных возможностей. Тут вроде детей и подростков нет потому прочь юношеский максимализм, побольше реализма в взяглядах на ситуацию и уж тем более на её исправление, потому как требовать или хотеть можно много чего вот только кому и когда это всё делать сложно понять.

Добавлено 2 часа спустя:

#46223 Alex300:

Dayver занят логированием.

Кстати не только им, еще, практически закончил, актуализацию украинского языкового пакета.

Добавлено 17 минут спустя:

#46221 Kabak:

Можно было бы добавить автоматическое добавление обязательных для заполнения экстраполей при регистрации пользователя в формы модуля Users

Эту задачу частично можно решить по другому

#46224 Kabak:

Это жёсткая привязка к имени экстраполя в TPL.   а я говорю об автоматическом сканировании модулем USERS созданных в движке экстраполей для обязательного заполнения пользователем при регистрации. 

Например, делаю сайт для дачников и нужно обязательное для заполнения при регистрации экстрополе НОМЕР ДОМА   я его создаю в экстраполях движка и не должен париться с TPL модуля USERS  этот модуль должен сам вписывать такие обязательные для заполнения поля во все свои формы

Допустим есть дефолтный шаблон users.register.tpl ... допустим вы захотели создать таблице пользователей екстраполе(я) .... создали, далее два пути - либо кастомно вставляете для каждого поля свою пару {USERS_REGISTER_имяПоля} и {USERS_REGISTER_имяПоля_TITLE}. Либо идёте вторым путём - вставляете в шаблон секцию EXTRAFLD и только два тега {USERS_REGISTER_EXTRAFLD} и {USERS_REGISTER_EXTRAFLD_TITLE} типа вот так:

...
<tr>
						<td>{PHP.L.Password}:</td>
						<td>{USERS_REGISTER_PASSWORD} *</td>
					</tr>
					<tr>
						<td>{PHP.L.users_confirmpass}:</td>
						<td>{USERS_REGISTER_PASSWORDREPEAT} *</td>
					</tr>
<!-- BEGIN: EXTRAFLD -->
					<tr>
						<td>{USERS_REGISTER_EXTRAFLD_TITLE}:</td>
						<td>{USERS_REGISTER_EXTRAFLD}</td>
					</tr>
<!-- END: EXTRAFLD -->
...

Тогда после создания каждого нового екстраполя не нужно будет вставлять теги для него и эта секция отобразит все созданные поля. 

Это немного не вписывается в задачу автоматического вывода только обязательных полей но при желании можно это подумать как решить. Надо проверять можно ли обойти  отображение поля в секции проверкой обязательности ... может быть без доработки нельзя но по памяти утверждать что нельзя не стану

Добавлено 11 минут спустя:

<!-- BEGIN: EXTRAFLD -->
					<!-- IF {PHP.exfld.field_required} -->
                    <tr>
                        <td>{USERS_REGISTER_EXTRAFLD_TITLE}:</td>
                        <td>{USERS_REGISTER_EXTRAFLD}</td>
                    </tr>
					<!-- ENDIF -->
<!-- END: EXTRAFLD -->

#46249 Dayver:

#46224 Kabak:

Это жёсткая привязка к имени экстраполя в TPL.   а я говорю об автоматическом сканировании модулем USERS созданных в движке экстраполей для обязательного заполнения пользователем при регистрации. 

Например, делаю сайт для дачников и нужно обязательное для заполнения при регистрации экстрополе НОМЕР ДОМА   я его создаю в экстраполях движка и не должен париться с TPL модуля USERS  этот модуль должен сам вписывать такие обязательные для заполнения поля во все свои формы

Допустим есть дефолтный шаблон users.register.tpl ... допустим вы захотели создать таблице пользователей екстраполе(я) .... создали, далее два пути - либо кастомно вставляете для каждого поля свою пару {USERS_REGISTER_имяПоля} и {USERS_REGISTER_имяПоля_TITLE}. Либо идёте вторым путём - вставляете в шаблон секцию EXTRAFLD и только два тега {USERS_REGISTER_EXTRAFLD} и {USERS_REGISTER_EXTRAFLD_TITLE} типа вот так:

...
<tr>
						<td>{PHP.L.Password}:</td>
						<td>{USERS_REGISTER_PASSWORD} *</td>
					</tr>
					<tr>
						<td>{PHP.L.users_confirmpass}:</td>
						<td>{USERS_REGISTER_PASSWORDREPEAT} *</td>
					</tr>
<!-- BEGIN: EXTRAFLD -->
					<tr>
						<td>{USERS_REGISTER_EXTRAFLD_TITLE}:</td>
						<td>{USERS_REGISTER_EXTRAFLD}</td>
					</tr>
<!-- END: EXTRAFLD -->
...

Тогда после создания каждого нового екстраполя не нужно будет вставлять теги для него и эта секция отобразит все созданные поля. 

Это немного не вписывается в задачу автоматического вывода только обязательных полей но при желании можно это подумать как решить. Надо проверять можно ли обойти  отображение поля в секции проверкой обязательности ... может быть без доработки нельзя но по памяти утверждать что нельзя не стану

Добавлено 11 минут спустя:

<!-- BEGIN: EXTRAFLD -->
					<!-- IF {PHP.exfld.field_required} -->
                    <tr>
                        <td>{USERS_REGISTER_EXTRAFLD_TITLE}:</td>
                        <td>{USERS_REGISTER_EXTRAFLD}</td>
                    </tr>
					<!-- ENDIF -->
<!-- END: EXTRAFLD -->

 

А можно в TPL проверять экстрополя на обязательность через IF  и вставлять все которые помечены как обязательные для заполнения ?  т.е.  где-то создавать общий тег для таких полей в виде таблицы

<tr>
<td>{USERS_REGISTER_EXTRAFLD_TITLE}:</td>
<td>{USERS_REGISTER_EXTRAFLD}</td>
</tr>

Добавлено 8 минут спустя:

т.е.  в этот тэг движок должен автоматически запихивать все экстрополя которые помечены как обязательные для профиля пользователя. и соответственно они должны показываться  на форме при регистрации и в профиле пользователя

Добавлено 4 минуты спустя:

#46246 Alex300:

#46198 webitproff:

хм... странно.

дисковый кеш вычищен, в структуре синхронизацию делал, "Оставить ссылку в старом разделе " чекбокс не ставил

Добавлено 3 минуты спустя:

соответственно и подсчет количества тем и сообщений в них здесь не корректный

Исправлено этим коммитом

Прошу потестировать. Нужно скачать обновленную версию модуля форумов с мастер ветки, заменить ей то что установлена и обновить в админке. Далее можно в структуре "Синхронизировать все". Должно пересчитать. Надесь что в правках ничего не упустил. Их много.

Об ошибках лучше писать в issue на гитхабе #1689, чтобы не потерялись и заново открыть сам Issue.

при переносе темы получил

Fatal error

2023-03-30 05:42

SQL error 22001: String data, right truncated: 1406 Data too long for column 'log_group' at row 1

#0 /var/www/dacha/system/database.php(614): cot_diefatal('...')
#1 /var/www/dacha/system/functions.php(3368): CotDB->insert('...', Array)
#2 /var/www/dacha/modules/forums/inc/forums.topics.php(127): cot_log('...', '...', '...', '...')
#3 /var/www/dacha/modules/forums/forums.php(35): include('...')
#4 /var/www/dacha/index.php(180): require_once('...')

тема переехала, но с этой ошибкой на экране

#46268 Kabak:

2023-03-30 05:42

SQL error 22001: String data, right truncated: 1406 Data too long for column 'log_group' at row 1

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

INSERT INTO `cot_config` (`config_owner`, `config_cat`, `config_order`, `config_name`, `config_type`, `config_value`, `config_default`, `config_variants`, `config_text`) VALUES
('core','main','08','loggerlevel',2,'sec+adm+ext','sec+adm+ext','none,sec,adm,ext,sec+adm,sec+ext,adm+ext,sec+adm+ext,all','');

ALTER TABLE `cot_logger` MODIFY `log_group` varchar(64) DEFAULT 'adm';
ALTER TABLE `cot_logger` ADD `log_uid` int UNSIGNED NOT NULL DEFAULT '0';
ALTER TABLE `cot_logger` ADD `log_type` varchar(32) DEFAULT '';
ALTER TABLE `cot_logger` ADD `log_status` varchar(24) DEFAULT '';
ALTER TABLE `cot_logger` ADD `log_uri` varchar(255) DEFAULT '';

UPDATE `cot_logger` SET `log_group` = 'forums' WHERE `log_group` = 'for';
UPDATE `cot_logger` SET `log_group` = 'users' WHERE `log_group` = 'usr';
UPDATE `cot_logger` SET `log_group` = 'page' WHERE `log_group` = 'pag';

#46281 Kabak:

а сайт не ляжет ?  там сейчас живые люди ...      в вынужден был полностью удалить базу и накатить старую  потому,  что через  PHPMyadmin  не устанавливалась старая база поверх  новой  тогда ещё 0.9.21

Тогда наверное не стоит. Просто мастер ветка не очень подходит для продакшена и для рабочих сайтов лучше релизные версии использовать. А вот на тестовых сайтах, тут да мастер ветку обкатывать можно.

#46286 Kabak:

Встроенный поиск  ищет в комментариях ?

Нет, такой реализации нет в коробке

Добавлено 13 минут спустя:

#46286 Kabak:

Баг ?

поиск находит разделы на форуме которые должны быть скрыты от пользователей по уровню  ( например, от незарегистированных ) и потом можно на эти разделы перейти по ссылкам в поиске

писать в этих разделах не даёт, но читать можно  Баг ?

Вероятно да, что б не потерялось нужно в гитхабе отсавить заявку. Нужно внимательно проверить действительно ли имеет место быть даная ошибка

Надо проверить 2 кейса:

1) Тема находится в категории на чтение которой у пользователя нет прав. Т.e. у этой категории  https://www.site.ru/admin/rights?g=1 (g=1 - гости) в Module Forums структура сняты все галки.

• Выдает ли результаты поиска из этой категории?
• Дает ли посмотреть если зайти в темы по прямым ссылкам?

2) Тема находится в категории на чтение которой у пользователя есть права, но сама тема приватная (доступна только создателю и модеоаторам/админам)

Теже вопросы.

Это бы очень помогло.

Добавлено 1 день спустя:

Исправлено этим коммитом. Но пока не тестировал. Потестирую позже. И желательно еще чье-то тестирование. Если все ок, issue можно закрывать.

P.S. А почему это тема стала приватной?