Bug #104959
closed
typo3/cms-sys-note depends on typo3/cms-dashboard
100%
Description
The main branch on aac4f7be152b043b1a85b333b6fa03e2a67eabc0 fails with:
Uncaught TYPO3 Exception Invalid service "dashboard.widget.sys_notes.all": class "TYPO3\CMS\Dashboard\Widgets\WidgetInterface" not found while loading "TYPO3\CMS\SysNote\Widgets\PagesWithInternalNote".
thrown in file /var/www/html/vendor/symfony/dependency-injection/Compiler/AbstractRecursivePass.php
in line 177
The class TYPO3\CMS\SysNote\Widgets\PagesWithInternalNote uses/imports TYPO3\CMS\Dashboard\*.
Please add the dashboard to sys-note as dependncy.
Related: #104844
Updated by Rafael Kähm 4 months ago
- Related to Feature #104844: Dashboard - sys_note - Add widgets for listing all the sys_notes inside the TYPO3 system added
Updated by Garvin Hicking 4 months ago
- Category changed from composer to Dashboard
Updated by Garvin Hicking 4 months ago
- Status changed from New to Accepted
Thanks, this is an important catch! I believe adding a hard dependency here isn't the solution; we'd possible better register the dashboard widgets with a dependency check, like EXT:seo does in it's Configuration/Services.php file.
Updated by Gerrit Code Review 4 months ago
- Status changed from Accepted to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/86089
Updated by Gerrit Code Review 4 months ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/86089
Updated by Gerrit Code Review 4 months ago
Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/86089
Updated by Garvin Hicking 4 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset c1716095e044d2ea65722e0c9bd5c0a139a007a1.
Updated by