Documentation for toString() date format codes is incorrect

[pumalife]

value: 2024-01-17T00:00:00Z
value.toString('u')
should return the day number for the current week ( ie 1-7) as indicated in https://openrefine.org/docs/manual/grelfunctions#date-functions

but its returning 2024
ive tried this with a text value and converted it to date, manually specifying the format date text value. ie value.toDate('y/m/d').toString('u') and it still outputs 2024

[tfmorris]

The documentation is incorrect. You can find the correct field letters here: https://docs.oracle.com/javase/8/docs/api/java/time/format/DateTimeFormatter.html

The reference to https://docs.oracle.com/javase/8/docs/api/java/text/SimpleDateFormat.html is outdated, but we generally try to preserve compatibility, so I'm not sure what other incompatibilities are lurking here.

[tfmorris]

I went through the two pages and I think these are the differences

Old (SimpleDateFormat)	New (DateTimeFormatter)	Meaning	Comment
y	u	Year	y = year of era, which will be the same in most cases
F	[none]?	Day of week in month	no equivalent?
u	e	Day of week	e is actually "localized day of week"

[SymbolofMoon]

Hi @tfmorris ,

I have gone through the docs and found out that
E returns the day in Text (i.e in Tuesday) and e returns the number of week (2)

You can check the below screenshot.

Also please let me know if the issue is open for contribution

[mynk2611]

hello @tfmorris @thadguidry , i have solved this issue in PR #402

[thadguidry]

I'll have to defer to Tom on this particular issue, so that we get it 100% correct.