As someone who's put too much time in to the temporal attributes with no
real success, I'm +1 to any changes to improve it. It became a mess,
that we never managed to properly solved. I'd say feel free to even
just drop TemporalAttributeType and replace it with something more
sensible, even if it's more than one class. At one time I remember
thinking a few classes with a common superclass could have been useful.
I'm too far from the code now, but updates to it have my blessing, as
most any change to it will likely be an improvement. Just be sure to
test against cite 1.0 tests as well.
Chris
Justin Deoliveira wrote:
Hi all,
During my battle with wfs1.1 cite tests, one of the largest problems has
been handling dates properly. One of my major problems is that
TemporalAttributeType hard codes its class binding to a java.util.Date.
I need it to maintain the actually class passed in by whomever creating
the attribute. For instance, java.sql.Date, java.sql.Timestamp are used
to format dates so that they are jdbc friendly. Without them a normal
date will blow up.
So, I would like to add an additional constructor to
TemporalAttributeType to take the actual class of the type bindings. As
far as I can tell with just running the geotools tests this does not
impact much except for mif datastore. I have created a jira task with
the necessary patches.
http://jira.codehaus.org/browse/GEOT-1075
-Justin
--
Chris Holmes
The Open Planning Project
http://topp.openplans.org
begin:vcard
fn:Chris Holmes
n:Holmes;Chris
org:The Open Planning Project
adr:;;377 Broadway, 11th Floor;New York;NY;10013;USA
email;internet:[EMAIL PROTECTED]
title:VP, Strategic Development
x-mozilla-html:FALSE
url:http://topp.openplans.org
version:2.1
end:vcard
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
