Баги DLE v13.2

[mr. Freeman 11]

@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 запретить загружать изображения в модальном окне, но при этом разрешить загружать в доп. поля. Сейчас по сути ограничения какие то глупые, нельзя разрешить загружать только в доп. поля, т.е. можно просто набивать новость картинками которые в ней даже не используются.

[celsoft 6 076]

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 запретить загружать изображения в модальном окне, но при этом разрешить загружать в доп. поля. Сейчас по сути ограничения какие то глупые, нельзя разрешить загружать только в доп. поля, т.е. можно просто набивать новость картинками которые в ней даже не используются.

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

[mr. Freeman 11]

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

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

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

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

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

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

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

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

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

[celsoft 6 076]

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

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

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

 

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

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

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

 

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

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

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

[mr. Freeman 11]

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

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

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

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

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

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

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

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

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

[celsoft 6 076]

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

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

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

 

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

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

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

[mr. Freeman 11]

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

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

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

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

[celsoft 6 076]

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

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

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