update event sourcing for new interface

[prolic]

No description provided.

[codeliner]

I know it requires more work, but it would be nice if we rename _version to _aggregate_version to make the concept and difference between domain event version and event no in the streams explicit. What do you think @prolic ?

[prolic]

Ok no problem. Will update those 100 repos tomorrow :-)

Am 08.11.2016 01:24 schrieb "Alexander Miertsch" [email protected]:

@codeliner requested changes on this pull request.

In src/Aggregate/AggregateRepository.php
#34 (review)
:

•            $snapshot->lastVersion() + 1
  

•        );
  

•    } else {
  

•        $metadataMatcher = new MetadataMatcher();
  

•        $metadataMatcher = $metadataMatcher->withMetadataMatch(
  

•            '_aggregate_type',
  

•            Operator::EQUALS(),
  

•            $this->aggregateType->toString()
  

•        );
  

•        $metadataMatcher = $metadataMatcher->withMetadataMatch(
  

•            '_aggregate_id',
  

•            Operator::EQUALS(),
  

•            $aggregateId
  

•        );
  

•        $metadataMatcher = $metadataMatcher->withMetadataMatch(
  

•            '_version',
  

I know it requires more work, but it would be nice if we rename _version
to _aggregate_version to make the concept and difference between domain
event version and event no in the streams explicit. What do you think
@prolic https://github.com/prolic ?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#34 (review),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYEvE1xwPfb11pqL1J5oC2-r4z0PusAks5q717UgaJpZM4KrSKk
.