CMS DataLife Engine - Система управления сайтами

Sign in to follow this  
MrLOLs

Spoiler с кириллицей DLE 11.2

Recommended Posts

У одного меня после обновления с 11.1 до 11.2 не формируется спойлер если использовать кириллицу в заголовке

например BB-код:

[spoiler=Тест]Текст спойлера[/spoiler]

выдает спойлер с заголовком "Показать / Скрыть текст" и текстом внутри "Тест"

ВВ-код:

[spoiler=Test]Текст для спойлера[/spoiler]

срабатывает нормально

Edited by MrLOLs

Share this post


Link to post
Share on other sites

Нашел в чем ошибка) Этот баг проявляется только на PHP версии 5.4 и выше. Исправьте пожалуйста, и не забудьте протестировать на PHP 7.

Edited by MrLOLs

Share this post


Link to post
Share on other sites

Поподробнее описать то можно?
Какая функция где косячит...

  • Upvote 1

Share this post


Link to post
Share on other sites

Я не программист PHP поетому не стал исправлять, но явно в файле engine/classes/parse.class.php. Буду ждать патча от разработчиков.

Share this post


Link to post
Share on other sites
4 часа назад, MrLOLs сказал:

Я не программист PHP поетому не стал исправлять, но явно в файле engine/classes/parse.class.php. Буду ждать патча от разработчиков.

 

9 часов назад, MrLOLs сказал:

Нашел в чем ошибка

 

  • Upvote 1

Share this post


Link to post
Share on other sites

Есть такая проблема. В файле engine/classes/parse.class.php найдите

	function build_spoiler( $matches=array() ) {
		global $lang;

замените на 

	function build_spoiler( $matches=array() ) {
		global $lang, $config;

Share this post


Link to post
Share on other sites
4 часа назад, celsoft сказал:

Есть такая проблема. В файле engine/classes/parse.class.php найдите


	function build_spoiler( $matches=array() ) {
		global $lang;

замените на 


	function build_spoiler( $matches=array() ) {
		global $lang, $config;

А релиз v11.2 может тоже поправите учитывая огромное обилие свежеиспечённых багов?
Особенно учитывая что не малая их часть из-за нового парсера...

  • Upvote 1

Share this post


Link to post
Share on other sites
6 часов назад, SKYNET74 сказал:

А релиз v11.2 может тоже поправите учитывая огромное обилие свежеиспечённых багов?

Нет. По истечении, срок внесения изменений в дистрибутивы, очень короткий.

 

6 часов назад, SKYNET74 сказал:

А релиз v11.2 может тоже поправите учитывая огромное обилие свежеиспечённых багов?

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

Share this post


Link to post
Share on other sites
7 часов назад, celsoft сказал:

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

Были проблемы с парсером же до этого ещё, но они уже исправлены в дистрибутиве.

  • Upvote 1

Share this post


Link to post
Share on other sites
3 часа назад, SKYNET74 сказал:

Были проблемы с парсером же до этого ещё, но они уже исправлены в дистрибутиве.

Ключевое слово в том что они уже исправлены в дистрибутиве. А вы же писали про повторное обновление из за обилия свежеиспечённых багов. И вопрос был, о каком обилии свежеиспечённых идет речь.

Share this post


Link to post
Share on other sites
1 час назад, celsoft сказал:

Ключевое слово в том что они уже исправлены в дистрибутиве. А вы же писали про повторное обновление из за обилия свежеиспечённых багов. И вопрос был, о каком обилии свежеиспечённых идет речь.

Ну мы то все знаем, сколько их ещё всплывёт в недалеком времени... ;)
Вот например серьёзный баг:

И он судя по всему в релизе не пофиксен.

  • Upvote 1

Share this post


Link to post
Share on other sites
6 минут назад, SKYNET74 сказал:

Ну мы то все знаем, сколько их ещё всплывёт в недалеком времени... ;)

Для этого и выпускаются новые версии скрипта, а не правится дистрибутив по "каждому чиху". Все ошибки которые обнаружены между версиями, собираются и исправляются с выходом новой версии. Как собственно всегда и у любых программных продуктов, а не только в DLE.

Share this post


Link to post
Share on other sites
6 минут назад, celsoft сказал:

Для этого и выпускаются новые версии скрипта, а не правится дистрибутив по "каждому чиху". Все ошибки которые обнаружены между версиями, собираются и исправляются с выходом новой версии. Как собственно всегда и у любых программных продуктов, а не только в DLE.

Проблема в том что архитектура DLE подразумевает большое количество переработок кода для доведения его до нормального уровня, и у DLE нет никаких latest и stable, такое ощущение иногда, что выливают всегда latest и чувствуешь себя бета-тестером...
Было бы чудесно, подумать об промежуточных stable релизах, в которых нет нового функционала, но пофиксены ВСЕ известные баги на момент его выхода.

  • Upvote 1

Share this post


Link to post
Share on other sites

Поверьте, в DLE вполне всё стабильно, вы бы ZennoPoster поюзали, вот там последние годы ты ощущаешь себя вечным бета-тестором. А в DLE я использую правило - 2 недели после релиза уходят на отлов багов, происходит обновление дистрибутива и можно смело пользоваться. Идеального ничего не бывает.

Share this post


Link to post
Share on other sites
3 часа назад, Kolbaser сказал:

Поверьте, в DLE вполне всё стабильно, вы бы ZennoPoster поюзали, вот там последние годы ты ощущаешь себя вечным бета-тестором. А в DLE я использую правило - 2 недели после релиза уходят на отлов багов, происходит обновление дистрибутива и можно смело пользоваться. Идеального ничего не бывает.

Вы просто как обыватель не замечаете внутренних проблем, но это не значит что их нет, они есть, и они откладывают отпечаток уже на весь проект.
Из последнего примерами служили:
мусор в _subscribe
дубликаты индекса поиска по доп.полям, которые у многих до сих пор в базе мирно поживают и генерируют бесполезную нагрузку
дубликаты ID категорий у новостей (тут слава тебе великому линусу удалось убедить разработчика, но последствия "багованной" функции уже необратимо остались на сайтах).

  • Upvote 1

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this