We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Not sure if thats a problem for normal use cases, but using HTML escaping or h() in CakePHP you get
Some "<b>demo</b>" string
transformed into
Some "<b>demo</b>" string
with >/</" escaped
>
<
"
But with this BBCode parser the " remain it seems:
Some "<b>demo</b>" string
Depending on where those chars appear it could break some layouts maybe. But not sure.
I tried to first escape h($text) before adding it into the converter, but then the tags needing " characters to work wont parse anymore.
The text was updated successfully, but these errors were encountered:
The issue can be seen here: https://github.com/dereuromark/cakephp-markup/blob/master/tests/TestCase/Bbcode/DecodaBbcodeTest.php#L42
Sorry, something went wrong.
There is one test case for that so it is expected for now.
https://github.com/milesj/decoda/blob/e84411a/tests/Decoda/DecodaTest.php#L247
I wonder if this could break some HTML or have other side effects.
It only make issue on tag attributes value.
Except I miss something.
No branches or pull requests
Not sure if thats a problem for normal use cases, but using HTML escaping
or h() in CakePHP
you get
transformed into
with
>
/<
/"
escapedBut with this BBCode parser the " remain it seems:
Depending on where those chars appear it could break some layouts maybe. But not sure.
I tried to first escape h($text) before adding it into the converter, but then the tags needing " characters to work wont parse anymore.
The text was updated successfully, but these errors were encountered: