From 4f63d1283765caa36884df0ada16b753b3ab73ca Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Fri, 4 May 2018 18:53:39 +1000
Subject: [PATCH] Consolidated 'Part' app views

- Improved templating
- Part tree open/close status stored to session
---
 InvenTree/part/templates/part/cat_link.html   |   1 +
 InvenTree/part/templates/part/category.html   | 157 ++++++++++++++++++
 .../part/templates/part/category_detail.html  |  93 -----------
 .../part/templates/part/category_parts.html   |  63 -------
 .../part/category_subcategories.html          |  11 --
 InvenTree/part/templates/part/index.html      |  77 ---------
 .../part/templates/part/part_app_base.html    |  24 ++-
 InvenTree/part/templates/part/part_base.html  |   4 +-
 .../part/templates/part/subcategories.html    |  27 +++
 InvenTree/part/views.py                       |   4 +-
 InvenTree/static/css/inventree.css            |   4 +-
 InvenTree/static/script/sidenav.js            |  43 +++++
 InvenTree/static/script/trees.js              |  20 ---
 InvenTree/templates/base.html                 |  27 +--
 14 files changed, 262 insertions(+), 293 deletions(-)
 create mode 100644 InvenTree/part/templates/part/category.html
 delete mode 100644 InvenTree/part/templates/part/category_detail.html
 delete mode 100644 InvenTree/part/templates/part/category_parts.html
 delete mode 100644 InvenTree/part/templates/part/category_subcategories.html
 delete mode 100644 InvenTree/part/templates/part/index.html
 create mode 100644 InvenTree/part/templates/part/subcategories.html
 delete mode 100644 InvenTree/static/script/trees.js

diff --git a/InvenTree/part/templates/part/cat_link.html b/InvenTree/part/templates/part/cat_link.html
index 02ec88c45c..181f10ebb6 100644
--- a/InvenTree/part/templates/part/cat_link.html
+++ b/InvenTree/part/templates/part/cat_link.html
@@ -1,6 +1,7 @@
 <div class='navigation'>
 <nav aria-label="breadcrumb">
   <ol class="breadcrumb">
+    <li><a href='#' id='toggle-part-tree'><b>+</b></a></li>
     <li class="breadcrumb-item{% if category is None %} active" aria-current="page{% endif %}"><a href="/part/">Parts</a></li>
 {% if category %}
 {% for path_item in category.parentpath %}
