Skip to content
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

Update wrong layout update xml handle installed in CMS Home Page by default #11863

Conversation

adrian-martinez-interactiv4
Copy link
Contributor

@adrian-martinez-interactiv4 adrian-martinez-interactiv4 commented Oct 30, 2017

Preinstalled xml layout update handle in Home Page is invalid, element action is not expected anymore under referenceContainer node:
captura de pantalla 2017-10-30 a las 1 02 34

captura de pantalla 2017-10-30 a las 1 03 01

Exception handling for this validation will be done when changes from PR #11857 are applied.

Description

This legacy code shows a demo of how can be removed right.reports.product.viewed and right.reports.product.compared from right column, but it is outdated; the instructions are not valid and referenced blocks have changed.

This PR suggest changing that preinstalled layout update handle, adapting reference to compared block and removing the viewed block reference, since this last one is now handled via widget and is not at the right column by default, so there is no need to try to remove it:

<referenceContainer name="right">
    <referenceBlock name="catalog.compare.sidebar" remove="true" />
</referenceContainer>

Contribution 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 on Travis CI are green)

Related Pull Requests

#11891

@dmanners dmanners self-assigned this Oct 30, 2017
@dmanners dmanners added this to the October 2017 milestone Oct 30, 2017
@adrian-martinez-interactiv4 adrian-martinez-interactiv4 changed the title Update wrong layout update xml handle installed in CMS Home Page by d… Update wrong layout update xml handle installed in CMS Home Page by default Oct 31, 2017
@okorshenko okorshenko modified the milestones: October 2017, November 2017 Nov 1, 2017
@magento-team magento-team merged commit 4533705 into magento:2.2-develop Nov 6, 2017
magento-team pushed a commit that referenced this pull request Nov 6, 2017
magento-team pushed a commit that referenced this pull request Nov 6, 2017
[EngCom] Public Pull Requests - 2.2-develop
 - MAGETWO-83278: Add validation for number of street lines #12013
 - MAGETWO-83277: [backport 2.2] Magento 2 Store Code validation regex: doesn't support uppercase letters in store code #12011
 - MAGETWO-82991: Product attribute creation page handles Storefront tab visibility wrong #11770
 - MAGETWO-82976: Update wrong layout update xml handle installed in CMS Home Page by d… #11863
 - MAGETWO-82952: Check attribute unique between same fields in magento commerce #11620
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants