To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=56229





------- Additional comments from [EMAIL PROTECTED] Wed Oct 17 10:56:38 +0000 
2007 -------
fme->lijian: Thank you very much. Almost good, I think. Just two more remarks:

1) I think the first if statement in the for loop is not correct. Imagine the 
case

Table
  Row
    Cell 1
      Section
        Para 1
      /Section 
      Para2
    /Cell 1
    Cell 2
        Para 3
    /Cell 1
  /Row
/Table
  
Not, if cell1 is protected, lcl_FindNextCell does not correctly set the index to
Para3, instead it returns a value != 0 indicating a failure. So I think the if
statement should check if we have already reached the end of the (outer) table.
And aTmp only has to be incremented in case that pNd is a start node.

2) The last content of a cell does not necessarily have to be a text node in the
outer table, if can also be an inner table. Just try it: Select the whole
content of a table cell and execute "insert table". The outer table cell will
only contain a inner table cell with no surrounding text nodes. We should
consider this case as well.


---------------------------------------------------------------------
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]

Reply via email to