In last week's email I mentioned that I had submitted a patch to Gerrit that splits up ORptExport::exportContainer into some logical chunks.
I spent the end of last week going through code review for the patch with my mentor Michael, and I'm happy to report the patch has been merged. Thanks for all the help and patience Michael! The patch can be viewed here https://gerrit.libreoffice.org/c/core/+/188189 During my weekly meeting with my mentors it was decided that it would be greatly beneficial to break all of the work I have done up until now into several commits and submit them as one Gerrit patch. Locally I had several branches that I had been working on different aspects of the project and so I've spent the first part of this week merging these into one branch and slightly refining the commits so that each commit is one logical change to the code base. I also continued work on the new export filter. I'm happy to report that I've succeeded in implementing the second filter and changing the save/export paths that start here https://opengrok.libreoffice.org/xref/core/reportdesign/source/core/api/ReportEngineJFree.cxx?r=9a9334b6926829fb890a48047af3bd76571bf542#175 With my latest patch you can save or export a Report without using the Pentaho Report Engine. Unfortunately the patch isn't ready for Gerrit as while trying to reduce some code redundancy between the ORptExport(old filter) and ORptExport2(new filter) I've crossed some wires and the output is a bit mangled at the moment. I should have it sorted by tomorrow afternoon and will update it to Gerrit when it's fixed. After that's finished the plan remains to move on to implementing Grouping which I hope to be able to talk about next week. That's it for now, have a good week, Adam Seskunas
