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

Sign in to follow this  
mr. Freeman

Баги DLE v13.2

Recommended Posts

@celsoft, поправьте перед выходом DLE v13.3.
 

1. Код в файле /engine/ajax/editnews.php вообще ничего не знает о таблице _post_extras_cats, отсюда неадекватное поведение CMS в плане вывода новостей, т.к. при использовании AJAX, записи в эту таблицу не добавляются.

2. Если в настройках новостей стоит редактор Froala, и при этом для группы пользователя стоит allow_html=0, то при открытии модального окна управления загруженными картинками и файлами, будет JS ошибка, т.к. dle_image_upload не определён.

3. Если в настройках группы в allow_image_upload=1 и max_images=-1, то можно загрузить одну картинку в любое доп. поле. Логичнее бы при -1 запретить загружать изображения в модальном окне, но при этом разрешить загружать в доп. поля. Сейчас по сути ограничения какие то глупые, нельзя разрешить загружать только в доп. поля, т.е. можно просто набивать новость картинками которые в ней даже не используются.

Share this post


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

1. Код в файле /engine/ajax/editnews.php вообще ничего не знает о таблице _post_extras_cats, отсюда неадекватное поведение CMS в плане вывода новостей, т.к. при использовании AJAX, записи в эту таблицу не добавляются.

В быстром редактировании нет редактирования категорий. И ничего в эту таблицу там добавляться не должно. Соответственно ничего неадекватного в этом плане быть не может, при использовании оригинального скрипта.

 

1 час назад, mr. Freeman сказал:

2. Если в настройках новостей стоит редактор Froala, и при этом для группы пользователя стоит allow_html=0, то при открытии модального окна управления загруженными картинками и файлами, будет JS ошибка, т.к. dle_image_upload не определён.

Нет такой проблемы. Более того вы не прочитали описание этой настройки, а там написано:

Цитата

Данная опция разрешает использование HTML при добавлении с сайта. Внимание, если вы данную опцию отключите, а в настройках скрипта включен WYSIWYG редактор, то для данной группы редактор будет автоматически переключен в BB редактор.

другими словами при отключении HTML редактор Froala не может показываться на сайте, он также автоматически отключается, потому как визуальный редактор это чистый HTML код.

 

2 часа назад, mr. Freeman сказал:

3. Если в настройках группы в allow_image_upload=1 и max_images=-1, то можно загрузить одну картинку в любое доп. поле. Логичнее бы при -1 запретить загружать изображения в модальном окне, но при этом разрешить загружать в доп. поля. Сейчас по сути ограничения какие то глупые, нельзя разрешить загружать только в доп. поля, т.е. можно просто набивать новость картинками которые в ней даже не используются.

Для доп. полей действуют настройки доп. полей. И ничего другого. Настройки групп определяют настройки для всей публикации в целом. Доп это поле и просто поле, неважно все поля в публикации равноценны и одинаковы. А эта настройка на все поля, т.е. для всей публикации.

Share this post


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

В быстром редактировании нет редактирования категорий. И ничего в эту таблицу там добавляться не должно. Соответственно ничего неадекватного в этом плане быть не может, при использовании оригинального скрипта.

А как же опция "Опубликовать на сайте"? Вы про неё совсем забыли? Стоило бы попробовать сначала воспроизвести, прежде чем утверждать.

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

Нет такой проблемы. Более того вы не прочитали описание этой настройки, а там написано:

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

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

Для доп. полей действуют настройки доп. полей. И ничего другого. Настройки групп определяют настройки для всей публикации в целом. Доп это поле и просто поле, неважно все поля в публикации равноценны и одинаковы. А эта настройка на все поля, т.е. для всей публикации.

т.е. вы считаете что -1 = 1 в PHP логике?
Что делать с неиспользуемыми в тексте изображениями? Было бы куда логичнее дать возможность запрета загрузки изображений группе в редакторе, и оставить отдельно возможность загрузки изображений в доп. поля, это куда логичнее для обычного пользователя сайта чем разбираться как загрузить изображения в редакторе. Всё остальное уже делается легко и просто в шаблоне, и все довольны. Тоже самое и с файлами, а то например в одной категории нужно дать возможность прикрепить один файл, в другой два, в третьей три, а настройка количества файлов одна на все категории. Тут бы как раз пригодились доп. поля и запрет загрузки в редакторе, если значение равно -1.

Share this post


Link to post
Share on other sites
2 минуты назад, mr. Freeman сказал:

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

