[Bug 197408] Enable editting of connection properties for db connections

  • From:
  • To:
  • Subject: [Bug 197408] Enable editting of connection properties for db connections
  • Date: Wed, 13 Feb 2013 14:55:42 +0000
  • Auto-submitted: auto-generated

http://netbeans.org/bugzilla/show_bug.cgi?id=197408


 changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #131338|0                           |1
        is obsolete|                            |

--- Comment #9 from 

 2013-02-13 14:55:42 UTC ---
Created attachment 131350
  --> http://netbeans.org/bugzilla/attachment.cgi?id=131350
Proposed Patch v6

Updated patch

(In reply to comment #7)
> I see a lot of new methods, is DatabaseConnection.getConnectionProperties 
> the
> only new public method?
Yes, it was. The updated patch contains also new variant of public method
DatabaseConnection.create(...), see below.

> Even it's unrelated api. I see all other DBConnection methods have javadoc 
> and
> new ones have no javadoc.
Fixed.

> May it have sense to update javadoc for
> DatabaseConnection.getConnectionProperties to emphasize it will be copy of
> properties or changes will not affect a connection.
Sure. Fixed.

> I'm not sure what was initial use case, if it was supposed to have read 
> only access in public.
Some databases (in some cases) need specific connection properties to work
correctly (e.g. Sybase and CHARSET, according to the forum). The patch allows
users to specify these properties.

> How it is supposed to be used?
API clients can use the connection properties e.g. for
java.sql.Driver.connect(String url, Properties info)

IDE users can set the properties when creating a new database connection.

> And how to set properties?
It's true that if the API allows clients to create DatabaseConnection objects
and to get connection properties, it should also allow them to set the
connection properties. 
I've added a new variant of method DatabaseConnection.create that accepts a
Properties object.
Please check it.

(In reply to comment #8)
> > Do you know about any property that has to be set before testing the
> > connection?
> There are - informix need the INFORMIXSERVER property to be set either
> via JDBC  url or via properties. This has to be set to even be able to
> open a connection.
OK, so the properties need to be set in the same step.


Please review updated parts of the patch.
Thank you very much for your comments.

-- 
Configure bugmail: http://netbeans.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.


[Bug 197408] Enable editting of connection properties for db connections

jhavlin 02/12/2013

<Possible follow-up(s)>

[Bug 197408] Enable editting of connection properties for db connections

matthias42 02/12/2013

[Bug 197408] Enable editting of connection properties for db connections

jhavlin 02/13/2013

[Bug 197408] Enable editting of connection properties for db connections

jhavlin 02/13/2013

[Bug 197408] Enable editting of connection properties for db connections

sj-nb 02/13/2013

[Bug 197408] Enable editting of connection properties for db connections

matthias42 02/13/2013

[Bug 197408] Enable editting of connection properties for db connections

jhavlin 02/13/2013

[Bug 197408] Enable editting of connection properties for db connections

matthias42 02/14/2013

[Bug 197408] Enable editting of connection properties for db connections

jhavlin 02/20/2013

[Bug 197408] Enable editting of connection properties for db connections

jhavlin 02/21/2013

[Bug 197408] Enable editting of connection properties for db connections

quality 02/23/2013

Project Features

About this Project

DB was started in November 2009, is owned by Antonin Nebuzelsky, and has 113 members.
By use of this website, you agree to the NetBeans Policies and Terms of Use (revision 20131025.e7cbc9d). © 2013, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo
 
 
Close
loading
Please Confirm
Close