Merge pull request #2452 from ruohola/fix-order-line-too-short-sku

Core: Make `OrderLine.sku` `max_length` match that of `Product.sku`

CHANGELOG.md

@@ -12,6 +12,10 @@ List all changes after the last release here (newer on top). Each change on a se
 
 - Admin: change so TaxClassEditView is a FormPartView
 
+### Fixed
+
+- Core: Fix error that occurred when creating an order with a product which SKU was longer than 48 characters. 
+
 ## [2.6.0] - 2021-03-29
 
 ### Changed

shuup/core/migrations/0085_longer_order_line_sku.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.2.17 on 2021-03-31 08:49
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('shuup', '0084_tracking_url'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='orderline',
+            name='sku',
+            field=models.CharField(blank=True, max_length=128, verbose_name='line SKU'),
+        ),
+    ]

shuup/core/models/_order_lines.py

@@ -96,7 +96,7 @@ class AbstractOrderLine(MoneyPropped, models.Model, Priceful):
     )
     ordering = models.IntegerField(default=0, verbose_name=_("ordering"))
     type = EnumIntegerField(OrderLineType, default=OrderLineType.PRODUCT, verbose_name=_("line type"))
-    sku = models.CharField(max_length=48, blank=True, verbose_name=_("line SKU"))
+    sku = models.CharField(max_length=128, blank=True, verbose_name=_("line SKU"))
     text = models.CharField(max_length=256, verbose_name=_("line text"))
     accounting_identifier = models.CharField(max_length=32, blank=True, verbose_name=_("accounting identifier"))
     require_verification = models.BooleanField(default=False, verbose_name=_("require verification"))