Skip to content

[Backport to 2.3-develop] Fix #10682: Meta description and keywords transform to html entities #12955

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

Merged
merged 1 commit into from
Jan 3, 2018
Merged

[Backport to 2.3-develop] Fix #10682: Meta description and keywords transform to html entities #12955

merged 1 commit into from
Jan 3, 2018

Conversation

dverkade
Copy link
Member

@dverkade dverkade commented Jan 2, 2018

Description

Changed the htmlentities function to the htmlspecialchars function. It's not necessary to do HTML encoding on all characters as described in #10682. Since the content is used within HTML tags, we still need to encode special HTML chars in order to make sure that chars like "'& etc, will not break the page.

Backport

  1. Backport for PR: Fix #10682: Meta description and keywords transform to html entities #11829

Fixed Issues

  1. Meta description and keywords transform to html entities for non latin/cyrilic characters in category and product pages #10682

Changed the htmlentities function to the htmlspecialchars function. It's not necessary to do HTML encoding on all characters as described in #10682. Since the content is used within HTML tags, we still need to encode special HTML chars in order to make sure that chars like "'& etc, will not break the page.
@magento-engcom-team magento-engcom-team added bugfix Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release SQUASHTOBERFEST labels Jan 2, 2018
@dmanners dmanners self-assigned this Jan 3, 2018
@dmanners dmanners added this to the January 2018 milestone Jan 3, 2018
@magento-team magento-team merged commit c49c52a into magento:2.3-develop Jan 3, 2018
magento-team pushed a commit that referenced this pull request Jan 3, 2018
@magento-engcom-team magento-engcom-team added the Fixed in 2.3.x The issue has been fixed in 2.3 release line label Jan 3, 2018
@dverkade dverkade deleted the 2.3-Meta-description-and-keywords-transform-to-html-entities branch June 22, 2018 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Community Insider: Cream community-insider-contribution Fixed in 2.3.x The issue has been fixed in 2.3 release line Progress: accept Release Line: 2.3 Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants