From c2a35b6656bc3ecdc8465be114d067cf250ac344 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 23:00:38 +1000
Subject: [PATCH 1/8] Add 'active' field to Part model

---
 InvenTree/part/migrations/0012_part_active.py | 18 ++++++++++++++++++
 InvenTree/part/models.py                      |  3 +++
 2 files changed, 21 insertions(+)
 create mode 100644 InvenTree/part/migrations/0012_part_active.py

diff --git a/InvenTree/part/migrations/0012_part_active.py b/InvenTree/part/migrations/0012_part_active.py
new file mode 100644
index 0000000000..c2f3b55f6c
--- /dev/null
+++ b/InvenTree/part/migrations/0012_part_active.py
@@ -0,0 +1,18 @@
+# Generated by Django 2.2 on 2019-04-28 13:00
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('part', '0011_auto_20190428_0841'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='part',
+            name='active',
+            field=models.BooleanField(default=True, help_text='Is this part active?'),
+        ),
+    ]
diff --git a/InvenTree/part/models.py b/InvenTree/part/models.py
index 887c983511..04d5471f92 100644
--- a/InvenTree/part/models.py
+++ b/InvenTree/part/models.py
@@ -171,6 +171,9 @@ class Part(models.Model):
     # Can this part be sold to customers?
     salable = models.BooleanField(default=False, help_text="Can this part be sold to customers?")
 
+    # Is this part active?
+    active = models.BooleanField(default=True, help_text='Is this part active?')
+
     notes = models.TextField(blank=True)
 
     def __str__(self):

From 260d89005247d92250c0ea96062cd9c6c1715af4 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 23:08:54 +1000
Subject: [PATCH 2/8] If a part is not active, display an INACTIVE Message on
 part page

---
 InvenTree/part/templates/part/part_base.html | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/InvenTree/part/templates/part/part_base.html b/InvenTree/part/templates/part/part_base.html
index ceda740c16..79be275307 100644
--- a/InvenTree/part/templates/part/part_base.html
+++ b/InvenTree/part/templates/part/part_base.html
@@ -5,6 +5,11 @@
 {% block content %}
 
 <div class="row">
+    {% if part.active == False %}
+    <div class='alert alert-danger' style='display: block;'>
+        This part ({{ part.name }}) is not active:
+    </div>
+    {% endif %}
 <div class="col-sm-6">
     <div class="media">
       <div class="media-left">
@@ -16,7 +21,7 @@
         {% endif %}/>
       </div>
     <div class="media-body">
-        <h4>{{ part.name }}</h4>
+        <h4>{{ part.name }}{% if part.active == False %} <i>- INACTIVE</i>{% endif %}</h4>
         {% if part.description %}
         <p><i>{{ part.description }}</i></p>
         {% endif %}

From 495d414d54b01a71206b4f891729c1944be663b9 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 23:10:34 +1000
Subject: [PATCH 3/8] Hide some options if a part is not active

---
 InvenTree/part/templates/part/build.html  | 2 ++
 InvenTree/part/templates/part/detail.html | 7 ++++++-
 InvenTree/part/templates/part/stock.html  | 2 ++
 3 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/InvenTree/part/templates/part/build.html b/InvenTree/part/templates/part/build.html
index 999fe7e231..84642f4de8 100644
--- a/InvenTree/part/templates/part/build.html
+++ b/InvenTree/part/templates/part/build.html
@@ -7,7 +7,9 @@
 <h3>Part Builds</h3>
 
 <div id='button-toolbar'>
+    {% if part.active %}
     <button class="btn btn-success" id='start-build'>Start New Build</button>
+    {% endif %}
 </div>
 
 <table class='table table-striped table-condensed' data-toolbar='#button-toolbar' id='build-table'>
diff --git a/InvenTree/part/templates/part/detail.html b/InvenTree/part/templates/part/detail.html
index f0bdc49df2..2a8db4421a 100644
--- a/InvenTree/part/templates/part/detail.html
+++ b/InvenTree/part/templates/part/detail.html
@@ -17,7 +17,12 @@
                   <li><a href='#' id='duplicate-part' title='Duplicate Part'>Duplicate</a></li>
                   <li><a href="#" id='edit-part' title='Edit part'>Edit</a></li>
                   <li><a href='#' id='stocktake-part' title='Stocktake'>Stocktake</a></li>
-                  <li><a href="#" id='delete-part' title='Delete part'>Delete</a></li>
+                  <hr>
+                  {% if part.active %}
+                  <li><a href="#" id='deactivate-part' title='Deactivate part'>Deactivate</a></li>
+                  {% else %}
+                  <li><a href="#" id='activate-part' title='Activate part'>Activate</a></li>
+                  {% endif %}
                 </ul>
             </div>
         </h3>
diff --git a/InvenTree/part/templates/part/stock.html b/InvenTree/part/templates/part/stock.html
index a821652aae..fb620c1b7b 100644
--- a/InvenTree/part/templates/part/stock.html
+++ b/InvenTree/part/templates/part/stock.html
@@ -7,7 +7,9 @@
 <h3>Part Stock</h3>
 
 <div id='button-toolbar'>
+    {% if part.active %}
     <button class='btn btn-success' id='add-stock-item'>New Stock Item</button>
+    {% endif %}
     <div id='opt-dropdown' class="dropdown" style='float: right;'>
         <button id='stock-options' class="btn btn-primary dropdown-toggle" type="button" data-toggle="dropdown">Options
             <span class="caret"></span></button>

From 1a66a12d57e6d4a90d95ed68c4404a4d22ece6ab Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 23:15:30 +1000
Subject: [PATCH 4/8] Enable Part API to filter by active status

---
 InvenTree/part/api.py         | 2 ++
 InvenTree/part/serializers.py | 1 +
 2 files changed, 3 insertions(+)

diff --git a/InvenTree/part/api.py b/InvenTree/part/api.py
index 46a15cfc62..379c19c260 100644
--- a/InvenTree/part/api.py
+++ b/InvenTree/part/api.py
@@ -130,11 +130,13 @@ class PartList(generics.ListCreateAPIView):
     ]
 
     filter_fields = [
+        'category',
         'buildable',
         'consumable',
         'trackable',
         'purchaseable',
         'salable',
+        'active',
     ]
 
     ordering_fields = [
diff --git a/InvenTree/part/serializers.py b/InvenTree/part/serializers.py
index 0344e46fe7..c8130055b7 100644
--- a/InvenTree/part/serializers.py
+++ b/InvenTree/part/serializers.py
@@ -70,6 +70,7 @@ class PartSerializer(serializers.ModelSerializer):
             'consumable',
             'trackable',
             'salable',
+            'active',
         ]
 
 

From 492d705b39c2b4944757d169e84c36c184694cf0 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 23:22:54 +1000
Subject: [PATCH 5/8] Hide inactive parts in parts list

---
 InvenTree/part/templates/part/category.html | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/InvenTree/part/templates/part/category.html b/InvenTree/part/templates/part/category.html
index cc10828f80..3d514389bc 100644
--- a/InvenTree/part/templates/part/category.html
+++ b/InvenTree/part/templates/part/category.html
@@ -115,16 +115,15 @@
         method: 'get',
         pagination: true,
         rememberOrder: true,
-        {% if category %}
         queryParams: function(p) {
             return {
+                active: true,
                 {% if category %}
                 category: {{ category.id }},
                 include_child_categories: true,
                 {% endif %}
             }
         },
-        {% endif %}
         columns: [
             {
                 checkbox: true,
@@ -160,7 +159,7 @@
                     else {
                         return '';
                     }
-                }
+                }   
             },
             {
                 field: 'total_stock',

From 73f672495df4e96ad2dceb2c80624de1e497d33a Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 23:40:26 +1000
Subject: [PATCH 6/8] Use API to mark part as active/inactive

---
 InvenTree/part/serializers.py             |  1 +
 InvenTree/part/templates/part/detail.html | 27 +++++++++++++++++++++++
 InvenTree/static/script/inventree/api.js  | 17 +++++++++-----
 3 files changed, 39 insertions(+), 6 deletions(-)

diff --git a/InvenTree/part/serializers.py b/InvenTree/part/serializers.py
index c8130055b7..e5c23b730a 100644
--- a/InvenTree/part/serializers.py
+++ b/InvenTree/part/serializers.py
@@ -53,6 +53,7 @@ class PartSerializer(serializers.ModelSerializer):
 
     class Meta:
         model = Part
+        partial = True
         fields = [
             'pk',
             'url',  # Link to the part detail page
diff --git a/InvenTree/part/templates/part/detail.html b/InvenTree/part/templates/part/detail.html
index 2a8db4421a..7b148318c9 100644
--- a/InvenTree/part/templates/part/detail.html
+++ b/InvenTree/part/templates/part/detail.html
@@ -146,6 +146,33 @@
                         });
     });
 
+    $('#activate-part').click(function() {
+        inventreeUpdate(
+            "{% url 'api-part-detail' part.id %}",
+            {
+                active: true,
+            },
+            {
+                method: 'PATCH',
+                reloadOnSuccess: true,
+            }
+        );
+    });
+
+    $('#deactivate-part').click(function() {
+        inventreeUpdate(
+            "{% url 'api-part-detail' part.id %}",
+            {
+                active: false,
+            },
+            {
+                method: 'PATCH',
+                reloadOnSuccess: true,
+            }
+        );
+    });
+
+
     $('#delete-part').click(function() {
         launchDeleteForm(
                          "{% url 'part-delete' part.id %}",
diff --git a/InvenTree/static/script/inventree/api.js b/InvenTree/static/script/inventree/api.js
index fab1450a00..530e816f68 100644
--- a/InvenTree/static/script/inventree/api.js
+++ b/InvenTree/static/script/inventree/api.js
@@ -47,11 +47,7 @@ function inventreeUpdate(url, data={}, options={}) {
         data["_is_final"] = true;
     }
 
-    var method = 'put';
-
-    if ('method' in options) {
-        method = options.method;
-    }
+    var method = options.method || 'PUT';
 
     // Middleware token required for data update
     //var csrftoken = jQuery("[name=csrfmiddlewaretoken]").val();
@@ -69,12 +65,21 @@ function inventreeUpdate(url, data={}, options={}) {
         success: function(response, status) {
             response['_status_code'] = status;
             console.log('UPDATE object to ' + url + ' - result = ' + status);
+            if (options.success) {
+                options.success(response, status);
+            }
+            if (options.reloadOnSuccess) {
+                location.reload();
+            }
         },
         error: function(xhr, ajaxOptions, thrownError) {
             console.error('Error on UPDATE to ' + url);
             console.error(thrownError);
+            if (options.error) {
+                options.error(xhr, ajaxOptions, thrownError);
             }
-        });
+        }
+    });
 }
 
 // Return list of parts with optional filters

From 6612e91aec217c0284ce1bf3a8db01675a1d1497 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Sun, 28 Apr 2019 23:41:47 +1000
Subject: [PATCH 7/8] Remove ability to delete part via API

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

diff --git a/InvenTree/part/api.py b/InvenTree/part/api.py
index 379c19c260..774c3e18f7 100644
--- a/InvenTree/part/api.py
+++ b/InvenTree/part/api.py
@@ -70,7 +70,7 @@ class CategoryDetail(generics.RetrieveUpdateDestroyAPIView):
     queryset = PartCategory.objects.all()
 
 
-class PartDetail(generics.RetrieveUpdateDestroyAPIView):
+class PartDetail(generics.RetrieveUpdateAPIView):
     """ API endpoint for detail view of a single Part object """
     queryset = Part.objects.all()
     serializer_class = PartSerializer

From 07fbaa9dd9b89965682158fe49db08b3852a3923 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Mon, 29 Apr 2019 00:01:57 +1000
Subject: [PATCH 8/8] Test fix

- Part list API handles 'category' filtering a bit differently
- Probably need to address this
---
 InvenTree/part/api.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/InvenTree/part/api.py b/InvenTree/part/api.py
index 774c3e18f7..27a5b24894 100644
--- a/InvenTree/part/api.py
+++ b/InvenTree/part/api.py
@@ -130,7 +130,6 @@ class PartList(generics.ListCreateAPIView):
     ]
 
     filter_fields = [
-        'category',
         'buildable',
         'consumable',
         'trackable',