Project

General

Profile

Feature #8334

Clear log message in case of uploading marks

Added by Pavel Andrianov about 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
High
Category:
Bridge
Target version:
-
Start date:
08/11/2017
Due date:
08/11/2017
% Done:

100%

Estimated time:
Published in build:

Description

I tried to upload marks using Marks->upload. Something has gone wrong and there was an exception "ValueError: Unsupported mark data type: <class 'NoneType'>". After some time Eugeny and I could realize, that Bridge expects single mark, and the archive of them should be uploaded with Manager Tools. The web description "Unknown error" may stay the same (as it is not so easy to change), but the internal log description should be more clear.


Files

linux_alloc_irq.zip (2.09 MB) linux_alloc_irq.zip Evgeny Novikov, 08/11/2017 04:24 PM

History

#1

Updated by Evgeny Novikov about 3 years ago

  • Priority changed from Normal to High

A corresponding fix is trivial but a corresponding value may be high.

#2

Updated by Vladimir Gratinskiy about 3 years ago

  • Due date set to 08/11/2017
  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Fixed in "fix_8334".

#3

Updated by Evgeny Novikov about 3 years ago

Let's fix one more related issue in addition:

invalid literal for int() with base 10: '2.json'
Traceback (most recent call last):
  File "/home/novikov/work/klever/bridge/marks/views.py", line 401, in upload_marks
    res = UploadMark(request.user, f)
  File "/home/novikov/work/klever/bridge/marks/Download.py", line 125, in __init__
    self.mark = self.__upload_mark(archive)
  File "/home/novikov/work/klever/bridge/marks/Download.py", line 144, in __upload_mark
    version_id = int(file_name.replace('version-', ''))
ValueError: invalid literal for int() with base 10: '2.json'

#4

Updated by Vladimir Gratinskiy about 3 years ago

  • Status changed from Open to Resolved

Fixed.

#5

Updated by Evgeny Novikov about 3 years ago

  • Status changed from Resolved to Closed

I merged the branch to master in c10b25c3.

Also available in: Atom PDF