Issue 81755 - 'Table -> Split Table' crashes OOo
Summary: 'Table -> Split Table' crashes OOo
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: formatting (show other issues)
Version: OOo 2.3.0 RC1
Hardware: PC Windows XP
: P2 Trivial (vote)
Target Milestone: ---
Assignee: michael.ruess
QA Contact: issues@sw
URL:
Keywords: crash, regression
Depends on:
Blocks:
 
Reported: 2007-09-19 20:36 UTC by Rainer Bielefeld
Modified: 2013-08-07 14:42 UTC (History)
1 user (show)

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

Attachments
Sample document to reproduce the problem. (10.40 KB, application/vnd.oasis.opendocument.text)
2007-09-19 20:40 UTC, Rainer Bielefeld 		no flags Details
Different table models (9.80 KB, application/vnd.sun.xml.writer)
2007-09-25 09:57 UTC, andreas.martens 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Rainer Bielefeld 2007-09-19 20:36:21 UTC 
"2.3.0 RC3  Multilingual German version WIN XP: [680m5(Build9221)]" will crash
when you try to split the complex table in attached sample.odt

Steps to reproduce:

1. Open sample document
2. Mouseclick so that caret appears after colon after word "Fortsetzung"
3. press 'cursor right' and then 'cursor down'
4. Menu 'Table -> Split table'. Leave option "Copy heading" and press <OK>
   OOo will crash.

I sent 2 crash reportawth ID "r5uk3n" and "r4uk3n"
Comment 1 Rainer Bielefeld 2007-09-19 20:40:28 UTC 
Created attachment 48332 [details]
Sample document to reproduce the problem.
Comment 2 Rainer Bielefeld 2007-09-19 20:45:20 UTC 
This makes OOo 2.3.0 unusable for me, I will have to downgrade to 2.2
Comment 3 Rainer Bielefeld 2007-09-19 20:53:20 UTC 
Also reproducible with "2.3.0  Dev. Snapshot  WIN XP: [680m222(Build9183)]"
Comment 4 michael.ruess 2007-09-20 08:48:44 UTC 
MRU->AMA: looks like a regression of the new table model. Did not crash in OO 2.2.1.
You just need to place the cursor in the last cell (A9) on page 2, split the
table -> crash.
Comment 5 andreas.martens 2007-09-20 15:35:01 UTC 
ama->ama: Has to be fixed in ndtbl.cxx, SwNodes.SplitTables(..)
The new table has to get the same table model like the original table.
Comment 6 andreas.martens 2007-09-20 15:43:43 UTC 
BTW: Merging of different table models causes crashes, too.
Comment 7 Rainer Bielefeld 2007-09-20 16:58:13 UTC 
I see this as a show stopper for 2.3.0, especially due to "comments from ama Thu
Sep 20 14:43:43". I do not believe that we can risk such lots of crashes.
Comment 8 andreas.martens 2007-09-25 09:57:22 UTC 
Created attachment 48469 [details]
Different table models
Comment 9 andreas.martens 2007-09-25 10:01:46 UTC 
I've added a bug document with a lot of tables. It should not be possible to
merge tables of different table models because Writer becomes instable afterwards.
I fixed this "merge" issue in CWS sw8u9bf01, edtab.cxx.

And I fixed the crash of splitting old tables in ndtbl.cxx.

=> Fixed.
Comment 10 andreas.martens 2007-10-12 09:56:07 UTC 
Ready for QA
Comment 11 michael.ruess 2007-10-18 13:06:09 UTC 
Verified fix in CWS sw8u9bf01.
Comment 12 michael.ruess 2007-11-15 16:14:42 UTC 
Checked the fix in SRC680m236 and OOG680m9.