New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ChartUpdater does not work with Waterfall Charts. #265

Open
ahmet269 opened this Issue Dec 13, 2018 · 3 comments

Comments

Projects
None yet
2 participants
@ahmet269
Copy link

ahmet269 commented Dec 13, 2018

I am trying to update data in waterfall chart in word but chartupdater is unable to do this. ChartUpdater does not exist as ChartPart on the MainDocumentPart of word document but it exists as an ExtendedPart. Have you any ideas how to make this work in chart updater?

@DanielBailey1001001

This comment has been minimized.

Copy link

DanielBailey1001001 commented Dec 15, 2018

This may be a similar fix to #247, find this line of code and add your waterfall chart to the list:
else if (chartType == C.doughnutChart || chartType == C.ofPieChart || chartType == C.pie3DChart || chartType == C.pieChart)

Here is the link to the similar pull request that adds a radar chart:
f945426

@ahmet269

This comment has been minimized.

Copy link

ahmet269 commented Dec 15, 2018

Daniel thanks for the comment. This won’t work because the actual tags are different as well. Things like C.ser or C.f Xnames don’t work with waterfall charts because waterfall chart xml comes through with tag names cx:ser for example rather than c:ser.

I started to add code to handle these to chartupdater.cs so I am expanding it.

Just adding the chart type won’t work. Already tried that then I start to compare the xml and that’s when I realised it it different.

@ahmet269

This comment has been minimized.

Copy link

ahmet269 commented Jan 4, 2019

I managed to fix this issue by changing the current chart updater class and introducing the handler for waterfall chart. Xml structure is completely different as the waterfall chart is not a chart part of the document but it is a extended part. plus the namespace ends in chartex and chart so it needed quite bit of changing to handle both current charts and extended parts as well. plus found some more issue in the chart updater class where if you had multiple series, it was only using the properties of the firts series, this means that any properties you apply to different series were getting lost. i managed to fix this as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment