From 01f1ac49e32f9a1d07293b6f6cd163302b536fa7 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Thu, 19 Mar 2020 10:23:41 +1100
Subject: [PATCH] Improve SupplierPart detail page

---
 .../templates/company/supplier_part_base.html | 46 +++++++++++++++----
 .../templates/company/supplier_part_tabs.html |  3 --
 InvenTree/company/urls.py                     |  2 +-
 3 files changed, 39 insertions(+), 12 deletions(-)

diff --git a/InvenTree/company/templates/company/supplier_part_base.html b/InvenTree/company/templates/company/supplier_part_base.html
index f7e6ce88cc..76f75094f7 100644
--- a/InvenTree/company/templates/company/supplier_part_base.html
+++ b/InvenTree/company/templates/company/supplier_part_base.html
@@ -21,19 +21,49 @@ InvenTree | {% trans "Supplier Part" %}
                 </button>
             </div>
         </div>
-    </div>
-    <div class='col-sm-6'>
         <div class='media-left'>
-            <img class='part-thumb'
-            {% if part.part.image %}
-            src='{{ part.part.image.url }}'
-            {% else %}
-            src="{% static 'img/blank_image.png' %}"
-            {% endif %}/>
+                <img class='part-thumb'
+                {% if part.part.image %}
+                src='{{ part.part.image.url }}'
+                {% else %}
+                src="{% static 'img/blank_image.png' %}"
+                {% endif %}/>
         </div>
     </div>
+    <div class='col-sm-6'>
+        <h4>{% trans "Supplier Part Details" %}</h4>
+        <table class="table table-striped table-condensed">
+                <tr>
+                    <td>{% trans "Internal Part" %}</td>
+                    <td>
+                        {% if part.part %}
+                        <a href="{% url 'part-suppliers' part.part.id %}">{{ part.part.full_name }}</a>
+                        {% endif %}
+                    </td>
+                </tr>
+                <tr><td>{% trans "Supplier" %}</td><td><a href="{% url 'company-detail-parts' part.supplier.id %}">{{ part.supplier.name }}</a></td></tr>
+                <tr><td>{% trans "SKU" %}</td><td>{{ part.SKU }}</tr></tr>
+            {% if part.URL %}
+                <tr><td>{% trans "URL" %}</td><td><a href="{{ part.URL }}">{{ part.URL }}</a></td></tr>
+            {% endif %}
+            {% if part.description %}
+                <tr><td>{% trans "Description" %}</td><td>{{ part.description }}</td></tr>
+            {% endif %}
+            {% if part.manufacturer %}
+                <tr><td>{% trans "Manufacturer" %}</td><td>{{ part.manufacturer }}</td></tr>
+                <tr><td>{% trans "MPN" %}</td><td>{{ part.MPN }}</td></tr>
+            {% endif %}
+            {% if part.note %}
+                <tr><td>{% trans "Note" %}</td><td>{{ part.note }}</td></tr>
+            {% endif %}
+        </table>
+    </div>
 </div>
 
+
+
+
+
 <hr>
 
 <div class='container-fluid'>
diff --git a/InvenTree/company/templates/company/supplier_part_tabs.html b/InvenTree/company/templates/company/supplier_part_tabs.html
index 0bcd8aa4fa..b3a4389bbd 100644
--- a/InvenTree/company/templates/company/supplier_part_tabs.html
+++ b/InvenTree/company/templates/company/supplier_part_tabs.html
@@ -1,9 +1,6 @@
 {% load i18n %}
 
 <ul class='nav nav-tabs'>
-    <li{% if tab == 'details' %} class='active'{% endif %}>
-        <a href="{% url 'supplier-part-detail' part.id %}">{% trans "Details" %}</a>
-    </li>
     <li{% if tab == 'pricing' %} class='active'{% endif %}>
         <a href="{% url 'supplier-part-pricing' part.id %}">{% trans "Pricing" %}</a>
     </li>
diff --git a/InvenTree/company/urls.py b/InvenTree/company/urls.py
index 66cc0b9cd8..65eba73f84 100644
--- a/InvenTree/company/urls.py
+++ b/InvenTree/company/urls.py
@@ -53,7 +53,7 @@ supplier_part_detail_urls = [
     url(r'^orders/', views.SupplierPartDetail.as_view(template_name='company/supplier_part_orders.html'), name='supplier-part-orders'),
     url(r'^stock/', views.SupplierPartDetail.as_view(template_name='company/supplier_part_stock.html'), name='supplier-part-stock'),
 
-    url('^.*$', views.SupplierPartDetail.as_view(), name='supplier-part-detail'),
+    url('^.*$', views.SupplierPartDetail.as_view(template_name='company/supplier_part_pricing.html'), name='supplier-part-detail'),
 ]
 
 supplier_part_urls = [