This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 67074 - misleading message for Safely Delete
Summary: misleading message for Safely Delete
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: Refactoring (show other bugs)
Version: 5.x
Hardware: All All
: P3 blocker (vote)
Assignee: Jan Becicka
URL:
Keywords: UI
Depends on:
Blocks:
 
Reported: 2005-10-18 21:01 UTC by Patrick Keegan
Modified: 2007-04-03 18:02 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Keegan 2005-10-18 21:01:33 UTC
1. Choose Safely Delete for a method that is referenced by other code.
2. In the Safe Delete dialog box, click Next (ignoring the warning).

In the Refactoring window, the text reads "Safely delete method..."

This is wrong, because it is no longer a safe delete. Better to say something
like "Delete method ... and keep references to that method."
Comment 1 Jan Becicka 2005-10-19 10:20:52 UTC
Problem is, that this text is static. We don't know, whether there are some
references or not.
We should probably use:
"Delete method ... and keep references to that method if any."

Also there are similar messages for similar situations:

"Delete element(s) and keep references to them if any."
"Delete classes and and keep references to them if any."
"Delete methods and keep keep references to them if any."

Is it acceptable and English enough?
Comment 2 Patrick Keegan 2005-10-19 14:01:18 UTC
Yeah, I figured it might be a case of static text. Maybe it should be more
dynamic, but obviously that's not a problem to fix for this release.

I agree with your wording, except that you should put a comma (,) before "if
any", e.g.

"Delete element(s) and keep references to them, if any."
Comment 3 Jan Becicka 2005-10-19 15:45:38 UTC
OK. Thanks for help.

Checking in Bundle.properties;
/cvs/refactoring/src/org/netbeans/modules/refactoring/Bundle.properties,v  <-- 
Bundle.properties
new revision: 1.9; previous revision: 1.8
done
Checking in ui/Bundle.properties;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/Bundle.properties,v 
<--  Bundle.properties
new revision: 1.83; previous revision: 1.82
done