Bug 29960 - MultiUploader and Uploader error
Summary: MultiUploader and Uploader error
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
Version: UCS 3.1
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 3.1-0-errata
Assignee: Alexander Kläser
QA Contact: Dirk Wiesenthal
URL:
Keywords:
: 28818 (view as bug list)
Depends on:
Blocks: 29785
  Show dependency tree / graph
 
Reported: 2013-01-08 13:08 CET by Florian Best
Modified: 2013-01-15 15:14 CET (History)
3 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Customer ID:
Max CVSS v3 score:
best: Patch_Available+

Attachments
Patch for MultiUploader and umc/tools:parseError() (2.16 KB, patch)
2013-01-14 13:29 CET, Alexander Kläser 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2013-01-08 13:08:19 CET 
The umc.widgets.MultiUploader contains an error which causes the upload button to disappear. Also the umc.widgets.Uploader contains an error when trying to disable an destroyed widget.
Comment 1 Florian Best univentionstaff 2013-01-08 13:55:29 CET 
* yaml updated
* changes merged into 3.1-1
* errata package built
Comment 2 Alexander Kläser univentionstaff 2013-01-08 14:42:50 CET 
*** Bug 28818 has been marked as a duplicate of this bug. ***
Comment 3 Dirk Wiesenthal univentionstaff 2013-01-14 10:45:45 CET 
There seems to be another bug in MultiUploader. One cannot remove multiple files at once. Error: dojo.data.ItemFileWriteStore: Invalid item argument. Has to do with a not updated dojox.grid.DataSelection.selected
Comment 4 Alexander Kläser univentionstaff 2013-01-14 13:29:45 CET 
Created attachment 4983 [details]
Patch for MultiUploader and umc/tools:parseError()

For MultiUploader, a selection.clear() was missing prior to updating the list of files (otherwise some references were ill defined). Additionally, I adjusted the function umc/tools:parseError() as upload errors (e.g., too large files) where not handled correctly (instead an generic "server error" message would be displayed).
Comment 5 Alexander Kläser univentionstaff 2013-01-14 13:53:23 CET 
Package built in errata scope.

Added patch to 3.1 branch... package is building.

 univention-management-console-frontend (2.0.187-1) unstable; urgency=low
 .
   * MultiUploader: added missing clear for selected files; Bug #29960
   * umc/tools:parseError: added handling for upload errors; Bug #29960
Comment 6 Dirk Wiesenthal univentionstaff 2013-01-14 17:09:58 CET 
Änderung: OK
YAML: OK
3.1-1: OK
Changelog: OK
Comment 7 Moritz Muehlenhoff univentionstaff 2013-01-15 15:14:59 CET 
http://errata.univention.de/3.1-errata9.html