Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TASK] Provide basic compatibility with TYPO3 v12 #1078

Merged
merged 2 commits into from Feb 11, 2023

Conversation

bmack
Copy link
Contributor

@bmack bmack commented Nov 11, 2022

This change adds basic functionality for TYPO3 v12.

By using TYPO3 v12 the main issue is providing
compatibility with symfony 6.

This means, that all *Command->execute() methods
need the return type "int" added. Similar
logic applies to "isHidden(): bool".

In addition, various helper functions need
to have their return type added.

The "Helper::strlen()" was deprecated in Symfony 5.3 and removed in Symfony 6. The replacement is
"Helper::width()" which is solved by using
a helper class to support both Symfony 4+6.

Support for TYPO3 v12 was provided by using
"Typo3Version" with a few switches regarding
Doctrine DBAL 3.x support and Bootstrap
(PageRenderer::setCache is not needed anymore).

Manual tests executed with TYPO3 v12:

  • install:setup
  • backend:createadmin
  • upgrade:prepare

This change adds basic functionality for TYPO3 v12.

Support for TYPO3 v12 was provided by using
"Typo3Version" with a few switches regarding
Doctrine DBAL 3.x support and Bootstrap
(PageRenderer::setCache is not needed anymore).

Manual tests executed with TYPO3 v12:

* install:setup
* backend:createadmin
* upgrade:prepare
SIRprise
SIRprise previously approved these changes Nov 29, 2022
baschny
baschny previously approved these changes Feb 10, 2023
Copy link

@baschny baschny left a comment

Choose a reason for hiding this comment

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

I tested it, looks good to me as a first step, at least to get it compatible again with dev-main as stated in composer.json. Tested most of the provided commands in my test setup.

Maybe "Undraft" and merge this?

@helhum helhum dismissed stale reviews from baschny and SIRprise via 1406aa2 February 11, 2023 13:01
@helhum
Copy link
Member

helhum commented Feb 11, 2023

@baschny thanks for the heads up. If tests for 11 are green, I'll merge this, so that TYPO3 Console dev-main can be installed with TYPO3 dev-main. Still use at own risk, but if this helps, I'll gladly provide that.

@helhum helhum merged commit 25ed2e4 into TYPO3-Console:main Feb 11, 2023
@helhum helhum changed the title [DRAFT][TASK] Provide compatibility with TYPO3 v12 [TASK] Provide basic compatibility with TYPO3 v12 Feb 11, 2023
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.

None yet

5 participants