Throw an InvalidConfigException when entry is missing sectionId/fieldId/ownerId

brandonkelly · brandonkelly · commit 89e8e937fcb8 · 2024-07-15T05:00:30.000-07:00
#15345
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ## Unreleased
 
 - `craft\helpers\UrlHelper::actionUrl()` now returns URLs based on the primary site’s base URL (if it has one), for console requests if the `@web` alias wasn’t explicitly defined.
+- An exception is now thrown when attempting to save an entry that’s missing `sectionId` or `fieldId` + `ownerId` values. ([#15345](https://github.com/craftcms/cms/discussions/15345))
 - Fixed a bug where it wasn’t possible to expand/collapse descendants of disabled table rows within element select modals. ([#15337](https://github.com/craftcms/cms/issues/15337))
 - Fixed a bug where addresses were getting truncated within address cards. ([#15338](https://github.com/craftcms/cms/issues/15338))
 
diff --git a/src/elements/Entry.php b/src/elements/Entry.php
@@ -910,6 +910,10 @@ public function getSupportedSites(): array
             return $this->getField()->getSupportedSitesForElement($this);
         }
 
+        if (!isset($this->sectionId)) {
+            throw new InvalidConfigException('Either `sectionId` or `fieldId` + `ownerId` must be set on the entry.');
+        }
+
         $section = $this->getSection();
         /** @var Site[] $allSites */
         $allSites = ArrayHelper::index(Craft::$app->getSites()->getAllSites(true), 'id');
@@ -1027,6 +1031,10 @@ public function getUriFormat(): ?string
             return $this->getField()->getUriFormatForElement($this);
         }
 
+        if (!isset($this->sectionId)) {
+            throw new InvalidConfigException('Either `sectionId` or `fieldId` + `ownerId` must be set on the entry.');
+        }
+
         $sectionSiteSettings = $this->getSection()->getSiteSettings();
 
         if (!isset($sectionSiteSettings[$this->siteId])) {
