Expose preexistent logic in public function getVersionId

[bouland]

Need external access to this part inside beforeSave.

class ConcurrentListener implements EventListenerInterface
{
    public function implementedEvents()
    {
        return [
            'Model.beforeSave' => 'onModelBeforeChange',
            'Model.beforeDelete' => 'onModelBeforeChange',
        ];
    }

    public function onModelBeforeChange(Event $event, Entity $entity)
    {
        $table = $event->getSubject();
        if ($table->behaviors()->has('Version')) {
            $version_id = $table->getVersionId($entity);
            if ($version_id !== $entity->version_id) {
                $event->stopPropagation();
                $entity->setError('version_id', 'This entity has changed since you get it.');
                return false;
            }
        }
    }
}

[coveralls]

Coverage remained the same at 91.908% when pulling 5542910 on bouland:master into 4f0c098 on josegonzalez:master.

[josegonzalez]

Can you add a test for this new method?

[bouland]

Done, but coverage would not change since I only have moved code.

[bouland]

Did you see that checkstyle rules failed on vendor which are not ignored.
Maybe Travis config need update ?