Skip to content

update V5-dev - #106

Open
DavidBruchmann wants to merge 29 commits into
Kephson:mainfrom
DavidBruchmann:v5-dev
Open

update V5-dev#106
DavidBruchmann wants to merge 29 commits into
Kephson:mainfrom
DavidBruchmann:v5-dev

Conversation

@DavidBruchmann

Copy link
Copy Markdown
Collaborator
  • hydrate data for new Record-API, to assure that other extensions like powermail with PreviewRenderer get expected and expectable data.
  • Update README.
  • update deprecated code.

* Docker didn't provide additional value beyond ddev.
* Adjust version matrix for PHP and DBMS'.
* add JavaScript tests in previously existing ci.yml.
* Remove Docker related workflow file.
github workflows require at least node.js v24
* rename
* add CE with irre content for testing
…nderer
Fixes a crash where extensions (like EXT:powermail) using strict
inline PHPDoc type-hinting or native type checks (e.g., expecting
LazyRecordCollection) fail with a TypeError when rendered inside
shortcuts or containers.
The issue occurred because ShortcutPreviewRenderer manually fetched
referenced records and instantiated them using createFromDatabaseRow().
This bypassed TYPO3 v14's native RecordFieldTransformer, causing
complex TCA types like FlexForms or relations to be passed down as
flat, common PHP arrays instead of fully-cast Core object structures.
Changes:
- Replace shallow createFromDatabaseRow() with createResolvedRecordFromDatabaseRow()
- Guarantee full object hydration across the shortcut rendering pipeline
fixKephson#94, closein2code-pro/powermail#169
- resolve deprecated Issues
- leave comment beside false positives, which can be seen directly in the installTool, so that the items never have to be checked several times.
…ository
... to avoid deprecation notice in TYPO3 and CI pipeline
The code would evaluate always to false.
@DavidBruchmann

Copy link
Copy Markdown
CollaboratorAuthor

@Kephson this current state can be a new v5 Release, would be great if you could care about it!

Sign up for freeto 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.

1 participant

@DavidBruchmann