Issue 71244

Summary:	new chart: Updates of charts in writer
Product:	General	Reporter:	IngridvdM
Component:	chart	Assignee:	michael.ruess
Status:	CLOSED FIXED	QA Contact:	issues@graphics <issues>
Severity:	Trivial
Priority:	P3	CC:	issues, tony.galmiche.ooo
Version:	3.3.0 or older (OOo)	Keywords:	new_implementation
Target Milestone:	---
Hardware:	All
OS:	All
Issue Type:	TASK	Latest Confirmation in:	---
Developer Difficulty:	---

Description IngridvdM 2006-11-06 10:17:21 UTC

Consider charts in writer which get their data from writer tables.
When the data in the table is changed after the chart was created there are
different possibilities for the user to update the chart thus it displays the
new values from the table:

1: clicking F9 in a table does update all charts that get data from that table
2: choosing from the menu Tools/Update/Update All or Update All charts does
update all charts independently from where you are with teh cursor

Thomas, please imlpement this for the new chart also. When removing the old
chart code please check that there is no additional possibility overseen to
update chart data.

Comment 1 IngridvdM 2006-11-06 10:25:50 UTC

Thomas, please use the new API to provide the updated data, see Service:
::com::sun::star::chart2::data::DataSequence. This service describes an
interface XModifyBroadcaster. Please send the change notification after the
according user actions in writer and provide the new data within the Data Sequences.

Whne a chart is swapped out by OLE caching then please just load it.
Do not set the chart data via the old memchart nor via the old chart API anymore
please.
Thanks Ingrid

Comment 2 thomas.lange 2006-11-07 08:32:07 UTC

Comment 3 IngridvdM 2006-11-22 10:36:36 UTC

*** Issue 71844 has been marked as a duplicate of this issue. ***

Comment 4 thomas.lange 2006-12-07 13:20:02 UTC

TL->QA: Some things worthwhile to be checked to see if updating of chart is working.
(Please note that I do not know if they are working in the current old chart. If
they don't they need not to function right now with the new chart.)
Things to check updating for:
- add a single or more rows or columns
- remove a single or more rows or columns
- splitting and merging of adjacent tables
- removing the table
- changing the number format (will probably require a manual update)
- changing the content number <-> text or text with multiple paragraphs
  (will probably require a manual update)
- sorting of a table (will probably require a manual update)
- split join several cells of a table

Comment 5 thomas.lange 2006-12-13 15:04:45 UTC

Aside from 'undo' which still needs to be taken care of everything should be
functional now.
TL->MRU: Please check. Thanks!

Comment 6 thomas.lange 2006-12-14 10:24:23 UTC

Comment 7 IngridvdM 2007-01-18 11:48:14 UTC

changed target to 2.3

Comment 8 thomas.lange 2007-01-23 11:26:20 UTC

See also issue 73791 for the follow-up that is to fix the undo related functions.

Comment 9 thomas.lange 2007-02-19 12:49:46 UTC

TL->MRU: Please not that adding sth. to a table *may* update formula fields but
not the chart since chart basically has no means to get informed about e.g. a
new row.
However this depends on which cells the chart operates and where the new row
gets added. In some cases the new data might be included in others not.
If you like to be more precisely informed about this please ask.

Comment 10 michael.ruess 2007-04-13 14:19:07 UTC

Verified in CWS chart2mst3. Updating charts in Writer now works fine.

Comment 11 michael.ruess 2007-06-21 16:05:44 UTC

Checked in 680m214.