diff --git a/InvenTree/part/templates/part/category.html b/InvenTree/part/templates/part/category.html
new file mode 100644
index 0000000000..b287207a51
--- /dev/null
+++ b/InvenTree/part/templates/part/category.html
@@ -0,0 +1,157 @@
+{% extends "part/part_app_base.html" %}
+{% load static %}
+
+{% block content %}
+
+<div class='row'>
+<div class='col-sm-6'>
+    {% if category %}
+    <h3>{{ category.name }}</h3>
+    <p>{{ category.description }}</p>
+    {% else %}
+    <h3>Parts</h3>
+    {% endif %}
+</div>
+<div class='col-sm-6'>
+    <h3>
+    <div style='float: right;'>
+    <button class='btn btn-success' id='cat-create'>New Category</button>
+    {% if category %}
+    <div class="dropdown" style="float: right;">
+        <button class="btn btn-primary dropdown-toggle" type="button" data-toggle="dropdown">
+            Options
+            <span class="caret"></span>
+        </button>
+        <ul class="dropdown-menu">
+          <li><a href="#" id='cat-edit' title='Edit part category'>Edit</a></li>
+          <li><a href="#" id='cat-delete' title='Delete part category'>Delete</a></li>
+        </ul>
+    </div>
+    {% endif %}
+    </div>
+    </h3>
+</div>
+</div>
+
+{% if category %}
+{% include "part/subcategories.html" with children=category.children.all %}
+{% else %}
+{% include "part/subcategories.html" with children=children %}
+{% endif %}
+<hr>
+
+<table class='table table-striped table-condensed' id='part-table'>
+</table>
+
+{% endblock %}
+
+{% block js_ready %}
+{{ block.super }}
+
+    $("#cat-create").click(function() {
+        launchModalForm("#modal-form",
+                        "{% url 'category-create' %}",
+                        {
+                            follow: true,
+                            {% if category %}
+                            data: {
+                                category: {{ category.id }}
+                            }
+                            {% endif %}
+                        });
+    })
+
+    {% if category %}
+    $("#cat-edit").click(function () {
+        launchModalForm("#modal-form",
+                        "{% url 'category-edit' category.id %}",
+                        {
+                            reload: true
+                        },
+                       );
+        return false;
+    });
+
+    {% if category.parent %}
+    var redirect = "{% url 'category-detail' category.parent.id %}";
+    {% else %}
+    var redirect = "{% url 'part-index' %}";
+    {% endif %}
+
+    $('#cat-delete').click(function() {
+        launchDeleteForm('#modal-delete',
+                         "{% url 'category-delete' category.id %}",
+                         {
+                             redirect: redirect
+                         });
+    });
+
+    {% endif %}
+
+    $("#part-table").bootstrapTable({
+        sortable: true,
+        search: true,
+        sortName: 'description',
+        idField: 'pk',
+        method: 'get',
+        pagination: true,
+        rememberOrder: true,
+        {% if category %}
+        queryParams: function(p) {
+            return {
+                {% if category %}
+                category: {{ category.id }},
+                {% endif %}
+            }
+        },
+        {% endif %}
+        columns: [
+            {
+                checkbox: true,
+                title: 'Select',
+                searchable: false,
+            },
+            {
+                field: 'pk',
+                title: 'ID',
+                visible: false,
+            },
+            {
+                field: 'name',
+                title: 'Part',
+                sortable: true,
+                formatter: function(value, row, index, field) {
+                    return renderLink(value, row.url);
+                }
+            },
+            {
+                sortable: true,
+                field: 'description',
+                title: 'Description',
+            },
+            {% if category == None %}
+            {
+                sortable: true,
+                field: 'category',
+                title: 'Category',
+                formatter: function(value, row, index, field) {
+                    if (row.category) {
+                        return renderLink(row.category.name, row.category.url);
+                    }
+                    else {
+                        return '';
+                    }
+                }
+            },
+            {% endif %}
+            {
+                field: 'total_stock',
+                title: 'Stock',
+                searchable: false,
+                sortable: true,
+            }
+        ],
+        url: "{% url 'api-part-list' %}",
+    });
+
+{% endblock %}
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/category_detail.html b/InvenTree/part/templates/part/category_detail.html
deleted file mode 100644
index 2f810f6891..0000000000
--- a/InvenTree/part/templates/part/category_detail.html
+++ /dev/null
@@ -1,93 +0,0 @@
-{% extends "base.html" %}
-
-{% load static %}
-
-{% block content %}
-
-{% include "part/cat_link.html" with category=category %}
-
-<p>
-    <b>{{ category.name }}</b><br>
-    <i>{{ category.description }}</i>
-</p>
-
-{% if category.has_children %}
-<h4>Subcategories</h4>
-{% include "part/category_subcategories.html" with children=category.children.all %}
-{% endif %}
-
-{% if category.has_parts %}
-<table class='table table-striped table-condensed' id='part-table'>
-</table>
-{% endif %}
-
-<div class='container-fluid'>
-    <button type='button' class='btn btn-primary' id='create-cat'>
-        New Category
-    </button>
-    <button class="btn btn-info" id='edit-category'>Edit Category</button>
-    <button class="btn btn-success" id='create-part'>New Part</button>
-    <button class="btn btn-danger" id='delete-category'>Delete Category</button>
-
-    <button class='btn btn-primary' id='get-rows'>Do thing</button>
-</div>
-
-{% include 'modals.html' %}
-
-{% endblock %}
-
-{% block js_load %}
-<script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
-{% endblock %}
-{% block js_ready %}
-
-    $("#edit-category").click(function () {
-        launchModalForm("#modal-form",
-                        "{% url 'category-edit' category.id %}",
-                        {reload: true},
-                       );
-    });
-
-    {% if category.parent %}
-    var categoryRedirect = "{% url 'category-detail' category.parent.id %}";
-    {% else %}
-    var categoryRedirect = "{% url 'part-index' %}";
-    {% endif %}
-
-    $("#delete-category").click(function() {
-        launchDeleteForm("#modal-delete",
-                         "{% url 'category-delete' category.id %}",
-                         {
-                             redirect: categoryRedirect
-                         });
-    });
-
-    $("#create-cat").click(function() {
-        launchModalForm("#modal-form",
-                        "{% url 'category-create' %}",
-                        {
-                            follow: true,
-                            data: {
-                                category: {{ category.id }}
-                            }
-                        });
-    });
-
-    $("#create-part").click( function() {
-        launchModalForm("#modal-form",
-                        "{% url 'part-create' %}",
-                        {
-                            data: {
-                                category: {{ category.id }}
-                            },
-                            follow: true
-                        });
-    });
-
-    {% include "part/category_parts.html" with category=category %}
-
-    $("#get-rows").click( function() {
-        alert($("#part-table").bootstrapTable('getSelections'));
-    });
-
-{% endblock %}
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/category_parts.html b/InvenTree/part/templates/part/category_parts.html
deleted file mode 100644
index f389fee83a..0000000000
--- a/InvenTree/part/templates/part/category_parts.html
+++ /dev/null
@@ -1,63 +0,0 @@
-    $("#part-table").bootstrapTable({
-        sortable: true,
-        search: true,
-        sortName: 'description',
-        idField: 'pk',
-        method: 'get',
-        pagination: true,
-        rememberOrder: true,
-        {% if category %}
-        queryParams: function(p) {
-            return {
-                category: {{ category.id }},
-            }
-        },
-        {% endif %}
-        columns: [
-            {
-                checkbox: true,
-                title: 'Select',
-                searchable: false,
-            },
-            {
-                field: 'pk',
-                title: 'ID',
-                visible: false,
-            },
-            {
-                field: 'name',
-                title: 'Part',
-                sortable: true,
-                formatter: function(value, row, index, field) {
-                    return renderLink(value, row.url);
-                }
-            },
-            {
-                sortable: true,
-                field: 'description',
-                title: 'Description',
-            },
-            {% if category == None %}
-            {
-                sortable: true,
-                field: 'category',
-                title: 'Category',
-                formatter: function(value, row, index, field) {
-                    if (row.category) {
-                        return renderLink(row.category.name, row.category.url);
-                    }
-                    else {
-                        return '';
-                    }
-                }
-            },
-            {% endif %}
-            {
-                field: 'total_stock',
-                title: 'Stock',
-                searchable: false,
-                sortable: true,
-            }
-        ],
-        url: "{% url 'api-part-list' %}",
-    });
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/category_subcategories.html b/InvenTree/part/templates/part/category_subcategories.html
deleted file mode 100644
index 6b7f6e56a2..0000000000
--- a/InvenTree/part/templates/part/category_subcategories.html
+++ /dev/null
@@ -1,11 +0,0 @@
-<ul class="list-group">
-{% for child in children %}
-<li class="list-group-item">
-    <b><a href="{% url 'category-detail' child.id %}">{{ child.name }}</a></b>
-    {% if child.description %}
-    <i> - {{ child.description }}</i>
-    {% endif %}
-    <span class='badge'>{{ child.partcount }}</span>
-</li>
-{% endfor %}
-</ul>
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/index.html b/InvenTree/part/templates/part/index.html
deleted file mode 100644
index f928733eb4..0000000000
--- a/InvenTree/part/templates/part/index.html
+++ /dev/null
@@ -1,77 +0,0 @@
-{% extends "base.html" %}
-{% load static %}
-
-{% block css %}
-<link rel='stylesheet' href="{% static 'css/bootstrap-treeview.css' %}">
-{% endblock %}
-
-{% block content %}
-
-{% include "part/cat_link.html" with category=category %}
-
-{% include 'modals.html' %}
-
-{% if children.all|length > 0 %}
-<h4>Part Categories</h4>
-{% include "part/category_subcategories.html" with children=children %}
-{% endif %}
-
-<table class='table table-striped table-condensed' id='part-table'>
-
-<div class='container-fluid'>
-<button type='button' class='btn btn-primary' id='create-cat'>
-    New Category
-</button>
-<button class="btn btn-success" id='create-part'>New Part</button>
-</div>
-{% endblock %}
-
-{% block js_load %}
-
-<script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
-{% endblock %}
-{% block js_ready %}
-
-    $("#create-cat").click(function() {
-        launchModalForm("#modal-form",
-                        "{% url 'category-create' %}",
-                        {
-                            follow: true
-                        });
-    });
-
-    function loadTree() {
-        var requestData = {};
-
-        {% if category %}
-        requestData.category = {{ category.id }};
-        {% endif %}
-
-        $.ajax({
-            url: "{% url 'api-part-tree' %}",
-            type: 'get',
-            dataType: 'json',
-            data: requestData,
-            success: function (response) {
-                if (response.tree) {
-                    $("#part-tree").treeview({
-                        data: response.tree,
-                        enableLinks: true
-                    });
-                }
-            },
-            error: function (xhr, ajaxOptions, thrownError) {
-                alert('Error retrieving part tree:\n' + thrownError);
-            }
-        });
-    }
-
-    $("#create-part").click(function() {
-        launchModalForm("#modal-form", "{% url 'part-create' %}");
-    });
-
-    {% include "part/category_parts.html" %}
-
-    loadTree();
-
-{% endblock %}
diff --git a/InvenTree/part/templates/part/part_app_base.html b/InvenTree/part/templates/part/part_app_base.html
index 00f42b84a6..30c9b30b6d 100644
--- a/InvenTree/part/templates/part/part_app_base.html
+++ b/InvenTree/part/templates/part/part_app_base.html
@@ -1,6 +1,28 @@
 {% extends "base.html" %}
 
 {% block sidenav %}
