Перейти к публикации

Ограничения к-ва картинок...


Рекомендованные сообщения

Обратил внимание на такой момент.

При ограничении числа картинок в новостях их количество считается на основе к-ва записей в таблице

$count_images = count(explode( "|||", $row['images'] ));

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

if( file_exists( ROOT_DIR . "/uploads/posts/" . $folder_prefix . $dataimages ) ) {

Если по каким-то причинам на сервере физически будет недоступен файл картинки (сбой при сохранении, удален, не восстановлен из бекапа и т.д.) то возникает ситуация, когда мы видим в интерфейсе меньше картинок, чем считает двиг при проверке лимитов. В итоге нельзя загрузить новые, т.к. лимит посчитан с учетом физически отсутствующих на сервере картинок .

Удалить записи о "потерянных" картинках тоже нельзя, т.к. в интерфейсе они не отображаются и нет синхронизации записей о картинках с их физическим присутствием на сервере.

Ссылка на сообщение
Поделиться на других сайтах

Сразу не обратил внимания, но вот при выводе загруженных файлов к новостям - нет проверки на существование их на сервере и такие "потерянные" можно удалить :)

Изменено пользователем MSK
Ссылка на сообщение
Поделиться на других сайтах
11.01.2021 в 02:02, MSK сказал:

Сразу не обратил внимания, но вот при выводе загруженных файлов к новостям - нет проверки на существование их на сервере и такие "потерянные" можно удалить :)

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

Ссылка на сообщение
Поделиться на других сайтах
8 часов назад, Mr. Bot сказал:

Всё это каждый раз читается из файла

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

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

Второй пост - это продолжение первого, не склеился...

Уже стало привычным, что в теме "прием багов", Клиенты сами с собой общаются и нет реакции от поддержки ;)

Как и нет дорожной карты, которую просят очень давно :(

 

Ссылка на сообщение
Поделиться на других сайтах
11 часов назад, MSK сказал:

Как и нет дорожной карты, которую просят очень давно :(

Вы имеете в виду планы по развитию движка? Если так, то вы правы. Очень хотелось бы заранее знать, чего ожидать. По сути все нововведения основываются только на выборе и желании разработчика. Темы "Что мы хотим видеть в DLE" не более чем генераторы идей. Почему бы не дать пользователям выбирать, что действительно добавлять в новых версиях. Как пример, периодически создавать темы, в которых выкладывать все нововведения, которые возможно внедрить в новых версиях, и пусть пользователи голосуют, что реализовывать в первую очередь.

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

Ссылка на сообщение
Поделиться на других сайтах
9 часов назад, radrigo сказал:

Очень хотелось бы заранее знать, чего ожидать.

И это вероятно приведет еще к большему конфликту и недопониманию.

А так сейчас - все в предвкушении - что же будет нового....

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

Ссылка на сообщение
Поделиться на других сайтах
17.01.2021 в 12:58, MSK сказал:

но в итоге переделали.

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

17.01.2021 в 12:58, MSK сказал:

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

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

 

18.01.2021 в 10:18, kamensk сказал:

А так сейчас - все в предвкушении - что же будет нового....

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

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

Ссылка на сообщение
Поделиться на других сайтах

Присоединяйтесь к обсуждению

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

Гость
Ответить в тему...

×   Вставлено в виде отформатированного текста.   Вставить в виде обычного текста

  Разрешено не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отобразить как ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставить изображения напрямую. Загрузите или вставьте изображения по ссылке.

×
×
  • Создать...