Bug 115103

Summary: Date field is formatted as number when mail merge is run on DOC(X) files
Product: LibreOffice Reporter: Aron Budea <aron.budea>
Component: WriterAssignee: Aron Budea <aron.budea>
Status: RESOLVED FIXED    
Severity: normal CC: mikekaganski, xiscofauli
Priority: medium    
Version: 4.4.0.3 release   
Hardware: All   
OS: All   
Whiteboard: target:6.1.0
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 102998    
Attachments: Sample starting DOCX
Sample starting DOC
Sample XLS to be used as an address list

Description Aron Budea 2018-01-19 04:21:22 UTC
Created attachment 139207 [details]
Sample starting DOCX

- Open the attached DOC or DOCX, and run Tools -> Mail Merge Wizard...
- Skip to step 3, and add the attached XLS as address list.
- Do not specify an address block or salutation, just finish the merge.
- See the different merged values for the <Dates> field.

=> The dates are formatted as numbers instead (eg. 43099 instead of 2018-01-01).

Observed using LO 6.0.0.1 & 4.4.0.3 / Windows 7.
Comment 1 Aron Budea 2018-01-19 04:21:58 UTC
Created attachment 139208 [details]
Sample starting DOC
Comment 2 Aron Budea 2018-01-19 04:23:28 UTC
Created attachment 139209 [details]
Sample XLS to be used as an address list
Comment 3 Aron Budea 2018-01-29 11:17:12 UTC
Aron Budea committed a patch related to this issue.
It has been pushed to "master":

https://cgit.freedesktop.org/libreoffice/core/commit/?id=6feeb77552c38e3d9819611e9678470a44c00c84

tdf115103: find data source if sDataSource/sCommand are empty

It will be available in 6.1.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 4 Xisco Faulí 2018-05-02 09:24:44 UTC
A polite ping to Aron Budea: is this bug fixed? if so, could you
please close it as RESOLVED FIXED ? Thanks
Comment 5 Xisco Faulí 2018-08-02 08:59:27 UTC
(In reply to Xisco Faulí from comment #4)
> A polite ping to Aron Budea: is this bug fixed? if so, could you
> please close it as RESOLVED FIXED ? Thanks

ping...
Comment 6 Aron Budea 2018-08-02 12:04:53 UTC
(In reply to Xisco Faulí from comment #5)
> ping...

Ah, it needs a unit test, but I don't know how to approach that, so...
Comment 7 Xisco Faulí 2018-11-02 18:01:59 UTC
(In reply to Aron Budea from comment #6)
> (In reply to Xisco Faulí from comment #5)
> > ping...
> 
> Ah, it needs a unit test, but I don't know how to approach that, so...

Hi Aron,
You comment is from 3 months ago. Do you think we should just close this one without having a unittest ?
Comment 8 Aron Budea 2018-11-02 20:33:19 UTC
Yes, we should do that, I don't have the time (and the enthusiasm) to investigate how this could be tested properly. Thanks for the reminder!