Fixed recursively-nested content editing bug

https://app.frontapp.com/open/cnv_h4agcll?key=lMAQl8heAb97xslWPoAN1Acr-Duy_yAl

Author: brandonkelly

CHANGELOG.md

@@ -2,13 +2,16 @@
 
 ## Unreleased
 
+- Added `craft\base\NestedElementInterface::getOwners()`.
+- Added `craft\base\NestedElementTrait::getOwners()`.
 - Fixed a bug where entries’ and categories’ descendants’ URIs weren’t always getting updated after a parent’s URI changed. ([#17804](https://github.com/craftcms/cms/discussions/17804))
 - Fixed a bug where addresses weren’t saving `firstName`, `lastName`, and `fullName` values properly. ([#17807](https://github.com/craftcms/cms/issues/17807))
 - Fixed an n+1 query bug when working with Content Block fields. ([#17801](https://github.com/craftcms/cms/issues/17801))
 - Fixed a bug where element selector modals’ sidebars would go out of view if the browser had a custom font size. ([#17809](https://github.com/craftcms/cms/issues/17809))
 - Fixed a bug where relation field values weren’t getting updated properly when two elements were merged together. ([#17817](https://github.com/craftcms/cms/issues/17817))
 - Fixed an authorization error that could occur when editing a provisional draft of a nested element.
 - Fixed a bug where custom field values could be lost when changing an entry type. ([#17821](https://github.com/craftcms/cms/issues/17821))
+- Fixed a bug where non-admin users couldn’t edit recursively-nested elements if they didn’t have save permissions for the top-level element.
 - Fixed a styling issue.
 
 ## 5.8.16 - 2025-08-29

src/base/NestedElementInterface.php

@@ -82,6 +82,16 @@ public function getOwner(): ?ElementInterface;
      */
     public function setOwner(?ElementInterface $owner): void;
 
+    /**
+     * Returns each of the element’s owners
+     *
+     * @param array $criteria
+     * @return ElementInterface[]
+     * @throws InvalidConfigException if the element is misconfigured
+     * @since 5.8.17
+     */
+    public function getOwners(array $criteria = []): array;
+
     /**
      * Returns the field that contains the element.
      *

src/base/NestedElementTrait.php

@@ -113,6 +113,12 @@ public static function eagerLoadingMap(array $sourceElements, string $handle): a
      */
     private ElementInterface|false|null $_owner = null;
 
+    /**
+     * @var ElementInterface[]
+     * @see getOwners()
+     */
+    private array $_owners;
+
     public function __clone(): void
     {
         parent::__clone();
@@ -301,6 +307,32 @@ public function getOwner(): ?ElementInterface
         return $this->_owner ?: null;
     }
 
+    /**
+     * @@inheritdoc
+     */
+    public function getOwners(array $criteria = []): array
+    {
+        if (!isset($this->_owners)) {
+            $this->_owners = [];
+            $ownerType = $this->ownerType();
+            if ($ownerType) {
+                $ownerIds = (new Query())
+                    ->select('ownerId')
+                    ->from(Table::ELEMENTS_OWNERS)
+                    ->where(['elementId' => $this->id])
+                    ->column();
+                if (!empty($ownerIds)) {
+                    $query = $ownerType::find()
+                        ->id($ownerIds);
+                    Craft::configure($query, $criteria + $this->ownerCriteria());
+                    $this->_owners = $query->all();
+                }
+            }
+        }
+
+        return $this->_owners;
+    }
+
     private function ownerCriteria(): array
     {
         return [

src/fields/ContentBlock.php

@@ -344,7 +344,26 @@ public function canViewElement(NestedElementInterface $element, User $user): ?bo
     public function canSaveElement(NestedElementInterface $element, User $user): ?bool
     {
         $owner = $element->getOwner();
-        return $owner && Craft::$app->getElements()->canSave($owner, $user);
+
+        if (!$owner) {
+            return false;
+        }
+
+        if (Craft::$app->getElements()->canSave($owner, $user)) {
+            return true;
+        }
+
+        // Check all the owners. Maybe the user can save one of the other ones?
+        /** @phpstan-ignore-next-line  */
+        if (!Craft::$app->getElements()->canSave($owner, $user) && !$owner->getIsRevision()) {
+            foreach ($element->getOwners(['revisions' => false]) as $o) {
+                if ($o->id !== $owner->id && Craft::$app->getElements()->canSave($o, $user)) {
+                    return true;
+                }
+            }
+        }
+
+        return false;
     }
 
     /**

src/fields/Matrix.php

@@ -564,7 +564,26 @@ public function canViewElement(NestedElementInterface $element, User $user): ?bo
     public function canSaveElement(NestedElementInterface $element, User $user): ?bool
     {
         $owner = $element->getOwner();
-        return $owner && Craft::$app->getElements()->canSave($owner, $user);
+
+        if (!$owner) {
+            return false;
+        }
+
+        if (Craft::$app->getElements()->canSave($owner, $user)) {
+            return true;
+        }
+
+        // Check all the owners. Maybe the user can save one of the other ones?
+        /** @phpstan-ignore-next-line  */
+        if (!Craft::$app->getElements()->canSave($owner, $user) && !$owner->getIsRevision()) {
+            foreach ($element->getOwners(['revisions' => false]) as $o) {
+                if ($o->id !== $owner->id && Craft::$app->getElements()->canSave($o, $user)) {
+                    return true;
+                }
+            }
+        }
+
+        return false;
     }
 
     /**