Jump to content
konst222

Режутся кавычки в seopro - что делать?

Recommended Posts

Добрый день. Подскажите пожалуйста как можно поправить seopro, чтобы при поиске название товара не обрезалось после ковычки (не правильно формируется урл, ковычка приобразуется в ", соответственно все остальное идет как другой параметр, а не search). например на демо: http://joxi.ru/xAeb7YEiYZOyPr

Share this post


Link to post
Share on other sites

Сделайте проверку роута и подмену, это самое безопасное чтоб не сломать другое

Share this post


Link to post
Share on other sites

тоже на одном из проектов столкнулся с такой проблемой при включенном SeoPro.

Если в строку поиска ввести текст, содержащий кавычку, то массив GET не соответствует реальности. Например при запросе /search/?search=word1"word2 имеем такой дамп

array(3) {
  ["search"]=>
  string(5) "word1"
  ["quot;word2"]=>
  string(0) ""
  ["route"]=>
  string(14) "product/search"
}

и такую адресную строку search/?search=word1"word2=

При этом в средствах разработчика имеем соответствующий редирект. Его делает сеопро, т.к. при проверке

rawurldecode($url) != rawurldecode($seo)

имеет дело со следующими строками:

search/?search=word1"word2
search/?search=word1"word2=

Т.е. сеопро оперирует не преобразованными спецсимволами, в то время как $this->request->server['REQUEST_URI'] возвращает преобразованные.

решил вопрос заменой строки

return urldecode(http_build_query(array_diff_key($this->request->get, array_flip($exclude))));

на

return html_entity_decode(urldecode(http_build_query(array_diff_key($this->request->get, array_flip($exclude)))), ENT_QUOTES, "UTF-8");

Полет нормальный, сайт успешно работает. Но может есть какие-то подводные камни и у разработчиков есть какие-то другие мнения/решения?

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
You are posting as a guest. If you have an account, please sign in.
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.


  • Recently Browsing   0 members

    No registered users viewing this page.

×

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.