Merge pull request #3787 from craftcms/feature/pt-2323-5x-max-levels-…

…are-not-taken-into-account-for-structured

[5.x] Product type’s max levels setting not being respected

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Unreleased
 
+- Fixed a bug where a structured product type’s “Max Levels” setting wasn’t being respected. ([#3785](https://github.com/craftcms/commerce/issues/3785))
 - Fixed a bug where users could access the Commerce user screen when the current user didn’t have permission. 
 
 ## 5.2.6 - 2024-11-26

src/Plugin.php

@@ -257,7 +257,7 @@ public static function editions(): array
     /**
      * @inheritDoc
      */
-    public string $schemaVersion = '5.2.0.5';
+    public string $schemaVersion = '5.2.7.0';
 
     /**
      * @inheritdoc

src/migrations/m241128_174712_fix_maxLevels_structured_productTypes.php

@@ -0,0 +1,43 @@
+<?php
+
+namespace craft\commerce\migrations;
+
+use craft\commerce\db\Table;
+use craft\db\Migration;
+use craft\db\Query;
+use craft\db\Table as CraftTable;
+
+/**
+ * m241128_174712_fix_maxLevels_structured_productTypes migration.
+ */
+class m241128_174712_fix_maxLevels_structured_productTypes extends Migration
+{
+    /**
+     * @inheritdoc
+     */
+    public function safeUp(): bool
+    {
+        // Get all product types from Commerce project config
+        $structuredProductTypesWithMaxLevels = (new Query())
+            ->from(Table::PRODUCTTYPES)
+            ->where(['isStructure' => true])
+            ->andWhere(['not', ['maxLevels' => null]])
+            ->collect();
+
+        // Loop through and update the `maxLevels` column in the `structures` table
+        $structuredProductTypesWithMaxLevels->each(function($productType) {
+            $this->update(CraftTable::STRUCTURES, ['maxLevels' => $productType['maxLevels']], ['id' => $productType['structureId']]);
+        });
+
+        return true;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function safeDown(): bool
+    {
+        echo "m241128_174712_fix_maxLevels_structured_productTypes cannot be reverted.\n";
+        return false;
+    }
+}

src/services/ProductTypes.php

@@ -475,7 +475,7 @@ public function handleChangedProductType(ConfigEvent $event): void
                 $structureUid = $data['structure']['uid'];
                 $structure = Craft::$app->getStructures()->getStructureByUid($structureUid, true) ?? new Structure(['uid' => $structureUid]);
                 $isNewStructure = empty($structure->id);
-                $structure->maxLevels = $data['structure']['maxLevels'] ?? null;
+                $structure->maxLevels = $data['maxLevels'] ?? null;
                 Craft::$app->getStructures()->saveStructure($structure);
                 $productTypeRecord->structureId = $structure->id;
             } else {