-<h3>Parts</h3>
 <div id='part-tree'></div>
+{% endblock %}
+
+{% block pre_content %}
+
+{% if part %}
+{% include "part/cat_link.html" with category=part.category %}
+{% else %}
+{% include 'part/cat_link.html' with category=category %}
+{% endif %}
+
+{% endblock %}
+
+{% block js_ready %}
+{{ block.super }}
+    loadTree("{% url 'api-part-tree' %}",
+             "#part-tree");
+
+    $("#toggle-part-tree").click(function() {
+        toggleSideNav("#sidenav");
+        return false;
+    })
+
+    initSideNav();
 {% endblock %}
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/part_base.html b/InvenTree/part/templates/part/part_base.html
index b6736ed45b..8c622ff12a 100644
--- a/InvenTree/part/templates/part/part_base.html
+++ b/InvenTree/part/templates/part/part_base.html
@@ -4,8 +4,6 @@
 
 {% block content %}
 
-{% include "part/cat_link.html" with category=part.category %}
-
 <div class="row">
 <div class="col-sm-6">
     <div class="media">
@@ -85,7 +83,7 @@
 {% endblock %}
 
 {% block js_ready %}
-
+{{ block.super }}
     $("#part-thumb").click(function() {
         launchModalForm("#modal-form",
                         "{% url 'part-image' part.id %}",
diff --git a/InvenTree/part/templates/part/subcategories.html b/InvenTree/part/templates/part/subcategories.html
new file mode 100644
index 0000000000..4f8704e736
--- /dev/null
+++ b/InvenTree/part/templates/part/subcategories.html
@@ -0,0 +1,27 @@
+{% if children|length > 0 %}
+<hr>
+<div class="panel-group">
+ <div class="panel panel-default">
+   <div class="panel-heading">
+     <h4 class="panel-title">
+       <a data-toggle="collapse" href="#collapse1">Child Categories</a><span class='badge'>{{ children|length }}</span>
+     </h4>
+   </div>
+   <div id="collapse1" class="panel-collapse collapse">
+     <div class="panel-body">
+         <ul class="list-group">
+         {% for child in children %}
+         <li class="list-group-item">
+             <b><a href="{% url 'category-detail' child.id %}">{{ child.name }}</a></b>
+             {% if child.description %}
+             <i> - {{ child.description }}</i>
+             {% endif %}
+             <span class='badge'>{{ child.partcount }}</span>
+         </li>
+         {% endfor %}
+         </ul>
+     </div>
+   </div>
+ </div>
+</div>
+{% endif %}
\ No newline at end of file
diff --git a/InvenTree/part/views.py b/InvenTree/part/views.py
index f11a328af2..9bb42722eb 100644
--- a/InvenTree/part/views.py
+++ b/InvenTree/part/views.py
@@ -23,7 +23,7 @@ from InvenTree.views import AjaxCreateView, AjaxUpdateView, AjaxDeleteView
 
 class PartIndex(ListView):
     model = Part
-    template_name = 'part/index.html'
+    template_name = 'part/category.html'
     context_object_name = 'parts'
 
     def get_queryset(self):
@@ -128,7 +128,7 @@ class CategoryDetail(DetailView):
     model = PartCategory
     context_object_name = 'category'
     queryset = PartCategory.objects.all()
-    template_name = 'part/category_detail.html'
+    template_name = 'part/category.html'
 
 
 class CategoryEdit(AjaxUpdateView):
diff --git a/InvenTree/static/css/inventree.css b/InvenTree/static/css/inventree.css
index e4f01e97bf..70cee55d51 100644
--- a/InvenTree/static/css/inventree.css
+++ b/InvenTree/static/css/inventree.css
@@ -47,7 +47,7 @@
     margin-right: 50px;
     margin-left: 50px;
     width: 100%;
-    transition: 0.5s;
+    transition: 0.1s;
 }
 
 .body {
@@ -88,7 +88,7 @@
     position: fixed; /* Stay in place */
     background-color: #fff; /* Black*/
     overflow-x: hidden; /* Disable horizontal scroll */
-    transition: 0.5s; /* 0.5 second transition effect to slide in the sidenav */
+    transition: 0.1s; /* 0.5 second transition effect to slide in the sidenav */
 }
 
 .wrapper {
diff --git a/InvenTree/static/script/sidenav.js b/InvenTree/static/script/sidenav.js
index 1520b91ea7..382a126148 100644
--- a/InvenTree/static/script/sidenav.js
+++ b/InvenTree/static/script/sidenav.js
@@ -1,9 +1,52 @@
+function loadTree(url, tree, data) {
+
+    $.ajax({
+        url: url,
+        type: 'get',
+        dataType: 'json',
+        data: data,
+        success: function (response) {
+            if (response.tree) {
+                $(tree).treeview({
+                    data: response.tree,
+                    enableLinks: true
+                });
+            }
+        },
+        error: function (xhr, ajaxOptions, thrownError) {
+            //TODO
+        }
+    });
+}
+
 function openSideNav() {
     document.getElementById("sidenav").style.width = "250px";
     document.getElementById("inventree-content").style.marginLeft = "270px";
+
+    sessionStorage.setItem('inventree-sidenav-state', 'open');
 }
 
 function closeSideNav() {
     document.getElementById("sidenav").style.width = "0";
     document.getElementById("inventree-content").style.marginLeft = "50px";
+
+    sessionStorage.setItem('inventree-sidenav-state', 'closed');
+}
+
+function toggleSideNav(nav) {
+    if ($(nav).width() == 0) {
+        openSideNav();
+    }
+    else {
+        closeSideNav();
+    }
+}
+
+function initSideNav() {
+    if (sessionStorage.getItem("inventree-sidenav-state") && sessionStorage.getItem('inventree-sidenav-state') == 'open') {
+        openSideNav();
+    }
+    else {
+        closeSideNav();
+    }
 }
\ No newline at end of file
diff --git a/InvenTree/static/script/trees.js b/InvenTree/static/script/trees.js
deleted file mode 100644
index 2938f3366e..0000000000
--- a/InvenTree/static/script/trees.js
+++ /dev/null
@@ -1,20 +0,0 @@
-function loadTree(url, tree, data) {
-
-    $.ajax({
-        url: url,
-        type: 'get',
-        dataType: 'json',
-        data: data,
-        success: function (response) {
-            if (response.tree) {
-                $(tree).treeview({
-                    data: response.tree,
-                    enableLinks: true
-                });
-            }
-        },
-        error: function (xhr, ajaxOptions, thrownError) {
-            //TODO
-        }
-    });
-}
\ No newline at end of file
diff --git a/InvenTree/templates/base.html b/InvenTree/templates/base.html
index b06b0ad37d..32d928306b 100644
--- a/InvenTree/templates/base.html
+++ b/InvenTree/templates/base.html
@@ -38,11 +38,16 @@ InvenTree
 </div>
 
 <div class="container container-fluid inventree-content" id='inventree-content'>
+{% block pre_content %}
+{% endblock %}
 {% block content %}
 <!-- Each view fills in here.. -->
 {% endblock %}
+{% block post_content %}
+{% endblock %}
 </div>
 
+{% include 'modals.html' %}
 {% include 'notification.html' %}
 </div>
 
@@ -57,38 +62,18 @@ InvenTree
 <script type='text/javascript' src="{% static 'script/bootstrap-table-en-US.min.js' %}"></script>
 
 <script type='text/javascript' src="{% static 'script/tables.js' %}"></script>
-<script type='text/javascript' src="{% static 'script/trees.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/sidenav.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/notification.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/jquery.form.min.js' %}"></script>
-
+<script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
 {% block js_load %}
 {% endblock %}
 
 <script type='text/javascript'>
 $(document).ready(function () {
-
-    loadTree("{% url 'api-part-tree' %}",
-             "#part-tree");
-
-    loadTree("{% url 'api-stock-tree' %}",
-             "#stock-tree");
-
-    $('#logo').click(function() {
-        if ($("#sidenav").width() == 0) {
-            openSideNav();
-        }
-        else {
-            closeSideNav();
-        }
-    });
-
 {% block js_ready %}
-
 {% endblock %}
-
     showCachedAlerts();
-
 });
 </script>