neos / rector

Neos Rector Recipes for code migrations
6 stars 4 forks source link

Solve all todos in `contentrepository-90.php` #75

Open mhsdesign opened 2 months ago

mhsdesign commented 2 months ago

There are way more todos declared in the source than real issues in this repo.

so just to give awareness to that.

Every comment that is NOT marked with -> compatible with ES CR Node (nothing to do) is missing a migration!

That includes all methods on the TraversableNodeInterface

https://github.com/neos/rector/blob/efcb72511a36f925c31e529b4742f1af81888a73/config/set/contentrepository-90.php#L298-L317

As well as moving

https://github.com/neos/rector/blob/efcb72511a36f925c31e529b4742f1af81888a73/config/set/contentrepository-90.php#L229-L234

and creating nodes:

https://github.com/neos/rector/blob/efcb72511a36f925c31e529b4742f1af81888a73/config/set/contentrepository-90.php#L212-L214

And much more.