Project

General

Profile

Bug #9019

Downloading and uploading of marks does not work

Added by Ilja Zakharov 8 months ago. Updated 8 months ago.

Status:
Closed
Priority:
Immediate
Category:
Bridge
Target version:
Start date:
06/27/2018
Due date:
% Done:

100%

Estimated time:
Detected in build:
svn
Platform:
Published in build:

Description

I have an unsafe mark with several versions where I edited converted error trace. When I download it, delete and then upload I see the error:

In order to allow non-dict objects to be serialized set the safe parameter to False.
Traceback (most recent call last):
File "/work/zakharov/src/klever/bridge/bridge/CustomViews.py", line 61, in dispatch
return getattr(super(), 'dispatch')(request, *args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/django/views/generic/base.py", line 89, in dispatch
return handler(request, *args, **kwargs)
File "/work/zakharov/src/klever/bridge/bridge/CustomViews.py", line 94, in post
return JsonResponse(self.get_context_data())
File "/usr/local/lib/python3.6/dist-packages/django/http/response.py", line 503, in init
'In order to allow non-dict objects to be serialized set the '
TypeError: In order to allow non-dict objects to be serialized set the safe parameter to False.

However, the mark is uploaded but similarity of associated reports is incorrect, the relevant issue.


Related issues

Related to Klever - Bug #9017: Change of a converted error trace does not lead to assotiated leaf reports similarity recalculationClosed2018-06-27

History

#1 Updated by Ilja Zakharov 8 months ago

  • Related to Bug #9017: Change of a converted error trace does not lead to assotiated leaf reports similarity recalculation added

#2 Updated by Evgeny Novikov 8 months ago

  • Priority changed from Urgent to Immediate

This is definitely a regression.

#3 Updated by Vladimir Gratinskiy 8 months ago

  • % Done changed from 0 to 100
  • Status changed from New to Resolved

Fixed in "fix_marks".

#4 Updated by Ilja Zakharov 8 months ago

  • Status changed from Resolved to Verified

Works fine.

#5 Updated by Evgeny Novikov 8 months ago

  • Status changed from Verified to Closed

I merged the branch to master in aa3e6cc8.

Also available in: Atom PDF