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
Do not decode entities in page names #1179
Conversation
@@ -210,7 +210,7 @@ protected function prepare($objPage) | |||
} | |||
|
|||
// Assign the title and description | |||
$this->Template->title = StringUtil::stripInsertTags($this->replaceInsertTags($objLayout->titleTag)); // see #7097 | |||
$this->Template->title = $this->replaceInsertTags($objLayout->titleTag); |
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.
I wonder if we should strip_tags()
though? What if I'm using the {{br}}
insert tag? Or whatever insert tag that generates HTML?
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.
We did not strip HTML tags before, but I guess it cannot hurt. Added in 31a2454.
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.
The changes have been added in 2.6.3 (contao/core@28f57b7), unfortunately there's no note about why it was added.
I can already tell you there will be issues (like what @Toflar mentioned or the XML feed), but I think changing it is the correct approach and then we need to fix appearing issues in the long run.
I agree – even though the page title is never used in the XML sitemap. |
Fixes #473
Since insert tags are now removed recursively, we do not need the
StringUtil::stripInsertTags()
call in thePageRegular
class anymore.