Bug 115103 - Date field is formatted as number when mail merge is run on DOC(X) files
Summary: Date field is formatted as number when mail merge is run on DOC(X) files
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.4.0.3 release
Hardware: All All
: medium normal
Assignee: Aron Budea
URL:
Whiteboard: target:6.1.0
Keywords:
Depends on:
Blocks: Mail-Merge
  Show dependency treegraph
 
Reported: 2018-01-19 04:21 UTC by Aron Budea
Modified: 2018-11-02 20:33 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Sample starting DOCX (15.80 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2018-01-19 04:21 UTC, Aron Budea
Details
Sample starting DOC (22.50 KB, application/msword)
2018-01-19 04:21 UTC, Aron Budea
Details
Sample XLS to be used as an address list (25.00 KB, application/vnd.ms-excel)
2018-01-19 04:23 UTC, Aron Budea
Details

Note You need to log in before you can comment on or make changes to this bug.
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!