Issue 79921 - Resizing Chart might just resize the replacement image instead of getting a new one
Summary: Resizing Chart might just resize the replacement image instead of getting a n...
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-23 18:39 UTC by bjoern.milcke
Modified: 2013-02-24 21:21 UTC (History)
2 users (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 bjoern.milcke 2007-07-23 18:39:35 UTC 
1. Create a Calc document, enter some numbers
2. Create a Chart (Insert Chart, while the cursor is in a cell with a number)
3. Leave the chart's inplace mode
4. Resize the chart out-place (e.g. just make it wider)
=> You'll notice that the chart is recalculated, esp.: the font still has the
correct aspect ratio
5. Click on "Sheet2" (this swaps the chart out, maybe a running->loaded state
change?)
6. Click back to "Sheet 1"
7. Resize the chart again, (e.g. just make it wider)
=> The chart is NOT recalculated, esp.: the fonts are stretched, collapsed due
to a changed aspect ratio.

->MAV: I don't know if this is your place in the code to observe for this. This
is what I found out as difference between 4. and 7.:

When the chart is resized the method SdrOle2Obj::ImpSetVisAreaSize() is called.
In this method SfxInPlaceClient::GetClient() is called (see
svx/source/svdrwa/svdoole2.cxx:1534). In 4. this method returns an object, in 7.
it returns NULL. Maybe this is the error. Maybe the condition for a proper
resize "if ( pClient || bHasOwnClient )" some lines below is wrong.
Comment 1 mikhail.voytenko 2007-08-07 13:46:07 UTC 
The problem was that there was no client, and an object requires a correct
client to proceed with resizing. With the introduced in fwk72 fix the object
gets a lightwait client if there is no, so the replacement image is calculated
correctly.
Comment 2 mikhail.voytenko 2007-10-01 08:45:33 UTC 
MAV->KLA: Please verify the issue.
Comment 3 kla 2007-10-02 11:26:50 UTC 
seen ok in cws
Comment 4 p9w.vu.31122010 2008-02-07 10:33:04 UTC 
Verified fixed in m6, closing.