To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=76689
User od changed the following:
What |Old value |New value
================================================================================
Assigned to|od |cmc
--------------------------------------------------------------------------------
------- Additional comments from [EMAIL PROTECTED] Mon Apr 30 12:52:18 +0000
2007 -------
OD->CMC:
I've reviewed your patch. Unfortunately, it doesn't work as excepted. Please
revise your patch.
I've found the following problems:
(1) It doesn't work with the attached document <testWW8.doc>, because the
condition to trigger the restart of the sequence numbers doesn't become true in
method <SwWW8ImplReader::SetTxtFmtCollAndListLevel(..)>. I think the condition
should be pTxtNode->GetOutlineLevel() <> NO_NUMBERING and then method
<ResetLevelTiedSequences(..)> should be called with parameter
pTxtNode->GetOutlineLevel()
(2) The data structure <SwWW8ImplReader::maResetSequences> is filled up with
duplicate entries again and again, when a sequence field is imported and no need
for restart is found.
(3) Investigation of Microsoft Word 2003 reveals, that the switch \s can only
have an effect on the field, which has this switch. In your solution the \s
switch of a field has effect on the next field of the same sequence - see
attached document.
(4) Investigation of Microsoft Word 2003 reveals, that the switch \s has no
effect, if switch \r comes before the \s switch, but it can have an effect, if
the switch \r comes after the \s switch - see attached document.
(5) Investigation of Microsoft Word 2003 reveals, that the switch \s has no
effect, if the field isn't the first item after the corresponding heading - see
attached document.
(6) Your code doesn't consider, that the document can contain fields of the same
sequence with switch \s for different outline levels.
(7) Investigation of Microsoft Word 2003 reveals, that switch \r has no effect,
if switch \s comes before it, regardless of the situation, if the switch \s has
an effect or not. Can you please consider this in your revised patch.
---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
