Univention Bugzilla – Full Text Bug Listing |
Summary: | Every Grid resize throws a exception | ||
---|---|---|---|
Product: | UCS | Reporter: | Florian Best <best> |
Component: | UMC (Generic) | Assignee: | Florian Best <best> |
Status: | CLOSED FIXED | QA Contact: | Johannes Keiser <keiser> |
Severity: | normal | ||
Priority: | P5 | CC: | gohmann, klaeser, walkenhorst |
Version: | UCS 4.0 | Flags: | best:
Patch_Available+
|
Target Milestone: | UCS 4.0-0-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
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): | Cleanup, Error handling, External feedback | |
Max CVSS v3 score: | |||
Attachments: | patch |
Description
Florian Best
2014-11-13 15:19:43 CET
The error message is just due to cancelling a running Deferred (in this case tools.defer() for some resize event handling). I do not see whether there is a way around this in Dojo (well, patching dojo/Deferred). We received this as external feedback. A customer thought this was an real error. I think I was wrong. I guess, we just need an error handler to catch the cancel error. The following cancel is logged on the console: > d = new Deferred() > d.cancel() > c {message: undefined, log: false, stack: (...), dojoType: "cancel", name: "CancelError"…} > "Error > at Error (native) > ... Whereas the following lines will catch that error: > d = new Deferred() > d = d.then(null, function() { console.log('error caught!'); }) > d.cancel() > > error caught! > Created attachment 6477 [details]
patch
patch applied. YAML adapted. Fix: OK YAML: OK (fixed typo) I noticed a similar CancelError which is thrown with the new notifications. Could you please adjust this, as well? (In reply to Alexander Kläser from comment #7) > I noticed a similar CancelError which is thrown with the new notifications. > Could you please adjust this, as well? fixed, too. svn r57311 YAML adapted in svn r57314 Fix: OK YAML: OK |