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 189915 - "Target server not set properly" message does not give enough information
Summary: "Target server not set properly" message does not give enough information
Status: RESOLVED FIXED
Alias: None
Product: javaee
Classification: Unclassified
Component: Persistence (show other bugs)
Version: 7.0
Hardware: All All
: P3 normal (vote)
Assignee: Sergey Petrov
URL:
Keywords:
Depends on:
Blocks: 189312
  Show dependency tree
 
Reported: 2010-08-26 13:35 UTC by Ondrej Langr
Modified: 2011-03-24 09:49 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
the mentioned error message (123.23 KB, image/png)
2010-08-26 13:37 UTC, Ondrej Langr
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ondrej Langr 2010-08-26 13:35:49 UTC
In places where having the target server set properly is crucial to access full set of wizard functionality, a warning should specifically warn users that they are going to have problems later. 

Current error message was not clear enough and did not give participants enough information to decide how significant the problem is.

Not sure about component, please reassign if necessary, thank you.
Comment 1 Ondrej Langr 2010-08-26 13:37:16 UTC
Created attachment 101693 [details]
the mentioned error message
Comment 2 Sergey Petrov 2010-08-26 14:02:46 UTC
Not sure it's a defect. It say about dependency of jpa features and it's short and have some meaning.
In attached case it's really a warning and isn't an error message as entities itself are server independent, but as persistence unit and possible generated later session beans/controllers/jsf p[ages are server dependent there is a warning.
Do you have any suggestions?
Comment 3 Ondrej Langr 2010-08-31 11:33:02 UTC
CCing Ken from docs team who knows more about this issue.

Ken, could you suggest a proper error message for this situation?
Comment 4 Kenneth Ganfield 2010-09-09 15:19:51 UTC
Here is a suggestion for wording that might be stronger, is a bit more detailed and a bit longer:

The target server for the project is not set properly. Java Persistence features are server-dependent and some wizards in the IDE might not behave as expected if the target server is not specified.
Comment 5 Sergey Petrov 2010-09-09 17:06:13 UTC
Thanks, will try but it may not fit well in current layout.
Comment 6 Sergey Petrov 2010-09-09 17:08:15 UTC
Just one note, regarding "properly". Maven projects do not require server by default, is it ok to say "The target server for the project is not set properly." for this case? In case of ant if project is missed it's really some problem with server setup.
Comment 7 David Konecny 2010-09-10 01:14:56 UTC
This concrete example is getting obsolete by my fix of issue 188610. Ondro, I would appreciate if you could have a look at my solution and confirm that it is improvement. :-) Or I would revert it otherwise. Thx.
Comment 8 Sergey Petrov 2010-09-13 14:33:27 UTC
yes, with latest changes the issue for 'entities from db' is obsolete.
just tried other wizards with this text and it fit wizard window quite well in my opinion, but I need confirmation if text should be chaged in all these places: new persistence unit, new entity, new jpa controller(should be warning too, but there is an issue and it's not shown), new session bean from entity(the same).
Also some cases may need(or good) to be updated to use entities from db improvement.
Comment 9 Sergey Petrov 2011-03-23 17:11:43 UTC
changed to suggested text http://hg.netbeans.org/web-main/rev/7f01c4c61e44
Comment 10 Quality Engineering 2011-03-24 09:49:26 UTC
Integrated into 'main-golden', will be available in build *201103240400* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/7f01c4c61e44
User: Sergey B. Petrov <sj-nb@netbeans.org>
Log: fix #189915 - change to suggested text