InfoWindow : Reference Date+Time only displays Date in grid

Description

Steps to reproduce :
Log in System and open Info Invoice
Add a new column (Created / Reference = Date+Time) and save it.
Then, open the Info invoice panel in GW. Refresh. Created column only displays Date (time is missing).

Regards,

Nicolas

Environment

None

Activity

Show:
Nicolas Micoud
July 6, 2018, 8:11 AM

Hi ,
Tested and approved !

But I think it could be improved later by adding the VFormat column in AD_InfoColumn in order to "force" the desired format.
It could be confusing to have '06/07/2018' for Date and 6 juill 2018 00:00:00 CET".

I would expect something like "06/07/2018 00h00" ; that could be achived in another ticket

Nicolas

redhuan d. oon
July 6, 2018, 10:57 AM
Edited

OK done! The Date type will not be affected by this force pattern meant only for DateTime.
Pattern is "dd/M/yyyy hh:mm"
See attached screenshot

- new patch:

Carlos Ruiz
July 9, 2018, 5:28 PM

Thanks .

I committed your fix, but changed a bit the rendering.

Now it works also for Time reference, but I reverted the fixed formatting of the timestamp, because that must not be hardcoded, the date format - and the order how month, day, year are printed is language dependant, to hardcoding it would go against i18n.

, if we want some kind of configurable formatting I think it would be worthy to work on it on a different ticket - actually from what I remember the field AD_Language.DatePattern is doing nothing, maybe it could be used for that purpose and adding also DateTimePattern and TimePattern.

Regards,

Carlos Ruiz

Hiep Lq
July 14, 2018, 3:28 PM

raise index out of range exception when click posted button to see accounting info

Carlos Ruiz
July 14, 2018, 4:03 PM

Thanks , committed

Assignee

redhuan d. oon

Reporter

Nicolas Micoud

Labels

Tested By

None

Priority

Major
Configure