Mirrored_Repos/InvenTree
1
0
Fork 0
mirror of https://github.com/inventree/InvenTree synced 2024-08-30 18:33:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixed part image upload

Browse Source 
- Image was being saved twice
- Don't call super().post in AjaxUpdateView
- Instead, handle properly
This commit is contained in:
Oliver Walters 2019-05-04 00:37:08 +10:00
parent b6d6b648ef
commit 4af59f0ca9
2 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
InvenTree/InvenTree/views.py
View File

@ -201,10 +201,8 @@ class AjaxUpdateView(AjaxMixin, UpdateView):
""" """
super(UpdateView, self).get(request, *args, **kwargs) super(UpdateView, self).get(request, *args, **kwargs)
form = self.get_form()
return self.renderJsonResponse(request, form, context=self.get_context_data()) return self.renderJsonResponse(request, self.get_form(), context=self.get_context_data())
def post(self, request, *args, **kwargs): def post(self, request, *args, **kwargs):
""" Respond to POST request. """ Respond to POST request.
@ -215,7 +213,8 @@ class AjaxUpdateView(AjaxMixin, UpdateView):
- Otherwise, return sucess status - Otherwise, return sucess status
""" """
super(UpdateView, self).post(request, *args, **kwargs) # Make sure we have an object to point to
self.object = self.get_object()
form = self.get_form() form = self.get_form()
@ -225,7 +224,7 @@ class AjaxUpdateView(AjaxMixin, UpdateView):
if form.is_valid(): if form.is_valid():
obj = form.save() obj = form.save()
# Include context data about the updated object # Include context data about the updated object
data['pk'] = obj.id data['pk'] = obj.id

4
InvenTree/part/models.py
View File

@ -115,10 +115,10 @@ class Part(models.Model):
return reverse('part-detail', kwargs={'pk': self.id}) return reverse('part-detail', kwargs={'pk': self.id})
# Short name of the part # Short name of the part
name = models.CharField(max_length=100, unique=True, help_text='Part name (must be unique)') name = models.CharField(max_length=100, unique=True, blank=False, help_text='Part name (must be unique)')
# Longer description of the part (optional) # Longer description of the part (optional)
description = models.CharField(max_length=250, help_text='Part description') description = models.CharField(max_length=250, blank=False, help_text='Part description')
# Internal Part Number (optional) # Internal Part Number (optional)
# Potentially multiple parts map to the same internal IPN (variants?) # Potentially multiple parts map to the same internal IPN (variants?)