Merge pull request #423 from SchrodingersGat/supplier-part-buttons

Fix buttons and rendering for supplierpart page

.travis.yml

@@ -2,7 +2,8 @@ dist: xenial
 
 language: python
 python:
-    - 3.5
+    - 3.6
+    - 3.7
 
 addons:
     apt-packages:

InvenTree/build/api.py

@@ -26,7 +26,7 @@ class BuildList(generics.ListCreateAPIView):
     serializer_class = BuildSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -47,7 +47,7 @@ class BuildDetail(generics.RetrieveUpdateAPIView):
     serializer_class = BuildSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
 
@@ -80,7 +80,7 @@ class BuildItemList(generics.ListCreateAPIView):
         return query
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [

InvenTree/company/api.py

@@ -32,7 +32,7 @@ class CompanyList(generics.ListCreateAPIView):
     serializer_class = CompanySerializer
     queryset = Company.objects.all()
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -66,7 +66,7 @@ class CompanyDetail(generics.RetrieveUpdateDestroyAPIView):
     serializer_class = CompanySerializer
     
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
 
@@ -102,7 +102,7 @@ class SupplierPartList(generics.ListCreateAPIView):
     serializer_class = SupplierPartSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -135,7 +135,7 @@ class SupplierPartDetail(generics.RetrieveUpdateDestroyAPIView):
 
     queryset = SupplierPart.objects.all()
     serializer_class = SupplierPartSerializer
-    permission_classes = (permissions.IsAuthenticatedOrReadOnly,)
+    permission_classes = (permissions.IsAuthenticated,)
 
     read_only_fields = [
     ]
@@ -152,7 +152,7 @@ class SupplierPriceBreakList(generics.ListCreateAPIView):
     serializer_class = SupplierPriceBreakSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [

InvenTree/company/templates/company/partdetail.html

@@ -10,19 +10,26 @@ InvenTree | {{ company.name }} - Parts
 <div class='row'>
     <div class='col-sm-6'>
         <h3>Supplier Part</h3>
-        <p><a href="{% url 'company-detail-parts' part.supplier.id %}">{{ part.supplier.name }}</a> - {{ part.SKU }}</p>
+        <div class='btn-row'>
+            <div class='btn-group'>
+                <button type='button' class='btn btn-default btn-glyph' id='edit-part' title='Edit supplier part'>
+                    <span class='glyphicon glyphicon-edit'/>
+                </button>
+                <button type='button' class='btn btn-default btn-glyph' id='delete-part' title='Delete supplier part'>
+                    <span class='glyphicon glyphicon-trash'/>
+                </button>
+            </div>
+        </div>
     </div>
     <div class='col-sm-6'>
-        <h3>
-            <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='edit-part' title='Edit supplier part'>Edit</a></li>
-                  <li><a href="#" id='delete-part' title='Delete supplier part'>Delete</a></li>
-                </ul>
-            </div>
-        </h3>
+        <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 %}/>
+        </div>
     </div>
 </div>
 
@@ -30,17 +37,18 @@ InvenTree | {{ company.name }} - Parts
 
 <div class='row'>
 <div class='col-sm-6'>
+    <h4>Supplier Part Details</h4>
     <table class="table table-striped table-condensed">
-        <tr><td>Supplier</td><td><a href="{% url 'company-detail-parts' part.supplier.id %}">{{ part.supplier.name }}</a></td></tr>
-        <tr><td>SKU</td><td>{{ part.SKU }}</tr></tr>
         <tr>
             <td>Internal Part</td>
             <td>
-        {% if part.part %}
-            <a href="{% url 'part-suppliers' part.part.id %}">{{ part.part.full_name }}</a>
-        {% endif %}
+                {% if part.part %}
+                <a href="{% url 'part-suppliers' part.part.id %}">{{ part.part.full_name }}</a>
+                {% endif %}
             </td>
         </tr>
+        <tr><td>Supplier</td><td><a href="{% url 'company-detail-parts' part.supplier.id %}">{{ part.supplier.name }}</a></td></tr>
+        <tr><td>SKU</td><td>{{ part.SKU }}</tr></tr>
     {% if part.URL %}
         <tr><td>URL</td><td><a href="{{ part.URL }}">{{ part.URL }}</a></td></tr>
     {% endif %}
@@ -58,10 +66,8 @@ InvenTree | {{ company.name }} - Parts
 </div>
 
 <div class='col-sm-6'>
+    <h4>Pricing Information</h4>
 <table class="table table-striped table-condensed">
-    <tr>
-        <th colspan='2'>Pricing</th>
-    </tr>
     <tr><td>Order Multiple</td><td>{{ part.multiple }}</td></tr>
     {% if part.base_cost > 0 %}
     <tr><td>Base Price (Flat Fee)</td><td>{{ part.base_cost }}</td></tr>

InvenTree/part/api.py

@@ -54,7 +54,7 @@ class CategoryList(generics.ListCreateAPIView):
     serializer_class = CategorySerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -91,7 +91,7 @@ class PartDetail(generics.RetrieveUpdateAPIView):
     serializer_class = PartSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
 
@@ -178,7 +178,7 @@ class PartList(generics.ListCreateAPIView):
         return parts_list
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -243,7 +243,7 @@ class PartStarList(generics.ListCreateAPIView):
         return Response(serializer.data, status=status.HTTP_201_CREATED, headers=headers)
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -292,7 +292,7 @@ class BomList(generics.ListCreateAPIView):
         return queryset
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -314,7 +314,7 @@ class BomDetail(generics.RetrieveUpdateDestroyAPIView):
     serializer_class = BomItemSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
 

InvenTree/stock/api.py

@@ -57,7 +57,7 @@ class StockDetail(generics.RetrieveUpdateDestroyAPIView):
 
     queryset = StockItem.objects.all()
     serializer_class = StockItemSerializer
-    permission_classes = (permissions.IsAuthenticatedOrReadOnly,)
+    permission_classes = (permissions.IsAuthenticated,)
 
 
 class StockFilter(FilterSet):
@@ -83,7 +83,7 @@ class StockStocktake(APIView):
     """
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     def post(self, request, *args, **kwargs):
@@ -153,7 +153,7 @@ class StockMove(APIView):
     """ API endpoint for performing stock movements """
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     def post(self, request, *args, **kwargs):
@@ -227,7 +227,7 @@ class StockLocationList(generics.ListCreateAPIView):
     serializer_class = LocationSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -390,7 +390,7 @@ class StockList(generics.ListCreateAPIView):
     serializer_class = StockItemSerializer
 
     permission_classes = [
-        permissions.IsAuthenticatedOrReadOnly,
+        permissions.IsAuthenticated,
     ]
 
     filter_backends = [
@@ -412,7 +412,7 @@ class StockStocktakeEndpoint(generics.UpdateAPIView):
 
     queryset = StockItem.objects.all()
     serializer_class = StockQuantitySerializer
-    permission_classes = (permissions.IsAuthenticatedOrReadOnly,)
+    permission_classes = (permissions.IsAuthenticated,)
 
     def update(self, request, *args, **kwargs):
         object = self.get_object()
@@ -434,7 +434,7 @@ class StockTrackingList(generics.ListCreateAPIView):
 
     queryset = StockItemTracking.objects.all()
     serializer_class = StockTrackingSerializer
-    permission_classes = [permissions.IsAuthenticatedOrReadOnly]
+    permission_classes = [permissions.IsAuthenticated]
 
     filter_backends = [
         DjangoFilterBackend,
@@ -469,7 +469,7 @@ class LocationDetail(generics.RetrieveUpdateDestroyAPIView):
 
     queryset = StockLocation.objects.all()
     serializer_class = LocationSerializer
-    permission_classes = (permissions.IsAuthenticatedOrReadOnly,)
+    permission_classes = (permissions.IsAuthenticated,)
 
 
 stock_endpoints = [

InvenTree/users/views.py

@@ -12,7 +12,7 @@ class UserDetail(generics.RetrieveAPIView):
 
     queryset = User.objects.all()
     serializer_class = UserSerializer
-    permission_classes = (permissions.IsAuthenticatedOrReadOnly,)
+    permission_classes = (permissions.IsAuthenticated,)
 
 
 class UserList(generics.ListAPIView):
@@ -20,7 +20,7 @@ class UserList(generics.ListAPIView):
 
     queryset = User.objects.all()
     serializer_class = UserSerializer
-    permission_classes = (permissions.IsAuthenticatedOrReadOnly,)
+    permission_classes = (permissions.IsAuthenticated,)
 
 
 class GetAuthToken(ObtainAuthToken):