Hej, thanks for your fix - Im not deep enough in that but i think even if i were i would have following questions:

• i though we had Neos\Flow\Annotations\Identity as replacement for ORM\ID
• i cannot really fully grasp the implications without a new tests ^^
• isnt this a bugfix and should target the lowest version?

• i though we had Neos\Flow\Annotations\Identity as replacement for ORM\ID

The used annotation is not really relevant here. F
If you use annotation instead of var tags for that it would not be read either

• i cannot really fully grasp the implications without a new tests ^^

Is it a wish or requirement? Where should it be placed?

• isnt this a bugfix and should target the lowest version?

Agree, its more of s bugfix

Hi, i found that there are several tests already using @ORM\Id annotation but there have been none for @id or #[ORM\ID]

The reason @ORM\Id works even if we are only reading tags, is that the namespace is trimmed and we could have been writing @Nudelsuppe\Id see test 0323dfb

id say add extend one in Neos.Flow/Tests/Functional/Persistence/Fixtures and copy a @ORM\Id definition and add a #[ORM\ID] :)

And this now definitely qualifies as bugfix to the lowest version which attempts to actually provide orm attributes in flow :)

And this now definitely qualifies as bugfix to the lowest version which attempts to actually provide orm attributes in flow :)

That would be 8.4 i think