ntrtv 56 Опубликовано: 8 мая 2019 Рассказать Опубликовано: 8 мая 2019 Если новость в настоящий момент редактируется, и кто-то в админке пытается ее открыть, хорошо бы выходило предупреждение, что она открыта уже другим редактором (корреспондентом), так как получается что корреспондент написал материал, редактор редактирует, и тот вдруг хочет добавить 1 слово, и в итоге редакторские правки не сохраняются. 3 Цитата Ссылка на сообщение Поделиться на других сайтах
dimitron 34 Опубликовано: 8 мая 2019 Рассказать Опубликовано: 8 мая 2019 В 30.04.2019 в 22:09, YuriBtr сказал: Если определять новость по alt_name может возникнуть ситуация, когда разные новости с одинаковыми заголовками перенаправляются на новость, созданную первой из них. У меня такая проблема возникает с типом ЧПУ №3. Если в течение суток опубликовать например утром и вечером новость с заголовком "Требуются работники". То вечерняя новость не будет открываться вовсе, а будет идти редирект на утреннюю. И хотя редакция у нас маленькая и все проинструктированы, но все равно несколько раз в месяц случаются такие дубликаты ЧПУ URL. это можно решить, когда мы создаем или редактируем пост мы должны проверить в БД есть ли новость с таким alt_name. Алгоритм решения прост, сколько нашло записей (получаем количество записей и итерируем $count++). И просто к alt_name добавляем номер. И получим name_news.html, если есть name_news добавляем номер. И получиться name_news_01.html. И таким образом избавляемся от дублей. Цитата Ссылка на сообщение Поделиться на других сайтах
alukardua 27 Опубликовано: 8 мая 2019 Рассказать Опубликовано: 8 мая 2019 (изменено) 3 часа назад, dimitron сказал: это можно решить, когда мы создаем или редактируем пост мы должны проверить в БД есть ли новость с таким alt_name. Алгоритм решения прост, сколько нашло записей (получаем количество записей и итерируем $count++). И просто к alt_name добавляем номер. И получим name_news.html, если есть name_news добавляем номер. И получиться name_news_01.html. И таким образом избавляемся от дублей. не рациональное решение, так как будет дополнительные запросы к базе, что вызовет большую нагрузку оно еще куда не шло когда новостей 100-1000, а если 10000, то представь сколько времени цикл потратит Изменено 8 мая 2019 пользователем alukardua 1 Цитата Ссылка на сообщение Поделиться на других сайтах
Gameer 311 Опубликовано: 8 мая 2019 Рассказать Опубликовано: 8 мая 2019 (изменено) 1 час назад, alukardua сказал: не рациональное решение, так как будет дополнительные запросы к базе, что вызовет большую нагрузку оно еще куда не шло когда новостей 100-1000, а если 10000, то представь сколько времени цикл потратит ... $count = $db->super_query("SELECT COUNT(*) as count FROM " . PREFIX . "_post WHERE alt_name='{$alt_name}'")['count']; $count = $count > 0 ? '_' . ($count+1) : ''; ОЧЕНЬ БОЛЬШАЯ нагрузка. Изменено 8 мая 2019 пользователем Gameer Цитата Ссылка на сообщение Поделиться на других сайтах
YuriBtr 58 Опубликовано: 8 мая 2019 Рассказать Опубликовано: 8 мая 2019 5 часов назад, dimitron сказал: это можно решить, когда мы создаем или редактируем пост мы должны проверить в БД есть ли новость с таким alt_name. Алгоритм решения прост, сколько нашло записей (получаем количество записей и итерируем $count++). И просто к alt_name добавляем номер. И получим name_news.html, если есть name_news добавляем номер. И получиться name_news_01.html. И таким образом избавляемся от дублей. Этот метод ненадежный. Нельзя исключать того, что человек мог руками вбить иной порядковый номер в поле ЧПУ URL в полном редактировании и тем самым сбить нумерацию. В моем случае мы выбираем все что подходит по маске, и в этом результате ищем свободный вариант с добавлением цифры. Его конечно тоже надо оптимизировать, так как чем больше новостей с одним заголовком, тем дольше будет идти проверка, но это уже что-то. Такую же проверку "на лету" надо сделать и при изменении в поле ЧПУ URL в полном редактировании. Пусть движок говорит, что такой URL уже занят, рекомендуем добавить символы справа. Цитата Ссылка на сообщение Поделиться на других сайтах
webair 178 Опубликовано: 8 мая 2019 Рассказать Опубликовано: 8 мая 2019 2 часа назад, YuriBtr сказал: Пусть движок говорит, что такой URL уже занят В других CMS именно так. Цитата Ссылка на сообщение Поделиться на других сайтах
MSK 289 Опубликовано: 9 мая 2019 Рассказать Опубликовано: 9 мая 2019 В 08.05.2019 в 13:40, ntrtv сказал: Если новость в настоящий момент редактируется, и кто-то в админке пытается ее открыть, хорошо бы выходило предупреждение, что она открыта уже другим редактором (корреспондентом), так как получается что корреспондент написал материал, редактор редактирует, и тот вдруг хочет добавить 1 слово, и в итоге редакторские правки не сохраняются. Для этого необходима система сохранения версий новости, тогда можно будет и историю изменений контролировать, и кто что поменял, и предупреждать при сохранении, что версия была изменена автором во время проверки... 4 Цитата Ссылка на сообщение Поделиться на других сайтах
Gameer 311 Опубликовано: 10 мая 2019 Рассказать Опубликовано: 10 мая 2019 @celsoft было б хорошо если при обновлении плагина не участвовало выполнение запросов с ячейки при его установке. Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 6 094 Опубликовано: 10 мая 2019 Рассказать Опубликовано: 10 мая 2019 Автор 23 минуты назад, Gameer сказал: было б хорошо если при обновлении плагина не участвовало выполнение запросов с ячейки при его установке. Они и не выполняются при обновлении. Для обновления своя ячейка запросов. Цитата Ссылка на сообщение Поделиться на других сайтах
dimitron 34 Опубликовано: 12 мая 2019 Рассказать Опубликовано: 12 мая 2019 В 08.05.2019 в 20:51, alukardua сказал: не рациональное решение, так как будет дополнительные запросы к базе, что вызовет большую нагрузку оно еще куда не шло когда новостей 100-1000, а если 10000, то представь сколько времени цикл потратит нам главное в БД записать alt_name (редактирование и добавление постов это не столь частая процедура), Это же не новость показать 1000 посетителя одновременно. В основном сайт наполняют администраторы и журналисты. Да если у вас конечна сайт на котором пользователи новости добавляют то да нагрузка будет. Для обычного информационного сайта это самый оптимальный вариант. celsoft Обратите внимание пожалуйста может стоит добавить 4 вид ЧПУ. Тип 4 - ссылки на полную новость будут иметь вид http://site.ru/категория/подкатегория/имя новости.html Мы выше прям дискуссию целую подняли по этому поводу . Да это решается простомы дополнением к коду, но хотелось бы что бы это уже было вшито по дефолту. Цитата Ссылка на сообщение Поделиться на других сайтах
alukardua 27 Опубликовано: 12 мая 2019 Рассказать Опубликовано: 12 мая 2019 1 час назад, dimitron сказал: Тип 4 - ссылки на полную новость будут иметь вид http://site.ru/категория/подкатегория/имя новости.html так это 2й тип Тип 2 - ссылки на полную новость будут иметь вид http://site.ru/категория/подкатегория/id-имя новости.html а id принципе можно убрать в гугле полно хаков первые поиски сразу привели на этот же форум ))) Цитата Ссылка на сообщение Поделиться на других сайтах
YuriBtr 58 Опубликовано: 12 мая 2019 Рассказать Опубликовано: 12 мая 2019 2 часа назад, dimitron сказал: Обратите внимание пожалуйста может стоит добавить 4 вид ЧПУ. Тип 4 - ссылки на полную новость будут иметь вид http://site.ru/категория/подкатегория/имя новости.html ИМХО тогда надо выключать мультикатегории. Иначе у вас будет куча дублей, что отразится на SEO. Или каким то образом указывать главную категорию по которой резолвить новость. Цитата Ссылка на сообщение Поделиться на других сайтах
olifus 13 Опубликовано: 13 мая 2019 Рассказать Опубликовано: 13 мая 2019 Простой вопрос - чем мешает ID в url? Вы думаете пользователи обращают на него внимание (на смартфонах, если чпу с категорией или длинный домен, так id вообще не видно в адресной строке)? Влияние на СЕО - тоже чушь - посмотрите выдачу, в топах сидят сайты со всевозможными видами url. 2 Цитата Ссылка на сообщение Поделиться на других сайтах
Captain 625 Опубликовано: 13 мая 2019 Рассказать Опубликовано: 13 мая 2019 2 часа назад, olifus сказал: Простой вопрос - чем мешает ID в url? Вы думаете пользователи обращают на него внимание (на смартфонах, если чпу с категорией или длинный домен, так id вообще не видно в адресной строке)? Влияние на СЕО - тоже чушь - посмотрите выдачу, в топах сидят сайты со всевозможными видами url. Люди просто читают "слышал звон, да не знаю где он", один напечатал, другой переписал, третий у другого переписал своими словами и т.д., в итоге псевдоинфа столетней давности, которой следуют и думают, что это правильно, без всякого анализа своего сайта и перестройки под постоянно меняющиеся реалии, ведь такие статьи в топе ПС. 1 Цитата Ссылка на сообщение Поделиться на других сайтах
alex32 942 Опубликовано: 13 мая 2019 Рассказать Опубликовано: 13 мая 2019 3 часа назад, olifus сказал: Простой вопрос - чем мешает ID в url? Вы думаете пользователи обращают на него внимание (на смартфонах, если чпу с категорией или длинный домен, так id вообще не видно в адресной строке)? Влияние на СЕО - тоже чушь - посмотрите выдачу, в топах сидят сайты со всевозможными видами url. Они до сих пор считают, что h1 должен быть один на странице Цитата Ссылка на сообщение Поделиться на других сайтах
MSK 289 Опубликовано: 15 мая 2019 Рассказать Опубликовано: 15 мая 2019 Добавьте поддержку комментирования {* ... *} в рекламные материалы. Это позволит временно скрывать части кода. Цитата Ссылка на сообщение Поделиться на других сайтах
Sh1ning 23 Опубликовано: 18 мая 2019 Рассказать Опубликовано: 18 мая 2019 Поработать над поиском: 1. Поиск с учетом морфологии и исправления ошибок. 2. Статистика поисковых запросов 3. ЧПУ для поиска (с возможностью создания посадочных SEO страниц с результатами) 4. Отображение списка категорий, к которым принадлежат данные новости. 5. Вывод категорий в результатах поиска, если их название совпадает с поисковым запросом. 6. Перенаправление поисковых запросов на нужные страницы сайта. Возможность гибкой настройки вывода новостей в админпанели: 1. Задание столбцов для вывода (например изображение новости, название, дата добавления) - с возможностью сортировки. 2. Массовая обработка новостей + возможность задавать условия для cron, например проставить всем новостям у которых в наименование содержится "Значение1" дополнительное поле1 = Значение1 с возможностью создания профилей обработки. Нормальный полноценный маркетплейс с модулями DLE, вот честное слово - бегать по десяткам сайтов разработчиков и искать модули - то еще веселье. Хорошо, когда все находится в одном месте. Начните хотя бы с бесплатных модулей. 1 3 Цитата Ссылка на сообщение Поделиться на других сайтах
midav 4 Опубликовано: 19 мая 2019 Рассказать Опубликовано: 19 мая 2019 Разработчики реализуйте хотя бы к 14 версии чтобы весь движок был на SMARTY шаблонизаторе я дума это не проблема для вас такое сделать. Очень много проблем уйдет таких как если надо написать свой мини код в шаблоне либо использовать такие конструкции как (цыкли, условия). Бо вот примеру я хочу к примеру в краткой новтси проверить дату и добавить возможность добавить плашку что типа публикация нова и через 3 дня она ищизает. То мне пришло писать плагин которые это делать, А это можно было реализовать с помощью УСЛОВИЙ что предоставляет SMARTY. Таких примеров куча что если внедрить SMARTY движок выйдет на новый уровень и можно делеать очень крутые проекты намного быстрее и не не нужно будет писать так много Плагинов. Даже могли к 14 версии сделать эту версию для более продвинутых и оставить такую упрощеную версию. 2 Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 6 094 Опубликовано: 20 мая 2019 Рассказать Опубликовано: 20 мая 2019 Автор 11 часов назад, midav сказал: Разработчики реализуйте хотя бы к 14 версии чтобы весь движок был на SMARTY шаблонизаторе я дума это не проблема для вас такое сделать. Этого не будет никогда. Могу сказать совершенно точно. Потому как во первых многие люди любят DLE именно за простоту его шаблонизатора, во вторых это нарушит совместимость со всеми существующими шаблонами. Поэтому можете абсолютно точно этого не ждать. 4 4 Цитата Ссылка на сообщение Поделиться на других сайтах
Spyhog 65 Опубликовано: 20 мая 2019 Рассказать Опубликовано: 20 мая 2019 Предложения по поводу картинок: 1. Сделать вывод информации о картинке в шаблоне - размер, формат, вес. 2. Сделать возможность выводить подпись к картинке. 3. Если не сильно "утяжелит" движок, то можно добавить [xfvalue_image_url_X image="Nr"] для вывода оригинала определённой картинки из галереи. 4. Возможность загрузки картинок на поддомен для экономии места основного сайта. 1 Цитата Ссылка на сообщение Поделиться на других сайтах
YuriBtr 58 Опубликовано: 20 мая 2019 Рассказать Опубликовано: 20 мая 2019 4 часа назад, celsoft сказал: Этого не будет никогда. Могу сказать совершенно точно. Потому как во первых многие люди любят DLE именно за простоту его шаблонизатора, во вторых это нарушит совместимость со всеми существующими шаблонами. Поэтому можете абсолютно точно этого не ждать. Очень жаль, что у вас это вызывает такое категорическое отторжение. На самом деле в шаблонах DLE очень не хватает возможностей PHP (как это сделано в Smarty или Fenom), если вы не хотите внедрять шаблонизаторы (ведь можно просто создать новый тип шаблонов и обеспечить обратную совместимость), то сделайте хотя бы обработку некоторых PHP инструкций. Цитата Ссылка на сообщение Поделиться на других сайтах
Captain 625 Опубликовано: 20 мая 2019 Рассказать Опубликовано: 20 мая 2019 4 часа назад, YuriBtr сказал: Очень жаль, что у вас это вызывает такое категорическое отторжение. На самом деле в шаблонах DLE очень не хватает возможностей PHP (как это сделано в Smarty или Fenom), если вы не хотите внедрять шаблонизаторы (ведь можно просто создать новый тип шаблонов и обеспечить обратную совместимость), то сделайте хотя бы обработку некоторых PHP инструкций. Так напиши, если сможешь, а мы потестим. Цитата Ссылка на сообщение Поделиться на других сайтах
YuriBtr 58 Опубликовано: 20 мая 2019 Рассказать Опубликовано: 20 мая 2019 2 часа назад, Captain сказал: Так напиши, если сможешь, а мы потестим. Написать что??? Fenom и Smarty уже готовый продукт. Надо их подключать при обработке файлов шаблона с расширением например .tpl2 Подключение шаблонизаторов делается на уровне ядра. Если ядро не позволяет легко подключить шаблонизатор - видимо проблема в архитектуре движка. Отсюда еще одно пожелание, про которое я писал ранее - перевести движок DLE на MVC полностью. Цитата Ссылка на сообщение Поделиться на других сайтах
alukardua 27 Опубликовано: 20 мая 2019 Рассказать Опубликовано: 20 мая 2019 2 часа назад, YuriBtr сказал: Написать что??? Fenom и Smarty уже готовый продукт. Надо их подключать при обработке файлов шаблона с расширением например .tpl2 Подключение шаблонизаторов делается на уровне ядра. Если ядро не позволяет легко подключить шаблонизатор - видимо проблема в архитектуре движка. Отсюда еще одно пожелание, про которое я писал ранее - перевести движок DLE на MVC полностью. если бы ты удостоился бы порыться в файлах движка, то знал бы что DLE использует шаблонизатор Fenom Цитата Ссылка на сообщение Поделиться на других сайтах
Gameer 311 Опубликовано: 20 мая 2019 Рассказать Опубликовано: 20 мая 2019 2 часа назад, alukardua сказал: если бы ты удостоился бы порыться в файлах движка, то знал бы что DLE использует шаблонизатор Fenom Какой же ты тупой, DLE не использует Fenom. Пожалуйста, если ты такой все знающий и разбирающийся лучше других покажи мне файлы от шаблонизатора Fenom в DLE. Вот ссылка на репозиторий шаблонизатора Femom - https://github.com/fenom-template/fenom DLE же имеет один файл который причастен ко всему шаблонизатору - /engine/classes/templates.class.php Где связь между этими двумя ничем не связанными вещами? Перестань писать везде хрень думая что ты в ней разбираешься. 1 1 Цитата Ссылка на сообщение Поделиться на других сайтах
Рекомендованные сообщения
Присоединяйтесь к обсуждению
Вы можете опубликовать сообщение сейчас, а зарегистрироваться позже. Если у вас есть аккаунт, войдите в него для написания от своего имени.