Require Doctrine Migrations 3.x, support PHP 8.2 and TYPO3 v12 #20
Conversation
andreaswolf
changed the title
andreaswolf
force-pushed
the
raise-requirements
branch
from
af24b96
to
ece8f13
Compare
andreaswolf
force-pushed
the
raise-requirements
branch
from
ece8f13
to
489567b
Compare
* Implement a DependencyFactory for TYPO3 * Get rid of DoctrineService * Get rid of overwriting the now "final class" *Command from doctrine:migrate * Wire the dependencies in Services.yaml * Get rid of obsolete Commands.php * Add new commands introduced with doctrine/migrations v3: - migrations:dump-schema - migrations:generate - migrations:latest - migrations:list - migrations:up-to-date !!! Compatibility Notes: * `--dry-run` is currently not supported anymore for the "AbstractDataHandlerMigration" * An upgrade in the doctrine_migrationstatus is required, see README.md
|
Don't know if you wanted to continue working on it, but it turned up to be more work than expected at first, but it worked. Here is the completion: Please take a look, maybe we could finish that off and make a release compatible with TYPO3 v11 and v12 soon. |
To transform the version column from single version to fully qualified classname. See https://github.com/doctrine/migrations/blob/3.6.x/UPGRADE.md
andreaswolf
force-pushed
the
raise-requirements
branch
from
9273a28
to
5fd1f0b
Compare
|
There seems to be a regression in doctrine/migrations 3.7.1 or 3.7.0, since the pipelines worked with 3.6.0 (in PR #23). |
This is due to https://github.com/doctrine/migrations/pull/1363/files, but it seems to be a TYPO3 Core bug, where |
|
@andreaswolf great that you took over. I will have no time to follow-up on that in the next days, so either you continue to tackle that or I can also take a look at it in a near future. |
I opened a bug on TYPO3 Forge: https://forge.typo3.org/issues/102479 and pushed a patch already. |
The patch has been released in TYPO3 12.4.9, which should now become the minimum supported 12.x version of this extension. |
See https://forge.typo3.org/issues/102479 for the issue that makes this necessary.
No description provided.