Dynamic loading of themes / Enable & add themes with PHP

How may I add a theme to the theme-list with PHP?
I like to dynamically add the theme by an event listener.

I can set the active theme by PHP:

$activeTheme = $this->container->get('liip_theme.active_theme');
$activeTheme->setName('blue');

=> The active theme "blue" must be in the themes list ().

How to add any theme name to the "theme list" ?

The solution is:

$activeTheme->setThemes(array('blue'));

Maybe this should be added to the documentation.

I think it would be good if anybody could add this function to ActiveTheme class:

public function addTheme($theme) {
    if(is_array($theme)) {
        $this->themes = array_merge($theme, $this->themes);
    } else {
    array_push($this->themes, $theme);
    }
}

or:

public function addTheme($theme) {
    array_push($this->themes, $theme);
}

public function addThemes(array $themes) {
    $this->themes = array_merge($themes, $this->themes);
}

[oleg-andreyev]

Why do you want do that? To me theme list should be defined permanently in configs, otherwise it's something that you don't control.

We are using a dynamic system to upload, enable and disable themes. All settings should be saved in the database. Is there any way to disable the "it must be in the array (theme-list)"-feature? It would be great if the system would just try to find the theme based on the name (directory name = theme name).

[lsmith77]

I do agree that dynamic loading of themes is a legitimate use case.
Can you open a PR with the code, doc and test changes to add this?

[oleg-andreyev]

@JHGitty now it makes sense, I'm also +1 for idea.

Assetic problems in combination with dynamically add/enable themes.

An exception has been thrown during the rendering of a template ("Unable to generate a URL for the named route "_assetic_xxxxxxx_x" as such route does not exist.") in (bla)/Resources/themes/blue/layout.html.twig at line xxx.

I tried to dynamically add and set the theme with an event listener for kernel.controller

public function onKernelController(FilterControllerEvent $event) {
    /* @var Controller $controller */
    $controller = $event->getController();

    // ...

    if ($controller[0] instanceof BaseController) {
      // Set theme
      $activeTheme = $this->container->get('liip_theme.active_theme');
      $activeTheme->setThemes(array('blue'));
      $activeTheme->setName('blue');
    }
}

If I add and enable the theme in config.yml this problem seems to be fixed. But I need to dynamically add and enable themes.

This problem appears because I use an eventlistener which will be called after TemplateResourcesPass (https://github.com/liip/LiipThemeBundle/blob/master/DependencyInjection/Compiler/TemplateResourcesPass.php)

Same problem as Issue #25.

How should I set the theme with PHP to avoid this problem?

[lsmith77]

long time since I worked on that part of the code. we "inject" the theme directories into assetic here from my reading:
https://github.com/liip/LiipThemeBundle/blob/master/DependencyInjection/Compiler/TemplateResourcesPass.php#L68

now I am not sure if the assetic API allows for dynamically adding more directories but I doubt that this would be at all useful, since assetic's CLI dump command would then never know about those directories. meaning you would also have to do something to the dump process.

in the end, I would say its easier to just keep assetic resource introduction out of theme templates.

[lsmith77]

ping

I am not working on this. I just created the issue. I hope anybody else will do it.