BUGFIX: do not crash when node already disabled/enabled

Neos.ContentRepository.Core/Classes/CommandHandler/CommandResult.php

@@ -7,6 +7,7 @@
 use Neos\ContentRepository\Core\ContentRepository;
 use Neos\ContentRepository\Core\Projection\ProjectionInterface;
 use Neos\ContentRepository\Core\Projection\ProjectionStateInterface;
+use Neos\EventStore\Model\Event\Version;
 use Neos\EventStore\Model\EventStore\CommitResult;
 use Neos\EventStore\Model\Event\SequenceNumber;
 
@@ -25,6 +26,20 @@ public function __construct(
     ) {
     }
 
+    /**
+     * an empty command result which should not result in projection updates
+     * @return self
+     */
+    public static function empty(): self {
+        return new self(
+            PendingProjections::empty(),
+            new CommitResult(
+                Version::first(),
+                SequenceNumber::none()
+            )
+        );
+    }
+
     /**
      * Wait until all projections are up to date; i.e. have processed the events.
      *

Neos.ContentRepository.Core/Classes/CommandHandler/PendingProjections.php

@@ -85,6 +85,11 @@ public function __construct(
     ) {
     }
 
+    public static function empty(
+    ): self {
+        return new self(Projections::create(), []);
+    }
+
     public static function fromProjectionsAndEventsAndSequenceNumber(
         Projections $allProjections,
         Events $events,

Neos.ContentRepository.Core/Classes/EventStore/EventPersister.php

@@ -15,6 +15,7 @@
 use Neos\EventStore\Model\Event\EventId;
 use Neos\EventStore\Model\Event\EventMetadata;
 use Neos\EventStore\Model\Events;
+use Neos\EventStore\Model\EventStore\CommitResult;
 
 /**
  * Internal service to persist {@see EventInterface} with the proper normalization, and triggering the
@@ -39,6 +40,9 @@ public function __construct(
      */
     public function publishEvents(EventsToPublish $eventsToPublish): CommandResult
     {
+        if ($eventsToPublish->events->isEmpty()) {
+            return CommandResult::empty();
+        }
         // the following logic could also be done in an AppEventStore::commit method (being called
         // directly from the individual Command Handlers).
         $normalizedEvents = Events::fromArray(

Neos.ContentRepository.Core/Classes/EventStore/Events.php

@@ -49,4 +49,9 @@ public function map(\Closure $callback): array
     {
         return array_map($callback, $this->events);
     }
+
+    public function isEmpty(): bool
+    {
+        return empty($this->events);
+    }
 }

Neos.ContentRepository.Core/Classes/EventStore/EventsToPublish.php

@@ -24,4 +24,13 @@ public function __construct(
         public readonly ExpectedVersion $expectedVersion,
     ) {
     }
+
+    public static function empty(): self
+    {
+        return new EventsToPublish(
+            StreamName::fromString("empty"),
+            Events::fromArray([]),
+            ExpectedVersion::ANY()
+        );
+    }
 }

Neos.ContentRepository.Core/Classes/Feature/Common/ConstraintChecks.php

@@ -609,39 +609,6 @@ protected function requireNodeAggregateToNotOccupyDimensionSpacePoint(
         }
     }
 
-    /**
-     * @throws NodeAggregateCurrentlyDoesNotDisableDimensionSpacePoint
-     */
-    protected function requireNodeAggregateToDisableDimensionSpacePoint(
-        NodeAggregate $nodeAggregate,
-        DimensionSpacePoint $dimensionSpacePoint
-    ): void {
-        if (!$nodeAggregate->disablesDimensionSpacePoint($dimensionSpacePoint)) {
-            throw new NodeAggregateCurrentlyDoesNotDisableDimensionSpacePoint(
-                'Node aggregate "' . $nodeAggregate->nodeAggregateId->value
-                    . '" currently does not disable dimension space point '
-                    . json_encode($dimensionSpacePoint) . '.',
-                1557735431
-            );
-        }
-    }
-
-    /**
-     * @throws NodeAggregateCurrentlyDisablesDimensionSpacePoint
-     */
-    protected function requireNodeAggregateToNotDisableDimensionSpacePoint(
-        NodeAggregate $nodeAggregate,
-        DimensionSpacePoint $dimensionSpacePoint
-    ): void {
-        if ($nodeAggregate->disablesDimensionSpacePoint($dimensionSpacePoint)) {
-            throw new NodeAggregateCurrentlyDisablesDimensionSpacePoint(
-                'Node aggregate "' . $nodeAggregate->nodeAggregateId->value
-                    . '" currently disables dimension space point ' . json_encode($dimensionSpacePoint) . '.',
-                1555179563
-            );
-        }
-    }
-
     protected function validateReferenceProperties(
         ReferenceName $referenceName,
         PropertyValuesToWrite $referenceProperties,

Neos.ContentRepository.Core/Classes/Feature/NodeDisabling/NodeDisabling.php

@@ -60,10 +60,11 @@ private function handleDisableNodeAggregate(
             $nodeAggregate,
             $command->coveredDimensionSpacePoint
         );
-        $this->requireNodeAggregateToNotDisableDimensionSpacePoint(
-            $nodeAggregate,
-            $command->coveredDimensionSpacePoint
-        );
+
+        if ($nodeAggregate->disablesDimensionSpacePoint($command->coveredDimensionSpacePoint)) {
+            // already disabled, so we can return a no-operation.
+            return EventsToPublish::empty();
+        }
 
         $affectedDimensionSpacePoints = $command->nodeVariantSelectionStrategy
             ->resolveAffectedDimensionSpacePoints(
@@ -113,10 +114,11 @@ public function handleEnableNodeAggregate(
             $nodeAggregate,
             $command->coveredDimensionSpacePoint
         );
-        $this->requireNodeAggregateToDisableDimensionSpacePoint(
-            $nodeAggregate,
-            $command->coveredDimensionSpacePoint
-        );
+
+        if (!$nodeAggregate->disablesDimensionSpacePoint($command->coveredDimensionSpacePoint)) {
+            // already enabled, so we can return a no-operation.
+            return EventsToPublish::empty();
+        }
 
         $affectedDimensionSpacePoints = $command->nodeVariantSelectionStrategy
             ->resolveAffectedDimensionSpacePoints(