Comment 33 for bug 258705

Revision history for this message
In , julien2412 (serval2412-6) wrote :

(In reply to Leyan from comment #14)
> The bug is still present in version: 7.1.0.0.alpha1+
>
> I had a look, it is due to the "sort by X value" being done in the view part
> (in VDataSeries::doSortByXValues()) without updating the model data, while
> the tooltip text is generated by lcl_getDataPointValueText in
> ObjectNameProvider.cxx in the controller part, which has access to the model
> data, but not the view data and the VDataSeries correctly sorted.
>
> Does it mean the sort should be done elsewhere and apply to the model data
> as well?

Good job! Would you be interested in proposing a patch?
Indeed, it seems you made most of the way for this since I suppose you:
- downloaded LO sources
- built it
- analyzed the code + found root cause
- began to think about a fix
So the only things to do would be:
- provide license statement (just an email on LO dev)
- create gerrit account
- submit a patch
If yes, here's a link to take a look:
https://wiki.documentfoundation.org/Development/GetInvolved