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

Summary:	misleading message for Safely Delete
Product:	editor	Reporter:	Patrick Keegan <pkeegan>
Component:	Refactoring	Assignee:	Jan Becicka <jbecicka>
Status:	RESOLVED FIXED
Severity:	blocker	CC:	jrojcek
Priority:	P3	Keywords:	UI
Version:	5.x
Hardware:	All
OS:	All
Issue Type:	DEFECT	Exception Reporter:

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