Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug in wcms/wex/core/classes/text/Replace.php #4

Open
cryptoprof opened this issue Jul 27, 2019 · 1 comment
Open

Bug in wcms/wex/core/classes/text/Replace.php #4

cryptoprof opened this issue Jul 27, 2019 · 1 comment

Comments

@cryptoprof
Copy link

cryptoprof commented Jul 27, 2019

Буду писать на Русском.
Баг в версии 0.3.2
Если в комментариях и в заголовке одинаковые значения текста, то заменится текст в комментарии, но не в заголовке. Если в title есть такое же текст как в заголовке, заменит текст в заголовке см. пример файла в public в моем форке(https://github.com/cryptoprof/wcms/tree/feature/securityFix), можно скопировать в Ваш вариант реализации. Некорректно отрабатывает функция
str_replace_nth
Она проходит по всему тексту, в т.ч. по комментариям. По идее нужно будет добавить индексы для html элементов и по ним обновлять. Если будет время-присоединюсь и попробую сам доработать
И да, спасибо за труды по CMS)

@vedees
Copy link
Owner

vedees commented Aug 5, 2019

Как только добавлю отпишусь. Или если у тебя будет время доработать - присылай) спасибо)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants