all the date should be saved in the generalized time match format

Hello,

to be more in line with the other part of the application and to use the same format every where i think we should use generalized time match

this apply to all plugin having a date picker

(from redmine: issue id 4556, created on 2016-02-29, closed on 2017-01-24)

• Relations:
  • relates #4743 (closed)
• Changesets:
  • Revision 6f6ff93f by Côme Chilliet on 2016-04-26T02:37:46.000Z:

Fixes #4556 using the new GeneralizedTimeDateAttribute and LdapGeneralizedTime classes

I’m still trying to figure out how to handle this format in PHP.

It seems it can’t be done with DateTime, it can be approximated with "YmdHisO" but then it will output +0000 instead of Z which I’m not sure is well supported by applications. If the fraction feature is used it won’t work either.

I think I’m gonna have to extend DateAttribute to support this format with regexps

(from redmine: written on 2016-03-08)

I added a LdapGeneralizedTime helper class and a GeneralizedTimeDateAttribute type. It’s already in use in some plugins.

Possible candidates for a change of schema to GeneralizedTime format: aliasExpirationDate from alias plugin gosaVacationStart/gosaVacationStop from mail plugin

Should I change the type of these fields? How would we handle the conversion for existing installation? They are currently stored as timestamps, which seems standard enough. (Plus GeneralizedTime is not that easy to deal with, so for like sieve script or whatever it will be easier to have unix timestamp in vacation dates in my opinion)

(from redmine: written on 2016-04-26)

hello,

Alias seems a good candidate for me. gosaVacationStart/gosaVacationStop doesnt seems to be into the mail plugin ? i don't remember why

for migration as always in the fusiondirectory-setup command

(from redmine: written on 2016-04-28)

Benoit MORTIER wrote:

hello,

Alias seems a good candidate for me. gosaVacationStart/gosaVacationStop doesnt seems to be into the mail plugin ? i don't remember why

They only appear if the mail method says it can handle it. There are no mail method with support for this in the current plugins.

for migration as always in the fusiondirectory-setup command

It’s starting to be quite crowded already

(from redmine: written on 2016-04-29)

hello,

following the discussion of this morning, this should stay in 1.1 for now

Cheers

(from redmine: written on 2016-05-02)

Generalized time format is now in use in relevant attributes. If you think some attribute must use this format please open separate ticket.

(from redmine: written on 2017-01-24)

Close issue

(from redmine: written on 2017-01-24)

closed

added Changed label

removed Bugs label

added PJ1802-0188 label

added plugin-community plugin-personal plugin-ppolicy labels