Issue 86131 - WW8: document properties imported as empty fields
Summary: WW8: document properties imported as empty fields
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: open-import (show other issues)
Version: OOH680m7
Hardware: All All
: P2 Trivial (vote)
Target Milestone: ---
Assignee: michael.ruess
QA Contact: issues@sw
URL:
Keywords: regression
Depends on:
Blocks: 84957
  Show dependency tree
 
Reported: 2008-02-15 13:48 UTC by michael.ruess
Modified: 2013-08-07 14:43 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
Sample document (30.00 KB, application/msword)
2008-02-15 13:51 UTC, michael.ruess
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description michael.ruess 2008-02-15 13:48:13 UTC
Open attached document, the document properties (like title, comments and all
the statistic fields) are now imported as fields (mainly empty). "Edit.Fields"
command on these fields result in a crash.
In previous versions (like in 2.3.1) these fields.
MRU->MBA: maybe this was introduced by the "customproperties" CWS - if not could
you please reassign this issue to a responsible engineer? Thanks!
Comment 1 michael.ruess 2008-02-15 13:51:11 UTC
Created attachment 51530 [details]
Sample document
Comment 2 michael.ruess 2008-02-15 13:52:49 UTC
Description should be:
[...]
In previous versions (like in 2.3.1) these fields were imported as plain text.
[...]
Comment 3 mdxonefour 2008-02-15 14:01:57 UTC
added md to cc
added keyword regression
Comment 4 Mathias_Bauer 2008-02-16 00:18:04 UTC
This is an unfortunate result of a missmatch between three different steps:

(1) some time ago "custom properties" have been imported (import of all property
streams from Word, Excel and Powerpoint documents)
(2) Caolan's great patch identified fields referencing the "built-in" properties
(from "summary information" stream), leaving all others as "unknown".
(3) I added code that imports the "unknown" fields into custom fields - but
there is a third kind of properties: "built-in" props from Word in the "document
summary information" stream that I wasn't aware of and we will need a matching
table to import them properly (they are only identifiable by their Id in the
stream like the other "built-in" properties). The problem is that these
properties are not recognized as custom properties or built-in properties and so
the fields don't find any reference. This leads to the empty text. 

The crash happens because the dialog (IMHO) is buggy - it can't cope with an
empty listbox. Usually that doesn't happen but it's always better to be prepared
for this. Easy to fix that part. The more tricky stuff is how to make the fields
work for these properties.

That's the root cause, now I'm thinking about the solution. I have three
different ideas and currently I'm wandering which one is the best. 
Comment 5 Mathias_Bauer 2008-02-18 10:53:50 UTC
fixed in CWS builtinpropsfix
Comment 6 michael.ruess 2008-02-19 11:43:14 UTC
Fix verified in CWS builtinpropsfix.
Comment 7 michael.ruess 2008-02-20 09:16:49 UTC
Reassigning back to myself.
Comment 8 michael.ruess 2008-03-05 14:30:14 UTC
Checked in OOH680m8.
Comment 9 Mathias_Bauer 2008-03-26 12:48:17 UTC
Michael, please verify in 3.0 version also. 
Comment 10 michael.ruess 2008-03-27 16:37:20 UTC
Fixed.
Comment 11 michael.ruess 2008-03-27 16:38:29 UTC
Verified now also in OO3 CWS sw30bf02.
Comment 12 michael.ruess 2008-07-09 13:05:00 UTC
Checked in 300m23.