ThemeManager fails when removing a theme

Description

Steps to reproduce

  • Add a theme to your Integrated installation

  • Assign the theme to a website

  • Add a page to the website

  • Remove the theme from your Integrated installation

  • Result: exception on all pages of the channel. In this case the default theme should be used automatically

InvalidArgumentException in ThemeManager.php line 122:
Theme "thompson" not exists.
in ThemeManager.php line 122
at ThemeManager->setActiveTheme('thompson') in WebsiteConfigurationListener.php line 91
at WebsiteConfigurationListener->onKernelRequest(object(GetResponseEvent), 'kernel.request', object(TraceableEventDispatcher))
at call_user_func(array(object(WebsiteConfigurationListener), 'onKernelRequest'), object(GetResponseEvent), 'kernel.request', object(TraceableEventDispatcher)) in WrappedListener.php line 61
at WrappedListener->__invoke(object(GetResponseEvent), 'kernel.request', object(TraceableEventDispatcher))
at call_user_func(object(WrappedListener), object(GetResponseEvent), 'kernel.request', object(TraceableEventDispatcher)) in WrappedListener.php line 61
at WrappedListener->__invoke(object(GetResponseEvent), 'kernel.request', object(ContainerAwareEventDispatcher))

Requirements

  • Removing a theme or removing a base view from a theme shouldn't throw an exception, but should fallback to the default theme, default base view

  • Don't store the full path of a view, so changing a theme should affect all pages automatically

Deployment actions

None

Technical tasks

None

Activity

Pro tip: press M to comment

Marijn Otte 
May 5, 2017 at 12:48 PM

Please keep supporting upgraded 0.6 installations, to avoid the need of migrations/deployment actions.

So please support a full template path in locateTemplate when inserted.

·

API 
May 4, 2017 at 9:33 AM

.Notificatie preview gestuurd naar Integrated Marijn (marijn@e-active.nl)

·

Koen Prins 
May 3, 2017 at 9:11 AM

You'll have merge it yourself

·

API 
April 24, 2017 at 10:39 AM

.Schatting van 4.75 uur is geaccepteerd door Integrated Marijn (in opdracht ingevoerd door Marijn Otte).

·
Fixed

Details

Assignee

Client

Product owner

Original estimate

Sprint

Fix versions

OTM informatie

Created February 5, 2017 at 2:13 PM
Updated November 24, 2017 at 2:38 PM
Resolved May 8, 2017 at 6:46 PM