Magento2: Error with a nonexistent theme

Created on 23 Sep 2016 · 9Comments · Source: magento/magento2

Steps to reproduce

Add theme folder with registration.php.

<?php
/**
 * Copyright © 2016 Magento. All rights reserved.
 * See COPYING.txt for license details.
 */

\Magento\Framework\Component\ComponentRegistrar::register(
    \Magento\Framework\Component\ComponentRegistrar::THEME,
    'frontend/Magento/Simplicity',
    __DIR__
);

example_1

Fix mistake: 'frontend/Magento/Simplicity'=>'frontend/Vendor/Simplicity',`

example_2

Go to Magento/sample.

Expected result

Something like 404 not found or delete theme if theme folder is not found (and remove from table).

Actual result

Fatal error: Uncaught Error: Call to a member function getId() on null in /vendor\magento\framework\View\Model\Layout\Merge.php:751

example_3

Additional info

| Q | A |
| --- | --- |
| Magento version | 2.1.0 with sample data |
| PHP version | 7 |
| Operating system | Windows 10 x64 |

Frontend Format is not valid Ready for Work bug report non-issue

Source

springimport

Most helpful comment

It is critical theme removal leaves system in a state that does not results in exceptions. marking as a bug

piotrekkaminski on 24 Oct 2016

👍2

All 9 comments

@springimport thank you for your feedback.
Magento does not support Windows. You can fond Magento technology stack requirements here.
Can you confirm that issue is present as well when Linux is used as operating system?

veloraven on 26 Sep 2016

👎2

@veloraven Currently no.

I think this error is not related with operating system.

springimport on 26 Sep 2016

That is an improvement and in backlog already, tiket MAGETWO-46425. At the moment magento can't remove themes from db if it was removed/moved from file system.

sevos1984 on 3 Oct 2016

I understood, thanks for reply.

springimport on 4 Oct 2016

It is critical theme removal leaves system in a state that does not results in exceptions. marking as a bug

piotrekkaminski on 24 Oct 2016

👍2

adding report MAGETWO-46231 to make sure uninstall command removes the theme from DB

piotrekkaminski on 24 Oct 2016

Relates to #1596

tkacheva on 25 Oct 2016

@springimport, thank you for your report.
This seems to be correct Magento behavior. Please refer to the Community Forums or the Magento Stack Exchange site for advice or general discussion about this.
Otherwise you may submit Pull Request with the suggested changes.

magento-engcom-team on 13 Oct 2017

Hi @magento-engcom-team,

With all due respect, a comment from your colleague @piotrekkaminski (which can be seen above) makes it pretty clear that this is, in fact, a _bug_.
Could you please kindly explain\elaborate, what's changed since then and why? Thanks in advance.