Skip to content

Add TYPO3 14 compatibility#119

Open
froemken wants to merge 4 commits intonetworkteam:mainfrom
froemken:typo3_14_compatibility
Open

Add TYPO3 14 compatibility#119
froemken wants to merge 4 commits intonetworkteam:mainfrom
froemken:typo3_14_compatibility

Conversation

@froemken
Copy link

As InformationState class and enum has same name, I prefer removing old TYPO3 compatibility. Else you have to work with namespace aliases and version_compare.

@bmack
Copy link
Contributor

bmack commented Jan 14, 2026

This change makes it actually v13+v14 compatibility. Attached is a version that keeps compat with all existing versions.

networkteam_v14.patch

Copy link

@sebastian-iffland sebastian-iffland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍💯

);
$isActive
? (enum_exists(InformationStatusEnum::class) ? InformationStatusEnum::OK : InformationStatus::STATUS_OK)
: (enum_exists(InformationStatusEnum::class) ? InformationStatusEnum::ERROR : InformationStatus::STATUS_ERROR) );

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

White-space overload 😅

Suggested change
: (enum_exists(InformationStatusEnum::class) ? InformationStatusEnum::ERROR : InformationStatus::STATUS_ERROR) );
: (enum_exists(InformationStatusEnum::class) ? InformationStatusEnum::ERROR : InformationStatus::STATUS_ERROR));

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants