Bug 99997 - XPageCursor reports incorrect page numbers and xFootnotesSupplier.getFootnotes() empty (unless document is refreshed via XRefreshable)
Summary: XPageCursor reports incorrect page numbers and xFootnotesSupplier.getFootnote...
Status: RESOLVED INSUFFICIENTDATA
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: sdk (show other bugs)
Version:
(earliest affected)
5.1.2.2 release
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: needsDevAdvice
Depends on:
Blocks:
 
Reported: 2016-05-23 01:22 UTC by Stefan Corneliu Petrea
Modified: 2017-03-01 10:36 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
attached a document with footnotes (it can be used to reproduce the problem) (518.33 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2016-05-23 01:22 UTC, Stefan Corneliu Petrea
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Corneliu Petrea 2016-05-23 01:22:56 UTC
Created attachment 125237 [details]
attached a document with footnotes (it can be used to reproduce the problem)

So, if a document is not refreshed(via the usual way with XRefreshable, I'm specifically talking about the Writer application, because that's where
this situation was found), two things happen:
a) The page numbers reported by XPageCursor are off(confirmed on both 5.1.2.2 and 4.2.8.2)
b) The footnotes are not loaded at all (confirmed on 4.2.8.2). In effect, on a XSearchable search they do not show up in the results, but if the document is refreshed prior to doing the search, they show up just fine.

My suggestion is that either this should be specifically mentioned in the documentation and reflected in the UNO API examples or it should not be required for a user to refresh the document, and it should happen implicitly as part of the UNO API default behaviour.

I view this as the intuitive way of solving this problem.
I look forward to your thoughts on this.
Thank you for taking the time to read the bug report
Comment 1 Stefan Corneliu Petrea 2016-05-23 01:29:21 UTC
My concern is mostly on the consistency and expecations side.
Because if this was mentioned in the documentation, I'd be like, I'm ok with that
because I read it was a problem and I should take care of it, but I didn't find it in the docs.
On the other hand, I feel like this is a sane default to have, maybe ? not sure.
I mean, refreshing the document by default would incur some additional processing. Maybe the workload generated by that would be too high?
Comment 2 Aron Budea 2016-06-28 04:23:00 UTC
Would you mind providing a minimal, but complete code example that could be used to reproduce the issue together with the document?
Comment 3 QA Administrators 2017-01-31 00:29:40 UTC Comment hidden (obsolete)
Comment 4 QA Administrators 2017-03-01 10:36:56 UTC
Dear Bug Submitter,

Please read this message in its entirety before proceeding.

Your bug report is being closed as INSUFFICIENTDATA due to inactivity and
a lack of information which is needed in order to accurately
reproduce and confirm the problem. We encourage you to retest
your bug against the latest release. If the issue is still
present in the latest stable release, we need the following
information (please ignore any that you've already provided):

a) Provide details of your system including your operating
   system and the latest version of LibreOffice that you have
   confirmed the bug to be present

b) Provide easy to reproduce steps – the simpler the better

c) Provide any test case(s) which will help us confirm the problem

d) Provide screenshots of the problem if you think it might help

e) Read all comments and provide any requested information

Once all of this is done, please set the bug back to UNCONFIRMED
and we will attempt to reproduce the issue. Please do not:

a) respond via email 

b) update the version field in the bug or any of the other details
   on the top section of our bug tracker

Warm Regards,
QA Team

MassPing-NeedInfo-20170301