neos / rector

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

Critical migrations #25

Closed mhsdesign closed 7 months ago

mhsdesign commented 1 year ago
ahaeslich commented 1 year ago

Ah I love it 💖. Had also tought about collecting those todos in an issue 😄

dlubitz commented 1 year ago

@mhsdesign Could you more explain the required path replacements for fusion?

I already found a rector migration ${node.path} => ${Neos.Node.path(node)}: https://github.com/neos/rector/blob/618a7fde2af27629a522737bd09bacba3549e6d4/tests/ContentRepository90/Rules/FusionNodePathRector/Fixture/some_file.fusion.inc#L8-L8

Is this already done? Or is the migration outdated?

mhsdesign commented 1 year ago

okay i see, but we should also migrate q(node).property('_path') i like to abolish this syntax https://github.com/neos/neos-development-collection/issues/4208