directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Karasulu <aok...@bellsouth.net>
Subject Re: failing testGetGeneralizedTime()
Date Wed, 22 Dec 2004 15:13:09 GMT
Please proceed...
Thanks,
Alex

Enrique Rodriguez wrote:

> Alex et al,
>
> Regarding the failing timezone test, the draft reference at:
>
> http://ietf.org/internet-drafts/draft-ietf-ldapbis-syntaxes-09.txt
>
> states:
>
> "The "Z" form of <g-time-zone> SHOULD be used in preference to 
> <g-differential>."
>
> but in org.apache.eve.jndi.ibs.OperationalAttributeService you are 
> using <g-differential> 5 times with no explicit Z TimeZone to set the 
> String time on modified attributes.
>
> I propose:
>
> 1)  Move to "The "Z" form of <g-time-zone>" for this function, per the 
> draft.
>
> 2)  Replace DateUtils with the following 3 lines of code, as a 
> convenience method in the OperationalAttributeService, since that is 
> the only place this is used (best I can tell, quick IDE reference check).
>
>
> private String getGeneralizedTimeStamp()
> {
>   SimpleDateFormat date = new SimpleDateFormat( "yyyyMMddHHmmss'Z'" );
>   date.setTimeZone( new SimpleTimeZone(0,"Z") );
>   return date.format( new Date() );
> }
>
> ... or is there a reason to use <g-differential> in that service?  Say 
> the word and I'll commit this change.
>
> -enrique
>
>
>


Mime
View raw message