Opened 16 years ago
Closed 15 years ago
#282 closed defect (fixed)
Error message in Taskpane when fusion timeout is unexpected.
Reported by: | christinebao | Owned by: | madair |
---|---|---|---|
Priority: | P2 | Milestone: | Future |
Component: | Widgets | Version: | 1.1.1 |
Severity: | Major | Keywords: | |
Cc: | Browser: | All | |
External ID: | 1250065 | Operating System: | All |
state: | New |
Description ¶
From Autodesk QA:
Steps to reproduce:
Steps:
- Open one flexible web layout by IE.
- Wait for fusion timeout.
- Click Theme and Query.
Result:
The error messages on task pane are not same. They should not repeat and behave the same.
Change History (18)
by , 16 years ago
Attachment: | TaskPane.JPG added |
---|
comment:1 by , 16 years ago
External ID: | → 1250065 |
---|
comment:2 by , 16 years ago
Tech diagnosis:
After investigation, the reason of this defect is that different widgets use different format to display error message. They should be fixed to be consistent.
The following patches will fix them one by one.
by , 16 years ago
Attachment: | Expected.JPG added |
---|
comment:4 by , 16 years ago
Attach patch http://trac.osgeo.org/fusion/attachment/ticket/282/FixThemeErrorMessage.patch.
This patch fixed Theme widget. If errorMsg is the same as detailMsg (detailMsg is formatted by errorMsg + '\n' + stack information, if detailMsg is not longer than errorMsg by 5 characters, treat them as same), display errorMsg is enough; otherwise display them both.
comment:5 by , 16 years ago
Attach patch http://trac.osgeo.org/fusion/attachment/ticket/282/FixQueryErrorMessage.patch.
This patch fixed Query widget.
comment:6 by , 16 years ago
Attach patch http://trac.osgeo.org/fusion/attachment/ticket/282/FixFeatureInfoErrorMessage.patch.
This patch fixed FeatureInfo widget.
comment:7 by , 16 years ago
In each of the patches, you're introducing a chunk of code that looks like this:
<?php } else { ?> <?php if ($errorDetail == null || (strlen($errorDetail) - strlen($errorMsg) < 5)) { ?>
I think you could use "else if" instead...
<?php } else if ($errorDetail == null || (strlen($errorDetail) - strlen($errorMsg) < 5)) { ?>
And the code would be a little cleaner.
by , 16 years ago
Attachment: | FixThemeErrorMessage.patch added |
---|
by , 16 years ago
Attachment: | FixQueryErrorMessage.patch added |
---|
by , 16 years ago
Attachment: | FixFeatureInfoErrorMessage.patch added |
---|
comment:8 by , 16 years ago
comment:10 by , 16 years ago
Buffer widget is different from FeatureInfo, Query and Theme. Please see the attached image. The code is also different, and this need DM Solution to fix.
by , 16 years ago
Attachment: | Buffer.JPG added |
---|
comment:12 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed by submission: http://trac.osgeo.org/fusion/changeset/1953
Attach screen shot for reference: