Issue 36440 - crash on Undo in Impress
Summary: crash on Undo in Impress
Status: CLOSED DUPLICATE of issue 32921
Alias: None
Product: Impress
Classification: Application
Component: code (show other issues)
Version: 680m58
Hardware: All Linux, all
: P2 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: wolframgarten
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-11-01 12:16 UTC by mandreiana
Modified: 2004-11-05 07:34 UTC (History)
1 user (show)

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

Attachments
crash report (53.20 KB, application/vnd.sun.xml.impress)
2004-11-01 12:17 UTC, mandreiana 		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 mandreiana 2004-11-01 12:16:32 UTC 
I knew there were issues with Undo text in presentations in m56, so I tested in m58:

Start a new presentation
Insert slide -> choose 3rd type (header and bulleted text entries)
Insert text in a bulleted entry, press Enter
Press Ctrl+Z to Undo -> crash.

Attaching generated crash dump.
Comment 1 mandreiana 2004-11-01 12:17:03 UTC 
Created attachment 18800 [details]
crash report
Comment 2 wolframgarten 2004-11-01 12:45:07 UTC 
Crash when undoing text insertion is reproducible. Reassigned to Christian.
Comment 3 clippka 2004-11-02 14:32:10 UTC 
The problem is that during text edit the undo manager for text is not active. So
while doing text edit and executing undo, the last undo action from the drawing
layer undo manager is executed. In this case its the new page layout that
deletes the text object that is currently in edit mode causing a crash later
Comment 4 groucho266 2004-11-02 14:40:17 UTC 
Accepted/
Comment 5 mandreiana 2004-11-04 07:34:50 UTC 
Issue 36630 describes the same symptoms, but in Draw. Is that related?
Comment 6 groucho266 2004-11-04 15:55:20 UTC 
Duplicate, see below.
Comment 7 groucho266 2004-11-04 15:58:18 UTC 
This bug is fixed as a side effect of issue 32921.  Therefore, I set it to
duplicate.

One of the things issue 32921 fixes is the wrong order in which shells are
pushed on the shell stack by the ViewShellManager.  As a result the undo manager
of the edit engine is registered at the wrong shell.

*** This issue has been marked as a duplicate of 32921 ***
Comment 8 groucho266 2004-11-04 16:38:07 UTC 
As suggested above issue 36630 is another duplicate (to issue 32921).
Comment 9 wolframgarten 2004-11-05 07:34:38 UTC 
Closed.