From 0cc2e7184349521126c029b2dce85a4453f1cca8 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 19:00:59 +1000
Subject: [PATCH 01/15] Provide an after_render callback

- User callback after the form data are rendered
---
 InvenTree/static/script/inventree/modals.js | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/InvenTree/static/script/inventree/modals.js b/InvenTree/static/script/inventree/modals.js
index 67d4497674..6c6c449fb3 100644
--- a/InvenTree/static/script/inventree/modals.js
+++ b/InvenTree/static/script/inventree/modals.js
@@ -401,6 +401,10 @@ function handleModalForm(url, options) {
                     else {
                         if (response.html_form) {
                             injectModalForm(modal, response.html_form);
+
+                            if (options.after_render) {
+                                options.after_render(modal, response);
+                            }
                         }
                         else {
                             $(modal).modal('hide');
@@ -443,6 +447,7 @@ function launchModalForm(url, options = {}) {
      * submit_text - Text for the submit button (default = 'Submit')
      * close_text - Text for the close button (default = 'Close')
      * no_post - If true, only display form data, hide submit button, and disallow POST
+     * after_render - Callback function to run after form is rendered
      */
 
     var modal = options.modal || '#modal-form';
@@ -475,6 +480,10 @@ function launchModalForm(url, options = {}) {
             if (response.html_form) {
                 injectModalForm(modal, response.html_form);
 
+                if (options.after_render) {
+                    options.after_render(modal, response);
+                }
+
                 if (options.no_post) {
                     modalShowSubmitButton(modal, false);
                 } else {

From a96cd2240cc8efd9d378808ba02f1bbbbe8e2432 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 22:12:22 +1000
Subject: [PATCH 02/15] Function to insert a 'create new' button into a form

- Look for a label like <label for="id_<field>">
---
 InvenTree/static/css/inventree.css          | 10 ++++++++++
 InvenTree/static/script/inventree/modals.js | 12 ++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/InvenTree/static/css/inventree.css b/InvenTree/static/css/inventree.css
index e096a94994..a603513cee 100644
--- a/InvenTree/static/css/inventree.css
+++ b/InvenTree/static/css/inventree.css
@@ -192,6 +192,16 @@
     width: 100%;
 }
 
+/* Force a control-label div to be 100% width */
+.modal .control-label {
+    width: 100%;
+}
+
+.modal .control-label .btn {
+    padding-top: 3px;
+    padding-bottom: 3px;
+}
+
 /* The side navigation menu */
 .sidenav {
     height: 100%; /* 100% Full-height */
diff --git a/InvenTree/static/script/inventree/modals.js b/InvenTree/static/script/inventree/modals.js
index 6c6c449fb3..e4a7ed32fb 100644
--- a/InvenTree/static/script/inventree/modals.js
+++ b/InvenTree/static/script/inventree/modals.js
@@ -354,6 +354,18 @@ function injectModalForm(modal, form_html) {
 }
 
 
+function insertNewItemButton(modal, field, text) {
+
+    var html = "<span style='float: right;'>";
+
+    html += "<div type='button' class='btn btn-primary' id='btn-new-" + field + "'>" + text + "</div>";
+
+    html += "</span>";
+
+    $(modal).find('label[for="id_'+ field + '"]').append(html);
+}
+
+
 function handleModalForm(url, options) {
     /* Update a modal form after data are received from the server.
      * Manages POST requests until the form is successfully submitted.

From dc3abec899f3f2da8f0f4187dd67aefdb398a241 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 22:30:24 +1000
Subject: [PATCH 03/15] Add a secondary modal in modals.html

---
 InvenTree/static/css/inventree.css            | 14 ++++++--
 InvenTree/static/script/inventree/modals.js   | 32 +++++++++++++++--
 InvenTree/static/script/inventree/stock.js    | 26 --------------
 InvenTree/stock/templates/stock/location.html | 19 ++++++++++
 InvenTree/templates/modals.html               | 36 +++++++++----------
 5 files changed, 76 insertions(+), 51 deletions(-)

diff --git a/InvenTree/static/css/inventree.css b/InvenTree/static/css/inventree.css
index a603513cee..1658113edb 100644
--- a/InvenTree/static/css/inventree.css
+++ b/InvenTree/static/css/inventree.css
@@ -51,7 +51,7 @@
     background: #eee;
     display: none;
     position: absolute;
-    z-index: 999;
+    z-index: 400;
     border: 1px solid #555;
     max-width: 250px;
 }
@@ -159,7 +159,15 @@
 
 .modal {
   overflow: hidden;
-  z-index: 99999999;
+  z-index: 9999;
+}
+
+.modal-primary {
+    z-index: 10000;
+}
+
+.modal-secondary {
+    z-index: 11000;
 }
 
 .js-modal-form .checkbox {
@@ -227,7 +235,7 @@
     margin-top: 20px;
     width: 100%;
     padding: 20px;
-    z-index: 999999;
+    z-index: 5000;
     pointer-events: none; // Prevent this div from blocking links underneath
 }
 
diff --git a/InvenTree/static/script/inventree/modals.js b/InvenTree/static/script/inventree/modals.js
index e4a7ed32fb..8a2da5ff9c 100644
--- a/InvenTree/static/script/inventree/modals.js
+++ b/InvenTree/static/script/inventree/modals.js
@@ -354,15 +354,41 @@ function injectModalForm(modal, form_html) {
 }
 
 
-function insertNewItemButton(modal, field, text) {
+function insertNewItemButton(modal, options) {
+    /* Insert a button into a modal form, after a field label.
+     * Looks for a <label> tag inside the form with the attribute "for='id_<field>'"
+     * Inserts a button at the end of this lael element.
+     */
 
     var html = "<span style='float: right;'>";
 
-    html += "<div type='button' class='btn btn-primary' id='btn-new-" + field + "'>" + text + "</div>";
+    html += "<div type='button' class='btn btn-primary'";
+    
+    if (options.title) {
+        html += " title='" + options.title + "'";
+    }
+
+    html += " id='btn-new-" + options.field + "'>" + options.label + "</div>";
 
     html += "</span>";
 
-    $(modal).find('label[for="id_'+ field + '"]').append(html);
+    $(modal).find('label[for="id_'+ options.field + '"]').append(html);
+}
+
+
+function attachSecondaryModal(modal, options) {
+    /* Attach a secondary modal form to the primary modal form.
+     * Inserts a button into the primary form which, when clicked,
+     * will launch the secondary modal to do /something/ and then return a result.
+     * 
+     * options:
+     *  field: Name of the field to attach to
+     *  label: Button text
+     *  title: Hover text to display over button (optional)
+     */
+
+    // Insert the button
+    insertNewItemButton(modal, options);
 }
 
 
diff --git a/InvenTree/static/script/inventree/stock.js b/InvenTree/static/script/inventree/stock.js
index 222bca8f20..9f9c5fd40b 100644
--- a/InvenTree/static/script/inventree/stock.js
+++ b/InvenTree/static/script/inventree/stock.js
@@ -345,32 +345,6 @@ function moveStockItems(items, options) {
     });
 }
 
-function deleteStockItems(items, options) {
-
-    var modal = '#modal-delete';
-
-    if ('modal' in options) {
-        modal = options.modal;
-    }
-
-    if (items.length == 0) {
-        alert('No stock items selected');
-        return;
-    }
-
-    function doDelete(parts) {
-        //TODO
-    }
-
-    openModal({
-        modal: modal,
-        title: 'Delete ' + items.length + ' stock items'
-    });
-}
-
-
-
-
 function loadStockTable(table, options) {
 
     table.bootstrapTable({
diff --git a/InvenTree/stock/templates/stock/location.html b/InvenTree/stock/templates/stock/location.html
index c0212f2c5f..d3379c32e2 100644
--- a/InvenTree/stock/templates/stock/location.html
+++ b/InvenTree/stock/templates/stock/location.html
@@ -123,9 +123,28 @@
                                 {% if location %}
                                 location: {{ location.id }}
                                 {% endif %}
+                            },
+                            after_render: function(modal, response) {
+
+                                attachSecondaryModal(modal,
+                                    {
+                                        'field': 'part',
+                                        'label': 'New Part',
+                                        'title': 'Create New Part',
+                                    }
+                                );
+
+                                attachSecondaryModal(modal,
+                                    {
+                                        'field': 'location',
+                                        'label': 'New Location',
+                                        'title': 'Create New Location'
+                                    }
+                                );
                             }
                         });
 
+
         return false;
     });
 
diff --git a/InvenTree/templates/modals.html b/InvenTree/templates/modals.html
index 05c8c2a894..9423b14569 100644
--- a/InvenTree/templates/modals.html
+++ b/InvenTree/templates/modals.html
@@ -1,4 +1,4 @@
-<div class='modal fade modal-fixed-footer' tabindex='-1' role='dialog' id='modal-form'>
+<div class='modal fade modal-fixed-footer modal-primary' tabindex='-1' role='dialog' id='modal-form'>
     <div class='modal-dialog'>
         <div class='modal-content'>
             <div class="modal-header">
@@ -17,27 +17,25 @@
     </div>
 </div>
 
-<div class='modal fade modal-fixed-footer' tabindex='-1' role='dialog' id='modal-delete'>
-    <div class='modal-dialog'>
-        <div class='modal-content'>
-            <div class="modal-header">
-              <button type="button" class="close" data-dismiss="modal" aria-label="Close">
-                <span aria-hidden="true">&times;</span>
-              </button>
-              <h3 id='modal-title'>Confirm Item Deletion</h3>
-            </div>
-            <form action='' method='post' id='delete-form'>
-                {% csrf_token %}
-            </form>
-            <div class='modal-form-content'>
-            </div>
-            <div class='modal-footer'>
-                <button type='button' class='btn btn-default' data-dismiss='modal'>Cancel</button>
-                <button type='button' class='btn btn-danger' id='modal-form-delete'>Delete</button>
+<div class='modal fade modal-fixed-footer modal-secondary' tabindex='-1' role='dialog' id='modal-form-secondary'>
+        <div class='modal-dialog'>
+            <div class='modal-content'>
+                <div class="modal-header">
+                  <button type="button" class="close" data-dismiss="modal" aria-label="Close">
+                    <span aria-hidden="true">&times;</span>
+                  </button>
+                  <h3 id='modal-title'>Form Title Here</h3>
+                </div>
+                <div class='modal-form-content'>
+                </div>
+                <div class='modal-footer'>
+                    <button type='button' class='btn btn-default' id='modal-form-close' data-dismiss='modal'>Close</button>
+                    <button type='button' class='btn btn-primary' id='modal-form-submit'>Submit</button>
+                </div>
             </div>
         </div>
     </div>
-</div>
+    
 
 <div class='modal fade modal-fixed-footer' tabindex='-1' role='dialog' id='modal-question-dialog'>
         <div class='modal-dialog'>

From 226644e59ac76cd227a455b84dc40790304d0a12 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 22:37:45 +1000
Subject: [PATCH 04/15] Open a secondary modal over the top of a primary one

---
 InvenTree/static/script/inventree/modals.js   | 14 ++++++++++++++
 InvenTree/stock/templates/stock/location.html | 14 ++++++++------
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/InvenTree/static/script/inventree/modals.js b/InvenTree/static/script/inventree/modals.js
index 8a2da5ff9c..05a0473b42 100644
--- a/InvenTree/static/script/inventree/modals.js
+++ b/InvenTree/static/script/inventree/modals.js
@@ -385,10 +385,24 @@ function attachSecondaryModal(modal, options) {
      *  field: Name of the field to attach to
      *  label: Button text
      *  title: Hover text to display over button (optional)
+     *  url: URL for the secondary modal
+     *  query: Query params for the secondary modal
      */
 
     // Insert the button
     insertNewItemButton(modal, options);
+
+    // Add a callback to the button
+    $(modal).find("#btn-new-" + options.field).on('click', function() {
+
+        // Launch the secondary modal
+        launchModalForm(
+            options.url,
+            {
+                modal: '#modal-form-secondary',
+            }
+        );
+    });
 }
 
 
diff --git a/InvenTree/stock/templates/stock/location.html b/InvenTree/stock/templates/stock/location.html
index d3379c32e2..18aa369d0d 100644
--- a/InvenTree/stock/templates/stock/location.html
+++ b/InvenTree/stock/templates/stock/location.html
@@ -128,17 +128,19 @@
 
                                 attachSecondaryModal(modal,
                                     {
-                                        'field': 'part',
-                                        'label': 'New Part',
-                                        'title': 'Create New Part',
+                                        field: 'part',
+                                        label: 'New Part',
+                                        title: 'Create New Part',
+                                        url: "{% url 'part-create' %}",
                                     }
                                 );
 
                                 attachSecondaryModal(modal,
                                     {
-                                        'field': 'location',
-                                        'label': 'New Location',
-                                        'title': 'Create New Location'
+                                        field: 'location',
+                                        label: 'New Location',
+                                        title: 'Create New Location',
+                                        url: "{% url 'stock-location-create' %}",
                                     }
                                 );
                             }

From 21c238e96a51802486108beb5a541c92b873a197 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 22:44:57 +1000
Subject: [PATCH 05/15] Secondary modal is slightly smaller than primary

- So that the primary modal is still visible underneath
---
 InvenTree/static/css/inventree.css | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/InvenTree/static/css/inventree.css b/InvenTree/static/css/inventree.css
index 1658113edb..bca4560edf 100644
--- a/InvenTree/static/css/inventree.css
+++ b/InvenTree/static/css/inventree.css
@@ -178,6 +178,11 @@
     width: 45%;
 }
 
+.modal-secondary .modal-dialog {
+    width: 40%;
+    padding-top: 15px;
+}
+
 .modal-content h3 {
     margin-top: 3px;
     margin-bottom: 3px;

From a316681ec818f18f274bd47725326c342bb53ae1 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:04:00 +1000
Subject: [PATCH 06/15] Pass object data back to modal when creating new object

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

diff --git a/InvenTree/InvenTree/views.py b/InvenTree/InvenTree/views.py
index 041603f7fb..9d5e1a2dc4 100644
--- a/InvenTree/InvenTree/views.py
+++ b/InvenTree/InvenTree/views.py
@@ -245,6 +245,7 @@ class AjaxCreateView(AjaxMixin, CreateView):
 
             # Return the PK of the newly-created object
             data['pk'] = obj.pk
+            data['text'] = str(obj)
 
             try:
                 data['url'] = obj.get_absolute_url()
diff --git a/InvenTree/part/views.py b/InvenTree/part/views.py
index fef88197f7..55482d3284 100644
--- a/InvenTree/part/views.py
+++ b/InvenTree/part/views.py
@@ -202,6 +202,7 @@ class PartDuplicate(AjaxCreateView):
             part = form.save()
 
             data['pk'] = part.pk
+            data['text'] = str(part)
 
             deep_copy = str2bool(request.POST.get('deep_copy', False))
 
@@ -321,6 +322,7 @@ class PartCreate(AjaxCreateView):
             part = form.save()
 
             data['pk'] = part.pk
+            data['text'] = str(part)
 
             try:
                 data['url'] = part.get_absolute_url()

From be1b458634fdb45ed58f6d7833facddc244fa2e7 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:04:49 +1000
Subject: [PATCH 07/15] Upon successful creation (in a secondary modal) update
 the field in the primary modal

---
 InvenTree/static/script/inventree/modals.js | 16 +++++++++++++++-
 InvenTree/stock/forms.py                    |  2 +-
 2 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/InvenTree/static/script/inventree/modals.js b/InvenTree/static/script/inventree/modals.js
index 05a0473b42..4f9cb70068 100644
--- a/InvenTree/static/script/inventree/modals.js
+++ b/InvenTree/static/script/inventree/modals.js
@@ -102,7 +102,7 @@ function afterForm(response, options) {
 
     // Was a callback provided?
     if (options.success) {
-        options.success();
+        options.success(response);
     }
     else if (options.follow && response.url) {
         window.location.href = response.url;
@@ -400,6 +400,20 @@ function attachSecondaryModal(modal, options) {
             options.url,
             {
                 modal: '#modal-form-secondary',
+                success: function(response) {
+
+                    /* A successful object creation event should return a response which contains:
+                     *  - pk: ID of the newly created object
+                     *  - text: String descriptor of the newly created object
+                     *
+                     * So it is simply a matter of appending and selecting the new object!
+                     */
+
+                    var select = '#id_' + options.field;
+                    var option = new Option(response.text, response.pk, true, true)
+                    
+                    $(modal).find(select).append(option).trigger('change');
+                }
             }
         );
     });
diff --git a/InvenTree/stock/forms.py b/InvenTree/stock/forms.py
index f181a6db88..ee8592bbb9 100644
--- a/InvenTree/stock/forms.py
+++ b/InvenTree/stock/forms.py
@@ -32,8 +32,8 @@ class CreateStockItemForm(HelperForm):
             'part',
             'supplier_part',
             'location',
-            'batch',
             'quantity',
+            'batch',
             'delete_on_deplete',
             'status',
             'notes',

From 97c4e5acbf7569acc79663d79e6d4bcf0406ee2a Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:19:56 +1000
Subject: [PATCH 08/15] Add some secondary buttons to the part-create form

---
 InvenTree/part/templates/part/category.html | 36 ++++++++++++++++-----
 1 file changed, 28 insertions(+), 8 deletions(-)

diff --git a/InvenTree/part/templates/part/category.html b/InvenTree/part/templates/part/category.html
index 286b5fc418..82cf5e5833 100644
--- a/InvenTree/part/templates/part/category.html
+++ b/InvenTree/part/templates/part/category.html
@@ -82,15 +82,35 @@
 
     $("#part-create").click(function() {
         launchModalForm(
-                        "{% url 'part-create' %}",
+            "{% url 'part-create' %}",
+            {
+                follow: true,
+                data: {
+                {% if category %}
+                    category: {{ category.id }}
+                {% endif %}
+                },
+                after_render: function(modal, response) {
+                    attachSecondaryModal(modal,
                         {
-                            follow: true,
-                            data: {
-                            {% if category %}
-                                category: {{ category.id }}
-                            {% endif %}
-                            }
-                        });
+                            field: 'category',
+                            label: 'New Category',
+                            title: 'Create new Part Category',
+                            url: "{% url 'category-create' %}",
+                        }
+                    );
+
+                    attachSecondaryModal(modal,
+                        {
+                            field: 'default_location',
+                            label: 'New Location',
+                            title: 'Create new Stock Location',
+                            url: "{% url 'stock-location-create' %}",
+                        }
+                    );
+                }   
+            }
+        );
     });
 
     {% if category %}

From 9d50f2a6acaa03e6de1c84da7962102309e6a867 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:27:45 +1000
Subject: [PATCH 09/15] Toot toot all aboard the refactor tractor

- launchModalForm now accepts a 'secondary' parameter which is a list of secondary modals to attach
---
 InvenTree/part/templates/part/category.html   | 33 ++++++++----------
 InvenTree/static/script/inventree/modals.js   | 18 ++++++++++
 InvenTree/stock/templates/stock/location.html | 34 ++++++++-----------
 3 files changed, 46 insertions(+), 39 deletions(-)

diff --git a/InvenTree/part/templates/part/category.html b/InvenTree/part/templates/part/category.html
index 82cf5e5833..44a767b1d9 100644
--- a/InvenTree/part/templates/part/category.html
+++ b/InvenTree/part/templates/part/category.html
@@ -90,25 +90,20 @@
                     category: {{ category.id }}
                 {% endif %}
                 },
-                after_render: function(modal, response) {
-                    attachSecondaryModal(modal,
-                        {
-                            field: 'category',
-                            label: 'New Category',
-                            title: 'Create new Part Category',
-                            url: "{% url 'category-create' %}",
-                        }
-                    );
-
-                    attachSecondaryModal(modal,
-                        {
-                            field: 'default_location',
-                            label: 'New Location',
-                            title: 'Create new Stock Location',
-                            url: "{% url 'stock-location-create' %}",
-                        }
-                    );
-                }   
+                secondary: [
+                    {
+                        field: 'category',
+                        label: 'New Category',
+                        title: 'Create new Part Category',
+                        url: "{% url 'category-create' %}",
+                    },
+                    {
+                        field: 'default_location',
+                        label: 'New Location',
+                        title: 'Create new Stock Location',
+                        url: "{% url 'stock-location-create' %}",
+                    }
+                ]   
             }
         );
     });
diff --git a/InvenTree/static/script/inventree/modals.js b/InvenTree/static/script/inventree/modals.js
index 4f9cb70068..afb14ace9f 100644
--- a/InvenTree/static/script/inventree/modals.js
+++ b/InvenTree/static/script/inventree/modals.js
@@ -420,6 +420,15 @@ function attachSecondaryModal(modal, options) {
 }
 
 
+function attachSecondaries(modal, secondaries) {
+    /* Attach a provided list of secondary modals */
+
+    for (var i = 0; i < secondaries.length; i++) {
+        attachSecondaryModal(modal, secondaries[i]);
+    }
+}
+
+
 function handleModalForm(url, options) {
     /* Update a modal form after data are received from the server.
      * Manages POST requests until the form is successfully submitted.
@@ -471,6 +480,10 @@ function handleModalForm(url, options) {
                             if (options.after_render) {
                                 options.after_render(modal, response);
                             }
+
+                            if (options.secondary) {
+                                attachSecondaries(modal, options.secondary);
+                            }
                         }
                         else {
                             $(modal).modal('hide');
@@ -514,6 +527,7 @@ function launchModalForm(url, options = {}) {
      * close_text - Text for the close button (default = 'Close')
      * no_post - If true, only display form data, hide submit button, and disallow POST
      * after_render - Callback function to run after form is rendered
+     * secondary - List of secondary modals to attach
      */
 
     var modal = options.modal || '#modal-form';
@@ -550,6 +564,10 @@ function launchModalForm(url, options = {}) {
                     options.after_render(modal, response);
                 }
 
+                if (options.secondary) {
+                    attachSecondaries(modal, options.secondary);
+                }
+
                 if (options.no_post) {
                     modalShowSubmitButton(modal, false);
                 } else {
diff --git a/InvenTree/stock/templates/stock/location.html b/InvenTree/stock/templates/stock/location.html
index 18aa369d0d..786298d407 100644
--- a/InvenTree/stock/templates/stock/location.html
+++ b/InvenTree/stock/templates/stock/location.html
@@ -124,26 +124,20 @@
                                 location: {{ location.id }}
                                 {% endif %}
                             },
-                            after_render: function(modal, response) {
-
-                                attachSecondaryModal(modal,
-                                    {
-                                        field: 'part',
-                                        label: 'New Part',
-                                        title: 'Create New Part',
-                                        url: "{% url 'part-create' %}",
-                                    }
-                                );
-
-                                attachSecondaryModal(modal,
-                                    {
-                                        field: 'location',
-                                        label: 'New Location',
-                                        title: 'Create New Location',
-                                        url: "{% url 'stock-location-create' %}",
-                                    }
-                                );
-                            }
+                            secondary: [
+                                {
+                                    field: 'part',
+                                    label: 'New Part',
+                                    title: 'Create New Part',
+                                    url: "{% url 'part-create' %}",
+                                },
+                                {
+                                    field: 'location',
+                                    label: 'New Location',
+                                    title: 'Create New Location',
+                                    url: "{% url 'stock-location-create' %}",
+                                }
+                            ]
                         });
 
 

From 76e6094977a21f969bd6f3f39cd50fca2a7c8d24 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:30:32 +1000
Subject: [PATCH 10/15] Add secondary modal to BOM

---
 InvenTree/part/templates/part/bom.html | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/InvenTree/part/templates/part/bom.html b/InvenTree/part/templates/part/bom.html
index 295fed23d6..e75efdfe77 100644
--- a/InvenTree/part/templates/part/bom.html
+++ b/InvenTree/part/templates/part/bom.html
@@ -83,7 +83,15 @@
             {
                 success: function() {
                     $("#bom-table").bootstrapTable('refresh');
-                }
+                },
+                secondary: [
+                    {
+                        field: 'sub_part',
+                        label: 'New Part',
+                        title: 'Create New Part',
+                        url: "{% url 'part-create' %}",
+                    },
+                ]
             }
         );
     });

From 505b7941b116a789ae3b984e512f7cd87e097b9e Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:33:25 +1000
Subject: [PATCH 11/15] Add link to delete a part

- Part must first be deactivated
---
 InvenTree/part/templates/part/detail.html         |  9 +++++----
 InvenTree/part/templates/part/partial_delete.html | 12 +++++++++++-
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/InvenTree/part/templates/part/detail.html b/InvenTree/part/templates/part/detail.html
index a7bad6cae4..a3a9736a57 100644
--- a/InvenTree/part/templates/part/detail.html
+++ b/InvenTree/part/templates/part/detail.html
@@ -20,6 +20,7 @@
                 <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>
+                <li><a href='#' id='delete-part' title='Delete part'>Delete</a></li>
                 {% endif %}
             </ul>
         </div>
@@ -200,10 +201,10 @@
 
     $('#delete-part').click(function() {
         launchModalForm(
-                         "{% url 'part-delete' part.id %}",
-                         {
-                             redirect: {% if part.category %}"{% url 'category-detail' part.category.id %}"{% else %}"{% url 'part-index' %}"{% endif %}
-                         });
+            "{% url 'part-delete' part.id %}",
+            {
+                redirect: {% if part.category %}"{% url 'category-detail' part.category.id %}"{% else %}"{% url 'part-index' %}"{% endif %}
+            });
     });
 
 {% endblock %}
diff --git a/InvenTree/part/templates/part/partial_delete.html b/InvenTree/part/templates/part/partial_delete.html
index 6f2509996b..baa429e2a5 100644
--- a/InvenTree/part/templates/part/partial_delete.html
+++ b/InvenTree/part/templates/part/partial_delete.html
@@ -1,6 +1,11 @@
+{% extends "modal_form.html" %}
+
+{% block pre_form_content %}
+
 Are you sure you want to delete part '{{ part.full_name }}'?
 
 {% if part.used_in_count %}
+<hr>
 <p>This part is used in BOMs for {{ part.used_in_count }} other parts. If you delete this part, the BOMs for the following parts will be updated:
 <ul class="list-group">
     {% for child in part.used_in.all %}
@@ -10,6 +15,7 @@ Are you sure you want to delete part '{{ part.full_name }}'?
 {% endif %}
 
 {% if part.locations.all|length > 0 %}
+<hr>
 <p>There are {{ part.locations.all|length }} stock entries defined for this part. If you delete this part, the following stock entries will also be deleted:
     <ul class='list-group'>
         {% for stock in part.locations.all %}
@@ -20,6 +26,7 @@ Are you sure you want to delete part '{{ part.full_name }}'?
 {% endif %}
 
 {% if part.supplier_parts.all|length > 0 %}
+<hr>
 <p>There are {{ part.supplier_parts.all|length }} suppliers defined for this part. If you delete this part, the following supplier parts will also be deleted.
     <ul class='list-group'>
         {% for spart in part.supplier_parts.all %}
@@ -30,5 +37,8 @@ Are you sure you want to delete part '{{ part.full_name }}'?
 {% endif %}
 
 {% if part.serials.all|length > 0 %}
+<hr>
 <p>There are {{ part.serials.all|length }} unique parts tracked for '{{ part.full_name }}'. Deleting this part will permanently remove this tracking information.</p>
-{% endif %}
\ No newline at end of file
+{% endif %}
+
+{% endblock %}
\ No newline at end of file

From 0ffe2666b376eb10719909a4289bc4b6e2bca55c Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:36:22 +1000
Subject: [PATCH 12/15] Added some missing help text

---
 InvenTree/part/models.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/InvenTree/part/models.py b/InvenTree/part/models.py
index 3aeb94700a..d255a319f3 100644
--- a/InvenTree/part/models.py
+++ b/InvenTree/part/models.py
@@ -670,6 +670,7 @@ class BomItem(models.Model):
     # A link to the parent part
     # Each part will get a reverse lookup field 'bom_items'
     part = models.ForeignKey(Part, on_delete=models.CASCADE, related_name='bom_items',
+                             help_text='Select parent part',
                              limit_choices_to={
                                  'buildable': True,
                                  'active': True,
@@ -678,16 +679,17 @@ class BomItem(models.Model):
     # A link to the child item (sub-part)
     # Each part will get a reverse lookup field 'used_in'
     sub_part = models.ForeignKey(Part, on_delete=models.CASCADE, related_name='used_in',
+                                 help_text='Select part to be used in BOM',
                                  limit_choices_to={
                                      'consumable': True,
                                      'active': True
                                  })
 
     # Quantity required
-    quantity = models.PositiveIntegerField(default=1, validators=[MinValueValidator(0)])
+    quantity = models.PositiveIntegerField(default=1, validators=[MinValueValidator(0)], help_text='BOM quantity for this BOM item')
 
     # Note attached to this BOM line item
-    note = models.CharField(max_length=100, blank=True, help_text='Item notes')
+    note = models.CharField(max_length=100, blank=True, help_text='BOM item notes')
 
     def clean(self):
         """ Check validity of the BomItem model.
@@ -767,7 +769,7 @@ class SupplierPart(models.Model):
 
     MPN = models.CharField(max_length=100, blank=True, help_text='Manufacturer part number')
 
-    URL = models.URLField(blank=True)
+    URL = models.URLField(blank=True, help_text='URL for external supplier part link')
 
     description = models.CharField(max_length=250, blank=True, help_text='Supplier part description')
 

From 58cc1cda77407f718e4d451ccda7d7a824fa0de6 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:43:16 +1000
Subject: [PATCH 13/15] Only load api.js once

---
 InvenTree/build/templates/build/allocate.html       | 1 -
 InvenTree/part/templates/part/bom.html              | 1 -
 InvenTree/part/templates/part/category.html         | 1 -
 InvenTree/part/templates/part/detail.html           | 1 -
 InvenTree/part/templates/part/stock.html            | 1 -
 InvenTree/stock/templates/stock/location.html       | 1 -
 InvenTree/stock/templates/stock/stock_app_base.html | 1 -
 7 files changed, 7 deletions(-)

diff --git a/InvenTree/build/templates/build/allocate.html b/InvenTree/build/templates/build/allocate.html
index 9a0f045c68..5b66ef6b57 100644
--- a/InvenTree/build/templates/build/allocate.html
+++ b/InvenTree/build/templates/build/allocate.html
@@ -34,7 +34,6 @@ InvenTree | Allocate Parts
 
 {% block js_load %}
 {{ block.super }}
-<script src="{% static 'script/inventree/api.js' %}"></script>
 <script src="{% static 'script/inventree/part.js' %}"></script>
 <script src="{% static 'script/inventree/build.js' %}"></script>
 {% endblock %}
diff --git a/InvenTree/part/templates/part/bom.html b/InvenTree/part/templates/part/bom.html
index e75efdfe77..c7648cfd33 100644
--- a/InvenTree/part/templates/part/bom.html
+++ b/InvenTree/part/templates/part/bom.html
@@ -56,7 +56,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/api.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/part.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/bom.js' %}"></script>
 {% endblock %}
diff --git a/InvenTree/part/templates/part/category.html b/InvenTree/part/templates/part/category.html
index 44a767b1d9..c9f6aef94d 100644
--- a/InvenTree/part/templates/part/category.html
+++ b/InvenTree/part/templates/part/category.html
@@ -61,7 +61,6 @@
 {% endblock %}
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/api.js' %}"></script>
 <script type='text/javacript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 {% block js_ready %}
diff --git a/InvenTree/part/templates/part/detail.html b/InvenTree/part/templates/part/detail.html
index a3a9736a57..b06ab54a52 100644
--- a/InvenTree/part/templates/part/detail.html
+++ b/InvenTree/part/templates/part/detail.html
@@ -129,7 +129,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/api.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 
diff --git a/InvenTree/part/templates/part/stock.html b/InvenTree/part/templates/part/stock.html
index a329a25249..0a9747bc7c 100644
--- a/InvenTree/part/templates/part/stock.html
+++ b/InvenTree/part/templates/part/stock.html
@@ -37,7 +37,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/api.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 {% block js_ready %}
diff --git a/InvenTree/stock/templates/stock/location.html b/InvenTree/stock/templates/stock/location.html
index 786298d407..df58296b23 100644
--- a/InvenTree/stock/templates/stock/location.html
+++ b/InvenTree/stock/templates/stock/location.html
@@ -69,7 +69,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/api.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 {% block js_ready %}
diff --git a/InvenTree/stock/templates/stock/stock_app_base.html b/InvenTree/stock/templates/stock/stock_app_base.html
index 805147f6ad..b7bf307118 100644
--- a/InvenTree/stock/templates/stock/stock_app_base.html
+++ b/InvenTree/stock/templates/stock/stock_app_base.html
@@ -25,7 +25,6 @@ InvenTree | Stock
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/api.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 

From 2e5b3bfdf982fbbe7cf005b1d3a00984f6b9dd90 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:44:33 +1000
Subject: [PATCH 14/15] Prevent multiple loads of part.js

---
 InvenTree/part/templates/part/bom.html | 1 -
 InvenTree/templates/base.html          | 3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/InvenTree/part/templates/part/bom.html b/InvenTree/part/templates/part/bom.html
index c7648cfd33..3990bfab6c 100644
--- a/InvenTree/part/templates/part/bom.html
+++ b/InvenTree/part/templates/part/bom.html
@@ -56,7 +56,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/part.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/bom.js' %}"></script>
 {% endblock %}
 
diff --git a/InvenTree/templates/base.html b/InvenTree/templates/base.html
index 32a1502733..dd2f8bd706 100644
--- a/InvenTree/templates/base.html
+++ b/InvenTree/templates/base.html
@@ -95,10 +95,9 @@ InvenTree
 
 <script type='text/javascript' src="{% static 'script/inventree/inventree.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/api.js' %}"></script>
-<script type='text/javascript' src="{% static 'script/inventree/part.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/tables.js' %}"></script>
-<script type='text/javascript' src="{% static 'script/inventree/notification.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/modals.js' %}"></script>
+<script type='text/javascript' src="{% static 'script/inventree/notification.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/inventree/sidenav.js' %}"></script>
 {% block js_load %}
 {% endblock %}

From f979edef8e159e0f6dc6ef2fe7b3231059909198 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Tue, 14 May 2019 23:47:15 +1000
Subject: [PATCH 15/15] Prevent double loading of stock.js

---
 InvenTree/part/templates/part/category.html      | 1 -
 InvenTree/part/templates/part/detail.html        | 1 -
 InvenTree/part/templates/part/part_app_base.html | 1 +
 InvenTree/part/templates/part/stock.html         | 1 -
 InvenTree/stock/templates/stock/location.html    | 1 -
 5 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/InvenTree/part/templates/part/category.html b/InvenTree/part/templates/part/category.html
index c9f6aef94d..3abef26af9 100644
--- a/InvenTree/part/templates/part/category.html
+++ b/InvenTree/part/templates/part/category.html
@@ -61,7 +61,6 @@
 {% endblock %}
 {% block js_load %}
 {{ block.super }}
-<script type='text/javacript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 {% block js_ready %}
 {{ block.super }}
diff --git a/InvenTree/part/templates/part/detail.html b/InvenTree/part/templates/part/detail.html
index b06ab54a52..787e071036 100644
--- a/InvenTree/part/templates/part/detail.html
+++ b/InvenTree/part/templates/part/detail.html
@@ -129,7 +129,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 
 {% block js_ready %}
diff --git a/InvenTree/part/templates/part/part_app_base.html b/InvenTree/part/templates/part/part_app_base.html
index 95c3f26c28..adf9d67617 100644
--- a/InvenTree/part/templates/part/part_app_base.html
+++ b/InvenTree/part/templates/part/part_app_base.html
@@ -29,6 +29,7 @@ InvenTree | Part List
 {% block js_load %}
 {{ block.super }}
 <script type='text/javascript' src="{% static 'script/inventree/part.js' %}"></script>
+<script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 
 {% block js_ready %}
diff --git a/InvenTree/part/templates/part/stock.html b/InvenTree/part/templates/part/stock.html
index 0a9747bc7c..3311dafb8d 100644
--- a/InvenTree/part/templates/part/stock.html
+++ b/InvenTree/part/templates/part/stock.html
@@ -37,7 +37,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 {% block js_ready %}
 {{ block.super }}
diff --git a/InvenTree/stock/templates/stock/location.html b/InvenTree/stock/templates/stock/location.html
index df58296b23..f05b14c77b 100644
--- a/InvenTree/stock/templates/stock/location.html
+++ b/InvenTree/stock/templates/stock/location.html
@@ -69,7 +69,6 @@
 
 {% block js_load %}
 {{ block.super }}
-<script type='text/javascript' src="{% static 'script/inventree/stock.js' %}"></script>
 {% endblock %}
 {% block js_ready %}
 {{ block.super }}