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
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
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?

242

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.

242

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

src/wrtembed/KReportDesigner.cpp
1224

yes, should be fixed.

1268

yes, should be fixed.