Move script sources out of KReportData, into its own interface
Details
Details
kreportexample still runs. With associated changes to kexi, scirpting works in reports
Diff Detail
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.
Comment Actions
This pdf demonstrates a report with the section backgrounds set vi JS, and the PageNumber at the bottom of each page set by JS.
Comment Actions
Nice...
src/common/KReportScriptSource.h | ||
---|---|---|
25 | Not needed include | |
32 | Move the body to .cpp, can be in KReportScriptHandler.cpp | |
src/renderer/KReportPreRenderer_p.h | ||
73 | Let's not use m_ here. Other uses of m_ can be removed in a separate task (would be nice). | |
src/renderer/scripting/KReportScriptHandler.cpp | ||
99 | All this debug not needed in the repo | |
168–178 | Better: QMessageBox::setText: tr("Cannot evaluate script. Error: %1.") QMessageBox::setDetailedText: "Script code:\n%1" | |
src/wrtembed/KReportDesigner.cpp | ||
445 | Not needed in the repo | |
805 | Not needed in the repo | |
806 | Not needed in the repo | |
806–808 | QString() |
src/renderer/scripting/KReportScriptHandler.cpp | ||
---|---|---|
108 | not needed code |