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

[FEATURE] Add migration to associative items array keys #3400

Merged
merged 4 commits into from
Jun 22, 2023

Conversation

nhovratov
Copy link
Contributor

Fixes: #3399

@nhovratov
Copy link
Contributor Author

nhovratov commented Mar 23, 2023

Hi, first contribution here. I have no idea how php parser works. Need to learn it first to complete this PR. I don't mind, if someone picks this up. Test fixtures are set up.

And: Thank you for this great tool. It helped me already many times while updating TYPO3 instances.

Edit: Learned a bit how to use expressions / nodes and implemented the migration. Please have a look whether this is the way to go.

@sabbelasichon
Copy link
Owner

@nhovratov Thank you very much. Awesome to see new people contributing to the project. I will have a look soon.

@kevin-appelt
Copy link

I guess this could be the most wanted rector rule for TYPO3 v12 upgrades.
Anything I can do to get this merged? :)

@helsner
Copy link
Collaborator

helsner commented Jun 21, 2023

@sabbelasichon can you check along?
are there any reasons to not merge and freshly release?

@sabbelasichon sabbelasichon merged commit daa9a63 into sabbelasichon:main Jun 22, 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.

Feature: #99739 - Associative array keys for TCA items
4 participants