Merge pull request #2646 from matmair/small-fixes

Small style fixes

InvenTree/InvenTree/helpers.py

@@ -475,7 +475,6 @@ def extract_serial_numbers(serials, expected_quantity, next_number: int):
                     continue
             else:
                 errors.append(_("Invalid group: {g}").format(g=group))
-                continue
 
         # plus signals either
         # 1:  'start+':  expected number of serials, starting at start
@@ -500,7 +499,6 @@ def extract_serial_numbers(serials, expected_quantity, next_number: int):
             # no case
             else:
                 errors.append(_("Invalid group: {g}").format(g=group))
-                continue
 
         # Group should be a number
         elif group:

InvenTree/part/serializers.py

@@ -789,8 +789,7 @@ class BomImportExtractSerializer(DataFileExtractSerializer):
                 pass
 
         # No direct match, where else can we look?
-        if part is None:
-            if part_name or part_ipn:
+        if part is None and (part_name or part_ipn):
             queryset = Part.objects.all()
 
             if part_name:

InvenTree/part/test_bom_import.py

@@ -41,7 +41,7 @@ class BomUploadTest(InvenTreeAPITestCase):
                 assembly=False,
             )
 
-    def post_bom(self, filename, file_data, part=None, clear_existing=None, expected_code=None, content_type='text/plain'):
+    def post_bom(self, filename, file_data, clear_existing=None, expected_code=None, content_type='text/plain'):
 
         bom_file = SimpleUploadedFile(
             filename,
@@ -49,9 +49,6 @@ class BomUploadTest(InvenTreeAPITestCase):
             content_type=content_type,
         )
 
-        if part is None:
-            part = self.part.pk
-
         if clear_existing is None:
             clear_existing = False
 
@@ -189,7 +186,7 @@ class BomUploadTest(InvenTreeAPITestCase):
 
         self.assertIn('No part column specified', str(response.data))
 
-        response = self.post(
+        self.post(
             url,
             {
                 'rows': rows,

InvenTree/stock/api.py

@@ -516,9 +516,7 @@ class StockList(generics.ListCreateAPIView):
                 data['location'] = location.pk
 
         # An expiry date was *not* specified - try to infer it!
-        if 'expiry_date' not in data:
-
-            if part.default_expiry > 0:
+        if 'expiry_date' not in data and part.default_expiry > 0:
             data['expiry_date'] = datetime.now().date() + timedelta(days=part.default_expiry)
 
         # Attempt to extract serial numbers from submitted data

InvenTree/stock/test_api.py

@@ -393,7 +393,7 @@ class StockItemTest(StockAPITestCase):
         self.assertEqual(trackable_part.get_stock_count(), 0)
 
         # This should fail, incorrect serial number count
-        response = self.post(
+        self.post(
             self.list_url,
             data={
                 'part': trackable_part.pk,