Jump to content
Sign in to follow this  
Gameer

Быстрый поиск не находит новости с кавычками

Recommended Posts

Если в названии новости есть одинарные или двойные кавычки быстрый поиск эту новость не найдет, НО, полный поиск - находит.

7feIiXSlQ4a0mUgXTcbQGg.png

Проблема на DLE 12.0 UTF-8 как Демо так и лицензии.

Share this post


Link to post
Share on other sites

Принято. Будет исправлено в следующей версии скрипта.

Share this post


Link to post
Share on other sites
В 10.11.2017 в 15:21, celsoft сказал:

Принято. Будет исправлено в следующей версии скрипта.

Так же проблема с символами. В общем, в /engine/ajax/search.php идет преобразование в html сущности а в /engine/modules/search.php идет экранирование и удаление символов 

"\x60", "\t", "\n", "\r", ",", ";", ":", "[", "]", "{", "}", "=", "*", "^", "%", "$", "<", ">", "+", "-"

которые удаляются в функции strip_data, которая почему-то именно в файле /engine/modules/search.php а не в /engine/modules/functions.php что было-бы более логичней. В этих файлах /ajax/ и /modules/ разный подход к обработке поискового запроса - что странно.

3M9x5Qz5TlqkvkYRagxLDw.png

Edited by Gameer

Share this post


Link to post
Share on other sites

@celsoft проблема в новой версии осталась. И стало еще хуже, теперь быстрый поиск не находит новости с тире и т.п

 

Share this post


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

проблема в новой версии осталась. И стало еще хуже, теперь быстрый поиск не находит новости с тире и т.п

Никаких проблем, описанных в данной теме и постах, на новой версии не зафиксировано. Все ищется корректно. В чем именно у вас проблема, и по каким запросам не ищет?

Share this post


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

Никаких проблем, описанных в данной теме и постах, на новой версии не зафиксировано. Все ищется корректно. В чем именно у вас проблема, и по каким запросам не ищет?

Прошу прощения все работает. Тему можно закрыть.

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  

×
×
  • Create New...