Task #95359: Adapt documentation to showcase conditional service dependencies via ContainerBuilder::hasDefinition() - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Task #95359

closed

Adapt documentation to showcase conditional service dependencies via ContainerBuilder::hasDefinition()

Added by Benjamin Franzke about 3 years ago. Updated about 3 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Benjamin Franzke

Category:

Documentation

Target version:

11 LTS

Start date:

2021-09-25

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Sprint Focus:

Description

The current recommendation to use ExtensionManagementUtility::isLoaded was never officially supported from the technical point of view:
https://docs.typo3.org/c/typo3/cms-dashboard/master/en-us/Configuration/WidgetRegistration.html#services-php-file

Therefore proper stateless container-builder checks via `hasDefinition()` should be encouraged instead.
Also see https://review.typo3.org/c/Packages/TYPO3.CMS/+/71214 for an explanation for alternatives.

Related issues 2 (0 open — 2 closed)

Actions

Copy link

Updated by Benjamin Franzke about 3 years ago

Related to Task #95358: Avoid pushing PackageManager to ExtensionManagementUtility only to circumvent a documentation bug added

Actions

Copy link

Updated by Benjamin Franzke about 3 years ago

Related to Bug #95327: Push PackageManager to ExtensionManagementUtility in bootstrap added

Actions

Copy link

Updated by Benjamin Franzke about 3 years ago

Subject changed from Adapt documentation to showcase conditional service dependencies via ContainerBuilder::hasLoaded() to Adapt documentation to showcase conditional service dependencies via ContainerBuilder::hasDefinition()

Actions

Copy link

Updated by Gerrit Code Review about 3 years ago

Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/71272

Actions

Copy link

Updated by Gerrit Code Review about 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/71272

Actions

Copy link

Updated by Gerrit Code Review about 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/71272

Actions

Copy link