Skip to content

Conversation

@bubasuma
Copy link
Contributor

Description (*)

PageBuilder text content is broken after saving it with Magento variable directives within html attributes

Bug

  • MC-32883 Page builder issue with secure variable

Manual testing scenarios (*)

  1. Install Magento with PageBuilder
  2. Disable WYSIWYG (Stores > Configuration > General > Content Management > WYSIWYG Options). Set to any of these values: Disabled by Default, Disabled Completely.
  3. Go to Content > Elements > Pages and edit "Home Page"
  4. Add TEXT element to the page content and fill the content with the following html code:
<p><a title="Contact Us" href="{{config path="web/secure/base_url"}}contact-us">Contact Us</a></p>
  1. Save changes

ACTUAL RESULTS:

<p><a title="Contact Us" href="{{config path=" web="" secure="" base_url"}}contact-us"="">Contact Us</a></p>

EXPECTED RESULTS:

<p><a title="Contact Us" href="{{config path='web/secure/base_url'}}contact-us">Contact Us</a></p>

Questions or comments

Checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

- Fix page builder text content is broken after saving it with magento variable directives within html attributes
@bubasuma bubasuma force-pushed the magento-mpi-MC-32883 branch from 9c31c44 to ab21ac8 Compare April 28, 2020 20:02
@omiroshnichenko omiroshnichenko self-assigned this Apr 28, 2020
@omiroshnichenko
Copy link
Contributor

@magento run Functional Tests CE

@omiroshnichenko
Copy link
Contributor

@magento run all tests

@omiroshnichenko omiroshnichenko merged commit 4947b61 into develop May 1, 2020
@omiroshnichenko omiroshnichenko deleted the magento-mpi-MC-32883 branch July 20, 2020 17:58
magento-devops-reposync-svc pushed a commit that referenced this pull request Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants