Expose and use Engine's page size variable
ClosedPublic

Authored by leinir on Jun 12 2017, 12:40 PM.

Details

Summary

Engine has a pageSize variable which has been mostly unused, but which comes in very handy when getting large sets of data. This patch exposes that variable through a setter, and makes use of it internally whenever a request is made (so the page size is actually honoured the way it seems to have been intended)

Diff Detail

Repository
R304 KNewStuff
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
leinir created this revision.Jun 12 2017, 12:40 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJun 12 2017, 12:40 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
apol added a subscriber: apol.Jun 12 2017, 1:12 PM

I have no idea about the patch itself, why it wasn't exposed so far that is.

src/core/engine.cpp
144

I wouldn't pass an int as const&

As far as i can gather, it was simply never added because, well, it was never used in a lot of places... This really is more a case of equalising some features between requestData and other parts of the engine (so they can all be paginated by the size they really want to be).

src/core/engine.cpp
144

No, quite true - force of habit and all that :)

leinir updated this revision to Diff 15377.Jun 12 2017, 1:25 PM

Don't const & an int, that's just silly.

apol accepted this revision.Jun 12 2017, 4:57 PM

It all makes sense, so good from my side.

This revision is now accepted and ready to land.Jun 12 2017, 4:57 PM
This revision was automatically updated to reflect the committed changes.