Нет вы не прочитали что я написал. Неважно использовать или нет краткое и полное описание. При включении этой настройки, визуальный редактор Floara не подключается и не выводится, в том числе и для доп. полей. Ни при добавлении новостей, ни при редактировании, поэтому не может одновременно быть запрещен HTML и визуальный редактор, который генерирует  HTML. При условии что все выводится стандартными тегами DLE. И никаких проблем в этом вопросе не зафиксировано. Прежде чем написать ответ, то что написали вы обязательно проверяется. Было проверено и описанное вами, багов не зафиксировано.

 

3 минуты назад, mr. Freeman сказал:

А как же опция "Опубликовать на сайте"? Вы про неё совсем забыли? Стоило бы попробовать сначала воспроизвести, прежде чем утверждать.

Вы сейчас о чем? Видимо вы как то неверно описываете проблему и место проблемы. Быстрое редактирование это редактирование через AJAX на сайте. Оно доступно только уже для опубликованных новостей. О какой опции опубликовать на сайте и где вы пишите? Опишите подробно точное место где вы наблюдаете проблему. Чтобы можно было проверить.

 

7 минут назад, mr. Freeman сказал:

т.е. вы считаете что -1 = 1 в PHP логике?

Причем здесь логика PHP? Вы в админпанели цифры пишите и PHP код? Здесь действует человеческая логика и описание на русском языке каждой настройки, а не логика PHP. Настройка групп это общая на всю публикацию, на весь контент публикации. И неважно доп. поля, или просто поля

Share this post


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

Нет вы не прочитали что я написал. Неважно использовать или нет краткое и полное описание. При включении этой настройки, визуальный редактор Floara не подключается и не выводится, в том числе и для доп. полей. Ни при добавлении новостей, ни при редактировании, поэтому не может одновременно быть запрещен HTML и визуальный редактор, который генерирует  HTML. При условии что все выводится стандартными тегами DLE. И никаких проблем в этом вопросе не зафиксировано. Прежде чем написать ответ, то что написали вы обязательно проверяется. Было проверено и описанное вами, багов не зафиксировано.

Я и не говорил что там Floara отображается, там включается BBкод-редактор, в нём это и не работает.

22 минуты назад, celsoft сказал:

Вы сейчас о чем? Видимо вы как то неверно описываете проблему и место проблемы. Быстрое редактирование это редактирование через AJAX на сайте. Оно доступно только уже для опубликованных новостей. О какой опции опубликовать на сайте и где вы пишите? Опишите подробно точное место где вы наблюдаете проблему. Чтобы можно было проверить.

Файл /engine/ajax/editnews.php отвечает только за одну функцию, редактирование новости в модальном окне, и там есть опция публикации.
И если у юзера есть права на редактирование новости после публикации, то он может снять её с публикации, однако в _post_extras_cats записи так же останутся и будут нагружать сервер выборкой бесполезных данных. Ну а если администратор активирует так новость, то на сайте она не появится, думаю сами понимаете почему.

24 минуты назад, celsoft сказал:

Причем здесь логика PHP? Вы в админпанели цифры пишите и PHP код? Здесь действует человеческая логика и описание на русском языке каждой настройки, а не логика PHP. Настройка групп это общая на всю публикацию, на весь контент публикации. И неважно доп. поля, или просто поля

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

Share this post


Link to post
Share on other sites
12 часов назад, mr. Freeman сказал:

Я и не говорил что там Floara отображается, там включается BBкод-редактор, в нём это и не работает.

Не подтверждается проблема. Проверено на чистом дистрибутиве, все работает, более того dle_image_upload эта функция а не переменная и находится в том же файле где и все функции BB редактора, т.е. не работало бы все если бы этот файл не загрузился, а не одна кнопка.

 

13 часов назад, mr. Freeman сказал:

Файл /engine/ajax/editnews.php отвечает только за одну функцию, редактирование новости в модальном окне, и там есть опция публикации.
И если у юзера есть права на редактирование новости после публикации, то он может снять её с публикации, однако в _post_extras_cats записи так же останутся и будут нагружать сервер выборкой бесполезных данных. 

Вот теперь понятна суть проблемы. Будет исправлено в следующей версии

Share this post


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

Не подтверждается проблема. Проверено на чистом дистрибутиве, все работает, более того dle_image_upload эта функция а не переменная и находится в том же файле где и все функции BB редактора, т.е. не работало бы все если бы этот файл не загрузился, а не одна кнопка.

Изучу поподробнее зависимости и отпишусь тогда позже.

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

Share this post


Link to post
Share on other sites
2 часа назад, mr. Freeman сказал:

По не совсем верному поведению лимитов на загрузку картинок и файлов подумаете?

Я уже ответил по данному вопросу. Пересмотров в логике не планируется.

  • 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