Correct handling of auto save

[rliebi]

Fixes a bug in #62, #61

[limenet]

Thanks for the bugfix, you're correct. Unfortunately, this change is not backwards compatible (compared to 4.0.0). If I'm not mistaken, the method should read as follows:

    private function shouldHandle(AssetEvent|DataObjectEvent|DocumentEvent $event): bool
    {
        if (!self::$isEnabled) {
            return false;
        }

        $isAutoSave = $event->hasArgument('isAutoSave') && $event->getArgument('isAutoSave') === true;

        if ($event instanceof AssetEvent && (!$isAutoSave || $this->configurationRepository->shouldHandleAssetAutoSave())) {
            return true;
        }

        if ($event instanceof DataObjectEvent && (!$isAutoSave || $this->configurationRepository->shouldHandleDataObjectAutoSave())) {
            return true;
        }

        if ($event instanceof DocumentEvent && (!$isAutoSave || $this->configurationRepository->shouldHandleDocumentAutoSave())) {
            return true;
        }

        return false;
    }

What do you think?

[rliebi]

Your solution does the same thing as what I proposed but mine is slightly more readable, because of the early exit when there is no autosave.
We already assessed, that the event is one of AssetEvent,DataObjectEvent or DocumentEvent so imho there is no need for a check inside the if statements when we could exit early.

[limenet]

Sorry, my bad.