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 57734 - Connection pool properties are deleted after connection properties change
Summary: Connection pool properties are deleted after connection properties change
Status: CLOSED FIXED
Alias: None
Product: serverplugins
Classification: Unclassified
Component: Sun Appserver 8 (show other bugs)
Version: 4.x
Hardware: All All
: P2 blocker (vote)
Assignee: Nitya Doraisamy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-11 16:33 UTC by Jan Horvath
Modified: 2006-03-24 13:14 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Horvath 2005-04-11 16:33:34 UTC 
build 200504101930

-register appserver
-in runtime tab open appserver node/JDBC node/Connection pool node
-edit properties of some connection pool, change some property (e.g. username, password or url)
-close properties
-restart appserver
-open connection pool properties. all properties are deleted (e.g. username, password...)
Comment 1 Nitya Doraisamy 2005-04-11 21:03:28 UTC 
Optional properties set functionality was not working for JDBC Connection Pool
properly. So the modified properties were  not being propagated to the server

Simple fix. Diff attached. Fixed in Trunk build.

> Index: AppserverMgmtController.java
> ===================================================================
> RCS file:
/m/src/iplanet/s1dp/netbeans-j2ee/j2ee/appsrv-plugin/libbridge/org/netbeans/modules/j2ee/sun/ide/controllers/AppserverMgmtController.java,v
> retrieving revision 1.14
> diff -r1.14 AppserverMgmtController.java
> 1068a1069
> >                 ControllerUtil.setPropertyValue(cpConfig, props);
> 1073a1075
> >                 ControllerUtil.setPropertyValue(connConfig, props);
>
Comment 2 _ ludo 2005-04-11 21:15:04 UTC 
fix ok.
Comment 3 Jan Horvath 2005-04-13 14:08:03 UTC 
verified in build 20050413-0957
Comment 4 _ ludo 2005-04-13 22:52:39 UTC 
integrated in 4.1 branch