double clicking on a table in sybase no longer works because the sybase jdbc driver doesn't flag itself as needing upper/lower/mixed case so Bookmark.filterQuoteName defaults to converting to upper case. Arguably, the sybase jdbc driver is broken, but I'm not going to hold my breath for a fix so it seems to make more sense for Bookmark.filterQuoteName to default to the same casing that was passed. If this causes problems for other databases, then maybe it would make sense for the db adapter class to provide the flag based on its metadata, then it could be overriden for broken implementations like sybase.
Patch attached which changes the default action.
Log in to post a comment.