Fix error when not autosaving drafts, when validation fails and new b…

…lock nested in new block

CHANGELOG.md

@@ -6,6 +6,7 @@
 - Fixed a JavaScript error that occurred when pasting a block type
 - Fixed an incorrect German translation of 'Min Blocks' (thanks @alumpe)
 - Fixed a bug where the child blocks UI element wasn't appearing on the field layout designer sidebar on existing block types
+- Fixed a PHP error that occurred when saving an element that doesn't autosave drafts, if a validation error occurred and there was a new block nested in another new block
 
 ## 3.1.8 - 2022-07-15
 

src/Field.php

@@ -1107,6 +1107,7 @@ private function _createBlocksFromSerializedData(array $value, ElementInterface
         /** @var Block[] $blocks */
         $blocks = [];
         $prevBlock = null;
+        $sortOrderCounter = 1;
 
         foreach ($newSortOrder as $blockId) {
             if (isset($newBlockData[$blockId])) {
@@ -1171,6 +1172,7 @@ private function _createBlocksFromSerializedData(array $value, ElementInterface
             $block->setOwner($element);
             $block->oldLevel = $block->level;
             $block->level = $blockLevel;
+            $block->sortOrder = $sortOrderCounter++;
 
             if (isset($blockData['collapsed'])) {
                 $block->setCollapsed((bool)$blockData['collapsed']);

src/elements/db/BlockQuery.php

@@ -576,7 +576,11 @@ private function _getFilteredResult(): array
 
             if (method_exists($this, $method)) {
                 $currentFiltered = $this->$method($this->_allElements, $value);
-                $result = array_values(array_uintersect($result, $currentFiltered, fn($a, $b) => $a->lft <=> $b->lft));
+                $result = array_values(array_uintersect(
+                    $result,
+                    $currentFiltered,
+                    fn($a, $b) => $a->lft ? $a->lft <=> $b->lft : $a->sortOrder <=> $b->sortOrder
+                ));
             }
         }