From ba49b080b0154774c6543b5d0be5efa80efb253e Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 11:46:53 +1000
Subject: [PATCH 1/4] Some small tweaks

---
 InvenTree/part/models.py  | 2 +-
 InvenTree/stock/models.py | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/InvenTree/part/models.py b/InvenTree/part/models.py
index 4022b7ec71..887c983511 100644
--- a/InvenTree/part/models.py
+++ b/InvenTree/part/models.py
@@ -533,7 +533,7 @@ class SupplierPart(models.Model):
         return cost + self.base_cost
 
     def __str__(self):
-        return "{sku} - {supplier}".format(
+        return "{supplier} ({sku})".format(
             sku=self.SKU,
             supplier=self.supplier.name)
 
diff --git a/InvenTree/stock/models.py b/InvenTree/stock/models.py
index 636efeb293..e74c4c30d5 100644
--- a/InvenTree/stock/models.py
+++ b/InvenTree/stock/models.py
@@ -77,6 +77,14 @@ class StockItem(models.Model):
             )
 
     def clean(self):
+        """ Validate the StockItem object (separate to field validation)
+
+        The following validation checks are performed:
+
+        - The 'part' and 'supplier_part.part' fields cannot point to the same Part object
+        - The 'part' does not belong to itself
+        - Quantity must be 1 if the StockItem has a serial number
+        """ 
 
         # The 'supplier_part' field must point to the same part!
         try:

From a9224445b6336fe2289dada0232c7157d461069e Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 11:49:20 +1000
Subject: [PATCH 2/4] Added docstring

---
 InvenTree/InvenTree/serializers.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/InvenTree/InvenTree/serializers.py b/InvenTree/InvenTree/serializers.py
index 7149668cc4..4ad5d1b87a 100644
--- a/InvenTree/InvenTree/serializers.py
+++ b/InvenTree/InvenTree/serializers.py
@@ -37,6 +37,11 @@ class InvenTreeModelSerializer(serializers.ModelSerializer):
     """
 
     def validate(self, data):
+        """ Perform serializer validation.
+        In addition to running validators on the serializer fields,
+        this class ensures that the underlying model is also validated.
+        """
+        
         # Run any native validation checks first (may throw an ValidationError)
         data = super(serializers.ModelSerializer, self).validate(data)
 

From 2f7f49c116d3358c6756584ee31b48aa222e95bd Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 11:54:40 +1000
Subject: [PATCH 3/4] PEP fix

---
 InvenTree/stock/models.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/InvenTree/stock/models.py b/InvenTree/stock/models.py
index e74c4c30d5..2f61c068cd 100644
--- a/InvenTree/stock/models.py
+++ b/InvenTree/stock/models.py
@@ -84,7 +84,7 @@ class StockItem(models.Model):
         - The 'part' and 'supplier_part.part' fields cannot point to the same Part object
         - The 'part' does not belong to itself
         - Quantity must be 1 if the StockItem has a serial number
-        """ 
+        """
 
         # The 'supplier_part' field must point to the same part!
         try:

From 32bbbbd4048a20002c6c20eabd427277832336ce Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 19:41:03 +1000
Subject: [PATCH 4/4] Comments for pip requirements.txt

---
 requirements.txt | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/requirements.txt b/requirements.txt
index 6cf3834241..2bc353b3ee 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -1,15 +1,15 @@
-Django==2.2
-psycopg2>=2.8.1
-pillow>=5.0.0
-djangorestframework>=3.6.2
-django_filter>=1.0.2
-coreapi>=2.3.0
-pygments>=2.2.0
-tablib>=0.13.0
-django-crispy-forms>=1.7.2
-django-import-export>=1.0.0
-django-cleanup>=2.1.0
-django-qr-code==1.0.0
-flake8==3.3.0
-coverage>=4.5.3
-python-coveralls==2.9.1
\ No newline at end of file
+Django==2.2                     # Django package
+psycopg2>=2.8.1                 # PostgreSQL package 
+pillow>=5.0.0                   # Image manipulation
+djangorestframework>=3.6.2      # DRF framework
+django_filter>=1.0.2            # Extended filtering options
+coreapi>=2.3.0                  # API documentation
+pygments>=2.2.0                 # Syntax highlighting
+tablib>=0.13.0                  # Import / export data files
+django-crispy-forms>=1.7.2      # Form helpers
+django-import-export>=1.0.0     # Data import / export for admin interface
+django-cleanup>=2.1.0           # Manage deletion of old / unused uploaded files
+django-qr-code==1.0.0           # Generate QR codes
+flake8==3.3.0                   # PEP checking
+coverage>=4.5.3                 # Unit test coverage
+python-coveralls==2.9.1         # Coveralls linking (for Travis)
\ No newline at end of file