neos / neos-development-collection

The unified repository containing the Neos core packages, used for Neos development.
https://www.neos.io/
GNU General Public License v3.0
260 stars 220 forks source link

Replace `ExpectedVersion::ANY()` with proper version checks #5058

Open bwaidelich opened 1 month ago

bwaidelich commented 1 month ago

In the current core code we have 21 occurrences of ExpectedVersion::ANY(). Some of them are desired, for example in EventsToPublish::empty().

But all content-stream related events should always be published with the actual version of the respective CS (for example in SubtreeTagging::handleTagSubtree().

For workspace related events we should consider adding the workspace version to the read model, too, once those are integrated to the Content Graph projection (see #5038)