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
Hotfix custom sql filter inserttags #1490
Hotfix custom sql filter inserttags #1490
Conversation
works with strung together as well as nested insert tags
rework identify inserttags in customSql Query
secure inserttags have more then two colons
@discordier workss fine for #880 (comment) a) and workaround) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change the methods to private
src/Filter/Setting/CustomSql.php
Outdated
* | ||
* @return string | ||
* | ||
* @internal Only to be used via parseRequestVars(). | ||
*/ | ||
public function convertParameter($arrMatch) | ||
public function convertParameter($strMatch) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be made private now and the @internal
can be removed. This method is now being called from $this->checkTag()
.
src/Filter/Setting/CustomSql.php
Outdated
* | ||
* @return string | ||
* | ||
* @internal Only to be used internal as callback from parseSecureInsertTags(). | ||
*/ | ||
public function parseAndAddSecureInsertTagAsParameter($arrMatch) | ||
public function parseAndAddSecureInsertTagAsParameter($strMatch) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be made private now and the @internal
can be removed. This method is now being called from $this->checkTag()
.
Description
fixes the problem with strung together and nested inserttags
this should fix #880 (comment)
Checklist
@authors
in touched PHP files