| Summary: | The monitor tab should display an error when an unknown/invalid host is selected | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | amehrega | ||||||
| Component: | TPTP | Assignee: | Bing Xu <xubing> | ||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||
| Severity: | normal | ||||||||
| Priority: | P1 | CC: | ewchan, jkubasta, sluiman, xubing | ||||||
| Version: | unspecified | Keywords: | plan | ||||||
| Target Milestone: | --- | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| Whiteboard: | closed460 | ||||||||
| Attachments: |
|
||||||||
|
Description
amehrega
Ali. I am reassigning this bug to me. Bing, would you please try to resolve this bug in the 441 timeframe? Please ask Eugene to do the code review. Created attachment 87075 [details]
Patch
Add connection checking when the Monitor tab is activiated everytime. Display error message and hide all monitoring choices when can't connect to the host and port specified in Host tab.
Eugene, can you review the patch and if it's OK check it in. Thanks. Bing, I have reviewed the patch and it is good to go, please make sure the copyright year range is updated with 2008. Created attachment 88049 [details]
New patch with copyright year updated
Joanna, can you check in the patch for me. Thanks.
patch checked into Head. Thanks Bing. As of TPTP 4.6.0, TPTP is in maintenance mode and focusing on improving quality by resolving relevant enhancements/defects and increasing test coverage through test creation, automation, Build Verification Tests (BVTs), and expanded run-time execution. As part of the TPTP Bugzilla housecleaning process (see http://wiki.eclipse.org/Bugzilla_Housecleaning_Processes), this enhancement/defect is verified/closed by the Project Lead since this enhancement/defect has been resolved and unverified for more than 1 year and considered to be fixed. If this enhancement/defect is still unresolved and reproducible in the latest TPTP release (http://www.eclipse.org/tptp/home/downloads/), please re-open. |