Fixed #13097

Author: brandonkelly

CHANGELOG.md

@@ -13,6 +13,7 @@
 - Fixed an error that occured when sending the activation email for a new user, if there were any validation errors due to new group assignments. ([#13060](https://github.com/craftcms/cms/issues/13060))
 - Fixed a bug where the “Send an activation email now” user account checkbox was losing its value if there were any validation errors.
 - Fixed an error that could occur when creating a database backup on MySQL and MariaDB. ([#12996](https://github.com/craftcms/cms/issues/12996))
+- Fixed a bug where Edit Category screens were including a Parent field, even if the category group’s Max Levels was set to `1`. ([#13097](https://github.com/craftcms/cms/discussions/13097))
 
 ## 4.4.7.1 - 2023-04-15
 

src/elements/Category.php

@@ -641,41 +641,43 @@ protected function metaFieldsHtml(bool $static): string
             $this->slugFieldHtml($static),
         ];
 
-        $fields[] = (function() use ($static) {
-            if ($parentId = $this->getParentId()) {
-                $parent = Craft::$app->getCategories()->getCategoryById($parentId, $this->siteId, [
-                    'drafts' => null,
-                    'draftOf' => false,
-                ]);
-            } else {
-                // If the category already has structure data, use it. Otherwise, use its canonical category
-                /** @var self|null $parent */
-                $parent = self::find()
-                    ->siteId($this->siteId)
-                    ->ancestorOf($this->lft ? $this : ($this->getIsCanonical() ? $this->id : $this->getCanonical(true)))
-                    ->ancestorDist(1)
-                    ->drafts(null)
-                    ->draftOf(false)
-                    ->status(null)
-                    ->one();
-            }
+        $group = $this->getGroup();
 
-            $group = $this->getGroup();
+        if ($group->maxLevels !== 1) {
+            $fields[] = (function() use ($static, $group) {
+                if ($parentId = $this->getParentId()) {
+                    $parent = Craft::$app->getCategories()->getCategoryById($parentId, $this->siteId, [
+                        'drafts' => null,
+                        'draftOf' => false,
+                    ]);
+                } else {
+                    // If the category already has structure data, use it. Otherwise, use its canonical category
+                    /** @var self|null $parent */
+                    $parent = self::find()
+                        ->siteId($this->siteId)
+                        ->ancestorOf($this->lft ? $this : ($this->getIsCanonical() ? $this->id : $this->getCanonical(true)))
+                        ->ancestorDist(1)
+                        ->drafts(null)
+                        ->draftOf(false)
+                        ->status(null)
+                        ->one();
+                }
 
-            return Cp::elementSelectFieldHtml([
-                'label' => Craft::t('app', 'Parent'),
-                'id' => 'parentId',
-                'name' => 'parentId',
-                'elementType' => self::class,
-                'selectionLabel' => Craft::t('app', 'Choose'),
-                'sources' => ["group:$group->uid"],
-                'criteria' => $this->_parentOptionCriteria($group),
-                'limit' => 1,
-                'elements' => $parent ? [$parent] : [],
-                'disabled' => $static,
-                'describedBy' => 'parentId-label',
-            ]);
-        })();
+                return Cp::elementSelectFieldHtml([
+                    'label' => Craft::t('app', 'Parent'),
+                    'id' => 'parentId',
+                    'name' => 'parentId',
+                    'elementType' => self::class,
+                    'selectionLabel' => Craft::t('app', 'Choose'),
+                    'sources' => ["group:$group->uid"],
+                    'criteria' => $this->_parentOptionCriteria($group),
+                    'limit' => 1,
+                    'elements' => $parent ? [$parent] : [],
+                    'disabled' => $static,
+                    'describedBy' => 'parentId-label',
+                ]);
+            })();
+        }
 
         $fields[] = parent::metaFieldsHtml($static);