Needs review
Project:
Entity Print
Version:
3.0.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
28 Sep 2026 at 06:41 UTC
Updated:
1 Oct 2026 at 18:23 UTC
Jump to comment: Most recent
Comments
Comment #3
vladimirausComment #5
ayoub.elmansouri commentedI fixed a few issues that were causing PHPUnit test failures.
@VladimirAus
All changes i made are non-breaking and do not impact your work, except for one intentional revert in entity_print.services.yml — restored @theme_handler.
The @theme_handler argument was restored as the first argument of entity_print.asset_collector because AssetCollector::__construct() still expects ThemeHandlerInterface as its first parameter, and getCssLibraries() still actively uses it to retrieve the default theme:
$theme = $this->themeHandler->getTheme($this->themeHandler->getDefault());
Without it, injecting @extension.list.theme as the first argument causes a TypeError at runtime.