rather than relying on return values (send_package and prepare_package) move zipping of temporary files into the exporter class so it can be overridden during unit tests fix a small todo in mahara plugin