Fixed sku getting too long

In addition to the b38491e fix

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Release Notes for Craft Commerce
 
+## Unreleased
+
+- Fixed an SQL error that occurred when duplicating variants. ([#3727](https://github.com/craftcms/commerce/issues/3727))
+
 ## 5.2.0 - 2024-10-16
 
 ### Store Management

src/Plugin.php

@@ -254,7 +254,7 @@ public static function editions(): array
     /**
      * @inheritDoc
      */
-    public string $schemaVersion = '5.2.0.3';
+    public string $schemaVersion = '5.2.0.4';
 
     /**
      * @inheritdoc

src/migrations/m241017_072151_fix_temp_skus.php

@@ -0,0 +1,43 @@
+<?php
+
+namespace craft\commerce\migrations;
+
+use Craft;
+use craft\commerce\helpers\Purchasable;
+use craft\db\Migration;
+
+/**
+ * m241017_072151_fix_temp_skus migration.
+ */
+class m241017_072151_fix_temp_skus extends Migration
+{
+    /**
+     * @inheritdoc
+     */
+    public function safeUp(): bool
+    {
+        // get any sku that starts with __temp_ in the purchasables table, loop over them and replace with a new SKU
+        $purchasables = (new \craft\db\Query())
+            ->select(['id', 'sku'])
+            ->from('{{%commerce_purchasables}}')
+            ->where(['like', 'sku', '__temp_%', false])
+            ->all();
+
+        // Need a unique one per purchasable
+        foreach ($purchasables as $purchasable) {
+            $newSku = Purchasable::tempSku();
+            $this->update('{{%commerce_purchasables}}', ['sku' => $newSku], ['id' => $purchasable['id']]);
+        }
+
+        return true;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function safeDown(): bool
+    {
+        echo "m241017_072151_fix_temp_skus cannot be reverted.\n";
+        return false;
+    }
+}