API: Add more enum classes
ClosedPublic

Authored by staniek on Jul 17 2017, 5:46 PM.

Details

Summary

Add more enum classes.

FIXED-IN:3.1.0

Test Plan

Build KReport with tests enabled, run tests, run KReportExample.

Diff Detail

Repository
R14 KReport
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
staniek created this revision.Jul 17 2017, 5:46 PM
Restricted Application added a project: KReport. · View Herald TranscriptJul 17 2017, 5:46 PM
This revision was automatically updated to reflect the committed changes.

Some more qustions @piggz ...

src/common/KReportDocument.cpp
196

@piggz Change in logic but unsure here. Don't we want to iterate until PageFooterAny here?

241

Same question here - PageFooterAny?

src/wrtembed/KReportDesigner.cpp
1224

Same question here - PageFooterAny?

1268

Same question here - GroupFooter ?

piggz added inline comments.Jan 15 2018, 6:01 PM
src/common/KReportDocument.cpp
196

GroupFooter is the last section type in the enum.....but yes, section() doesnt handle Group* sections anyway, so will ignore the last 2.

241

Yes, section() will ignore Group* sections anyway.

src/wrtembed/KReportDesigner.cpp
1224

yes, should be fixed.

1268

yes, should be fixed.