[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

ecommerce/serializers/__init__.py

@@ -567,12 +567,12 @@ class OrderHistorySerializer(serializers.ModelSerializer):
     @extend_schema_field(serializers.ListField)
     def get_titles(self, instance):
         titles = []
-        # Access prefetched lines without method calls to avoid N+1 
-        lines = getattr(instance, '_prefetched_objects_cache', {}).get('lines', [])
+        # Access prefetched lines without method calls to avoid N+1
+        lines = getattr(instance, "_prefetched_objects_cache", {}).get("lines", [])
         if not lines:
             # Fallback if not prefetched
             lines = list(instance.lines.all())
-        
+
         product_ids = [line.product_version.field_dict["id"] for line in lines]
         products_by_id = models.Product.all_objects.in_bulk(product_ids)
 
@@ -710,12 +710,16 @@ def to_representation(self, instance):
             raise AttributeError  # noqa: TRY004
 
         # Access prefetched transactions without method calls to avoid N+1
-        transactions = getattr(instance, '_prefetched_objects_cache', {}).get('transactions', [])
+        transactions = getattr(instance, "_prefetched_objects_cache", {}).get(
+            "transactions", []
+        )
         if not transactions:
-            # Fallback if not prefetched 
+            # Fallback if not prefetched
             transactions = list(instance.transactions.all())
-        
-        transaction = max(transactions, key=lambda t: t.created_on) if transactions else None
+
+        transaction = (
+            max(transactions, key=lambda t: t.created_on) if transactions else None
+        )
 
         return transaction  # noqa: RET504
 
@@ -833,11 +837,13 @@ class Meta:
 class TransactionLineSerializer(serializers.BaseSerializer):
     def to_representation(self, instance):
         # Access prefetched discounts without method calls to avoid N+1
-        discounts = getattr(instance.order, '_prefetched_objects_cache', {}).get('discounts', [])
+        discounts = getattr(instance.order, "_prefetched_objects_cache", {}).get(
+            "discounts", []
+        )
         if not discounts:
             # Fallback if not prefetched
             discounts = list(instance.order.discounts.all())
-            
+
         coupon_redemption = discounts[0] if discounts else None
         discount = 0.0
 
@@ -909,11 +915,13 @@ def get_order(self, instance):
     def get_coupon(self, instance):
         """Get discount code from the discount redemption if available"""
         # Access prefetched discounts without method calls to avoid N+1
-        discounts = getattr(instance, '_prefetched_objects_cache', {}).get('discounts', [])
+        discounts = getattr(instance, "_prefetched_objects_cache", {}).get(
+            "discounts", []
+        )
         if not discounts:
             # Fallback if not prefetched
             discounts = list(instance.discounts.all())
-            
+
         coupon_redemption = discounts[0] if discounts else None
         if not coupon_redemption:
             return None

ecommerce/serializers/v0/__init__.py

@@ -654,11 +654,11 @@ class OrderHistorySerializer(serializers.ModelSerializer):
     def get_titles(self, instance):
         titles = []
         # Access prefetched lines without method calls to avoid N+1
-        lines = getattr(instance, '_prefetched_objects_cache', {}).get('lines', [])
+        lines = getattr(instance, "_prefetched_objects_cache", {}).get("lines", [])
         if not lines:
             # Fallback if not prefetched
             lines = list(instance.lines.all())
-            
+
         product_ids = [line.product_version.field_dict["id"] for line in lines]
         products_by_id = models.Product.all_objects.in_bulk(product_ids)
 
@@ -816,12 +816,16 @@ def to_representation(self, instance):
             raise AttributeError  # noqa: TRY004
 
         # Access prefetched transactions without method calls to avoid N+1
-        transactions = getattr(instance, '_prefetched_objects_cache', {}).get('transactions', [])
+        transactions = getattr(instance, "_prefetched_objects_cache", {}).get(
+            "transactions", []
+        )
         if not transactions:
             # Fallback if not prefetched
             transactions = list(instance.transactions.all())
-            
-        transaction = max(transactions, key=lambda t: t.created_on) if transactions else None
+
+        transaction = (
+            max(transactions, key=lambda t: t.created_on) if transactions else None
+        )
 
         return transaction  # noqa: RET504
 
@@ -966,11 +970,13 @@ def get_order(self, instance):
     def get_coupon(self, instance):
         """Get discount code from the discount redemption if available"""
         # Access prefetched discounts without method calls to avoid N+1
-        discounts = getattr(instance, '_prefetched_objects_cache', {}).get('discounts', [])
+        discounts = getattr(instance, "_prefetched_objects_cache", {}).get(
+            "discounts", []
+        )
         if not discounts:
             # Fallback if not prefetched
             discounts = list(instance.discounts.all())
-            
+
         coupon_redemption = discounts[0] if discounts else None
         if not coupon_redemption:
             return None