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





------- Additional comments from [email protected] Wed Jan  6 13:01:29 +0000 
2010 -------
AW: Indeed, when commenting in SdrGrafObj::SetGraphic(..) the lines
//      SetChanged();
//      BroadcastObjectChange();
all works well. The problem is indeed that SdrGrafObj::SetGraphic(..) gets
called from SdrGraphicLink::DataChanged(..).
Test shows that it's even enough to comment the 2nd line, thus the problem
really is the model change broadcast. The 1st line just leads to more repaints.
All this is triggered by SdrGrafObj::ImpUpdateGraphicLink() calls. AFAIK the
SlidePane refreshes on ModelChange the page preview for the page mentioned in
the hint. This re-creates the preview graphic and this needs to reload the 
graphic.
The tragedy here is that SdrGraphicLink::DataChanged(..) does not even INTEND to
change the model; it saves it's state and restores it in the most cases. This is
an UNWANTED model change (!)
What is (and was always) missing here is a SdrGrafObj::NbcSetGraphic(..). I
guess the one who added to rescue the mode state just did'nt want to add it.
Testing with adding it. Both commented lines will me moved to be not in the Nbc
method...

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