Issue 53165 - Drag'n'Drop of multi-selection in outliner crashes office
Summary: Drag'n'Drop of multi-selection in outliner crashes office
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: ui (show other issues)
Version: 680m123
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: wolframgarten
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-10 14:21 UTC by wolframgarten
Modified: 2005-08-13 14:16 UTC (History)
1 user (show)

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

Attachments
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 wolframgarten 2005-08-10 14:21:21 UTC 
Open the posted bugdoc, select two slides in outline view and move them to
another place: Office crashes. This is already visible in m106. Should the
target be set to 2.0.1?
Comment 1 clippka 2005-08-10 14:26:58 UTC 
Yes
Comment 2 clippka 2005-08-11 16:38:07 UTC 
Fixed,

the problem was that ParagraphInsertedHdl was not guarded against page order
changes. This issue didn't surface if only one slide at a time was inserted,
like when you do it manualy with the keyboard. Fixed by adding a 

OutlineViewPageChangesGuard aGuard(this);

to OutlineView::ParagraphInsertedHdl
Comment 3 clippka 2005-08-12 10:56:25 UTC 
same root cause as i53165
Comment 4 clippka 2005-08-12 10:56:50 UTC 
changing target
Comment 5 groucho266 2005-08-12 11:05:21 UTC 
Reviewed.
Comment 6 clippka 2005-08-12 11:58:46 UTC 
correction, same root cause as i53218
Comment 7 wolframgarten 2005-08-12 13:11:46 UTC 
Verified in CWS.
Comment 8 clippka 2005-08-12 13:31:43 UTC 
back to qa
Comment 9 clippka 2005-08-12 13:33:33 UTC 
.
Comment 10 clippka 2005-08-12 13:37:58 UTC 
set back to fixed
Comment 11 clippka 2005-08-12 13:38:50 UTC 
set back to verified
Comment 12 wolframgarten 2005-08-13 14:16:39 UTC 
Tested in master m124 on linux, windows, solaris. Closed.