Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 206511

Summary: Wrong confirmation message when deleting catalog
Product: z_Archived Reporter: Malcom Zung <zung>
Component: TPTP.monitoringAssignee: Yasuhisa Gotoh <gotohy>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P1 CC: apnan
Version: unspecifiedKeywords: plan
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard: closed460
Attachments:
Description Flags
Patch file
none
Updated patch none

Description Malcom Zung CLA 2007-10-16 14:20:23 EDT 
Build ID: TPTP 4.4.0.3

Steps To Reproduce:
1.Create a symptom catalog
2.In Log Navigator view, use context menu to delete the catalog
3.

Problem:
A confirmation dialog appears asking if it is OK to delete the log file.  This confirmation dialog should ask about symptom catalog, not log file.

More information:
Comment 1 Alex Nan CLA 2007-11-05 18:03:23 EST 
Targetting 4.5 i4.
Comment 2 Yasuhisa Gotoh CLA 2008-01-16 07:17:43 EST 
The confirmation message "Delete the selected log file(s)?\nThe original log file(s) will not be affected." is set within the following class:

org.eclipse.hyades.trace.ui.internal.actions.DeleteAction
    in the project org.eclipse.tptp.platform.common.ui.trace

Currently the only message shown above is used for all resource types.

Need further investigation on how to switch the confirmation message according to the resource type.
Comment 3 Yasuhisa Gotoh CLA 2008-01-18 06:53:42 EST 
Some correction in comment #2:

Currently the only message shown above is used for all file resources.
                                                       ^^^^^^^^^^^^^^
Comment 4 Yasuhisa Gotoh CLA 2008-01-18 06:55:14 EST 
Created attachment 87251 [details]
Patch file

Alex, please review the patch.
Comment 5 Alex Nan CLA 2008-01-20 16:34:17 EST 
Created attachment 87363 [details]
Updated patch

I have updated the message CONFIRM_DELM to be in synch with the rest of the confirmation messages although they are not entirely correct from NL point of view.
Please check in the patch.
Comment 6 Yasuhisa Gotoh CLA 2008-01-21 04:37:18 EST 
Alex, will you check in the patch for me since this should be applied to the platform project?
Comment 7 Alex Nan CLA 2008-01-21 10:23:27 EST 
Patch checked in.
Comment 8 Paul Slauenwhite CLA 2009-06-30 13:17:25 EDT 
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.