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 104292 - dialog when profiling project for first time
Summary: dialog when profiling project for first time
Status: RESOLVED FIXED
Alias: None
Product: profiler
Classification: Unclassified
Component: Base (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: Jiri Sedlacek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-18 16:46 UTC by Kenneth Ganfield
Modified: 2007-10-25 15:57 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
integrate profiler (13.67 KB, image/png)
2007-10-25 15:55 UTC, Jiri Sedlacek
Details
unintegrate profiler (10.88 KB, image/png)
2007-10-25 15:56 UTC, Jiri Sedlacek
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kenneth Ganfield 2007-05-18 16:46:07 UTC
The dialog that appears when profiling a project for the "first" time needs to
be reworded slightly.

Currently it says:
"This is the first time this project is to be profiled..."
It seems that this is not quite true.
Seems like it should say something like "NetBeans Profiler is not integrated
with this project. To integrate Profiler with the project, the Profiler needs to
..."

Not sure if "integrate" is the correct word.
There is an item under Advanced Commands that says to "Unintegrate Profiler from
..." which is i assume means undo what was done the first time the project is
profiled.

Technically, "unintegrate" is not a verb, and the opposite of integrate is
disintegrate, which we probably dont want to say.
Comment 1 Jiri Sedlacek 2007-06-26 15:28:45 UTC
Based on previous email communication, could you please suggest concrete new wording of the dialog for 6.0? Please note 
that this will be the only solution of automatic Profiler & Project integration, there will be no manual integration 
support in 6.0.

Also please suggest better wording for "Unintegrate Profiler from...", it should be short and contain display name of 
selected project.
Comment 2 Kenneth Ganfield 2007-06-27 17:08:24 UTC

I would suggest using "Restore build script for XXX" instead of "Unintegrate Profiler from XXX"
I am not entirely convinced that this is the best wording or the best way to describe how the project is prepared for
profiling.


Here are my suggestions for changes to the text in the dialogs.

Add import dialog:

1. Change title of dialog box to "Profile Project"
2. Change text in dialog to the following:

"The IDE needs to add an import task to the project build script to enable profiling.

Click OK to allow the IDE to modify the build script and create a backup of the original build script.

You can restore the original build script by choosing Profile > Advanced Commands > Restore build script from the main
menu."


Remove import dialog:
1. Change the title to "Restore Build Script"
2. Change text in dialog to the following:

"Click OK to restore the original build script for the project XXX.

The backup of the original build script was created when it was modified to enable profiling. 
Modifications made to the build script after the backup was created will be lost."
Comment 3 Jiri Sedlacek 2007-10-25 15:54:18 UTC
I've slightly changed the two dialogs, see the screenshots below. If not exactly incorrect, I would like to keep this
wording for 6.0.

I've kept the action name "Unintegrate", it seems to be quite widely used term in this area (check google for unintegrate).

I've also kept the term "Integration" instead of "Buildscript modification" because the integration here does more than
just a buildscript modification.
Comment 4 Jiri Sedlacek 2007-10-25 15:55:49 UTC
Created attachment 51694 [details]
integrate profiler
Comment 5 Jiri Sedlacek 2007-10-25 15:56:13 UTC
Created attachment 51695 [details]
unintegrate profiler
Comment 6 Jiri Sedlacek 2007-10-25 15:57:05 UTC
Fixed in trunk.

/cvs/profiler/modules/j2se/src/org/netbeans/modules/profiler/j2se/Bundle.properties,v  <--  Bundle.properties
new revision: 1.2; previous revision: 1.1
/cvs/profiler/modules/j2se/src/org/netbeans/modules/profiler/j2se/J2SEProjectTypeProfiler.java,v  <-- 
J2SEProjectTypeProfiler.java
new revision: 1.2; previous revision: 1.1
/cvs/profiler/modules/j2ee/src/org/netbeans/modules/profiler/j2ee/Bundle.properties,v  <--  Bundle.properties
new revision: 1.2; previous revision: 1.1
/cvs/profiler/modules/j2ee/src/org/netbeans/modules/profiler/j2ee/J2EEProjectTypeProfiler.java,v  <-- 
J2EEProjectTypeProfiler.java
new revision: 1.2; previous revision: 1.1
/cvs/profiler/modules/main/src/org/netbeans/modules/profiler/utils/ProjectUtilities.java,v  <--  ProjectUtilities.java
new revision: 1.2; previous revision: 1.1
/cvs/profiler/modules/main/src/org/netbeans/modules/profiler/utils/Bundle.properties,v  <--  Bundle.properties
new revision: 1.2; previous revision: 1.1