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

Base64 для SafeBlock и другие правила #16

Closed
Tigla opened this issue Sep 3, 2024 · 4 comments · Fixed by #19
Closed

Base64 для SafeBlock и другие правила #16

Tigla opened this issue Sep 3, 2024 · 4 comments · Fixed by #19
Assignees
Labels
bug Something isn't working

Comments

@Tigla
Copy link

Tigla commented Sep 3, 2024

Версия: 0.5.0

Описание ошибки

Не совсем верно работает логика SafeBlock.
В моем случаем к закодированному base64 было применено правило Number\DimensionSup, в результате чего получаю такое:

Screenshot 2024-09-03 at 21 13 10

Как я понимаю было совпадение m8, в результате получаю замену на m<sup>8</sup> и вследствие чего base64 становится не валидным и обратно уже не может вернуть html код.

У себя как временное решение отключил правила для чисел:

$typografed->disableRule('Number\*');

Видимо использование base64 не очень хорошее решение, или же в правилах сделать доп проверку что бы строка заменяемая не являлась base64.

@Tigla Tigla added the bug Something isn't working label Sep 3, 2024
@akhx akhx linked a pull request Sep 24, 2024 that will close this issue
@akhx akhx self-assigned this Sep 24, 2024
@akhx akhx closed this as completed in #19 Sep 25, 2024
@akhx
Copy link
Owner

akhx commented Sep 25, 2024

Добрый день.
Переработал SafeBlock, убрал вообще base64, ибо с ним было много проблем, теперь блоки сохраняются в памяти, проблем должно стать меньше.

@Tigla
Copy link
Author

Tigla commented Sep 25, 2024

Спасибо, как будет доступна новая версия в composer - проверю.

@Tigla
Copy link
Author

Tigla commented Sep 27, 2024

@akhx до сих пор не обновилась инфомарция о новой версии на packagist, v0.5.0 последняя версия числиться.

@akhx
Copy link
Owner

akhx commented Sep 27, 2024

@Tigla вручную обновил, появился тег, проверю что с авто обновлением

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants