Page layouts are hard-coded in Magento\Widget\Block\Adminhtml\Widget\Instance\Edit\Chooser\Container

[ericmorand]

Preconditions (*)

Magento 2.3.5

Steps to reproduce (*)

Have a look at the code source of Magento\Widget\Block\Adminhtml\Widget\Instance\Edit\Chooser\Container.

Expected result (*)

The function getPageLayouts() should return the actual list of page layouts declared by the different modules.

Actual result (*)

The function getPageLayouts() only returns the one provided by Magento_Theme, and they are hard-coded in the file:

    /**#@+
     * Frontend page layouts
     */
    const PAGE_LAYOUT_1COLUMN = '1column-center';
    const PAGE_LAYOUT_2COLUMNS_LEFT = '2columns-left';
    const PAGE_LAYOUT_2COLUMNS_RIGHT = '2columns-right';
    const PAGE_LAYOUT_3COLUMNS = '3columns';
    /**#@-*/

   [...]

  /**
     * Retrieve page layouts
     *
     * @return array
     */
    protected function getPageLayouts()
    {
        return [
            self::PAGE_LAYOUT_1COLUMN,
            self::PAGE_LAYOUT_2COLUMNS_LEFT,
            self::PAGE_LAYOUT_2COLUMNS_RIGHT,
            self::PAGE_LAYOUT_3COLUMNS,
        ];
    }

Workaround

I may be possible to override Magento\Widget\Block\Adminhtml\Widget\Instance\Edit\Chooser\Container and to inject an instance of Magento\Framework\View\Model\PageLayout\Config\BuilderInterface to grab the list of available page layouts from there. Untested though.

Thoughts

Basically, it makes any container with Label declared in a non-Magento Theme page layout not selectable from the back-office to inject widgets. But the simple fact that Magento_Widget assumes that some page layouts exist is concerning. This concern is actually confirmed by the fact that one of the four hard-coded page layouts doesn't even exist: 1column-center.

---

Please provide Severity assessment for the Issue as Reporter. This information will help during Confirmation and Issue triage processes.

• Severity: S0 - Affects critical data or functionality and leaves users without workaround.
• Severity: S1 - Affects critical data or functionality and forces users to employ a workaround.
• Severity: S2 - Affects non-critical data or functionality and forces users to employ a workaround.
• Severity: S3 - Affects non-critical data or functionality and does not force users to employ a workaround.
• Severity: S4 - Affects aesthetics, professional look and feel, “quality” or “usability”.

[m2-assistant]

Hi @ericmorand. Thank you for your report.
To help us process this issue please make sure that you provided the following information:

• Summary of the issue
• Information on your environment
• Steps to reproduce
• Expected and actual results

Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:

@magento give me 2.4-develop instance - upcoming 2.4.x release

For more details, please, review the Magento Contributor Assistant documentation.

Please, add a comment to assign the issue: @magento I am working on this

---

• Join Magento Community Engineering Slack and ask your questions in #github channel.

⚠️ According to the Magento Contribution requirements, all issues must go through the Community Contributions Triage process. Community Contributions Triage is a public meeting.

🕙 You can find the schedule on the Magento Community Calendar page.

📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket.

🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel

✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel

[ericmorand]

I can now confirm that overriding the class works perfectly:

<?php


namespace [Vendor]\[Theme]\Block\Adminhtml\Widget\Instance\Edit\Chooser;


class Container extends \Magento\Widget\Block\Adminhtml\Widget\Instance\Edit\Chooser\Container
{
    /** @var \Magento\Framework\View\Model\PageLayout\Config\BuilderInterface */
    protected $_pageLayoutConfigBuilder;

    public function __construct(
        \Magento\Framework\View\Element\Context $context,
        \Magento\Framework\View\Layout\ProcessorFactory $layoutProcessorFactory,
        \Magento\Theme\Model\ResourceModel\Theme\CollectionFactory $themesFactory,
        \Magento\Framework\View\Model\PageLayout\Config\BuilderInterface $pageLayoutConfigBuilder,
        array $data = []
    ) {
        $this->_pageLayoutConfigBuilder = $pageLayoutConfigBuilder;

        parent::__construct($context, $layoutProcessorFactory, $themesFactory, $data);
    }

    protected function getPageLayouts()
    {
        $pageLayoutsConfig = $this->_pageLayoutConfigBuilder->getPageLayoutsConfig();

        return array_keys($pageLayoutsConfig->getPageLayouts());
    }
}

This Container returns the actual list of page layouts instead of the hard-coded and wrong one of the core implementation.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 14 days if no further activity occurs. Is this issue still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? Thank you for your contributions!

[mrtuvn]

This issue still quite important at my point of view
CC: @ihor-sviziev @sidolov

[ihor-sviziev]

@ericmorand @mrtuvn, could you explain the use cases when these values need to be changed/customized?

[mrtuvn]

Should getPageLayouts get all layout instead return fixed value array ? Example i create custom page layout somewhere else in theme Should magento update while widget choose page layout from back office ? Maybe i missed something or magento update value from other place

[ihor-sviziev]

@mrtuvn yeah. Make sense! I think it should be marked with S3 P2.
@sidolov @sivaschenko @gabrieldagama, what do you think?

Hi @ericmorand,
Could you create a pull request with suggested changes from the following comment #30471 (comment) ?