Tim Bentley has proposed merging lp:~trb143/openlp/bugs into lp:openlp. Requested reviews: Andreas Preikschat (googol-hush) Raoul Snyman (raoul-snyman) Related bugs: Bug #693150 in OpenLP: "Custom Slide Display footer option" https://bugs.launchpad.net/openlp/+bug/693150 Bug #693202 in OpenLP: "delete theme" https://bugs.launchpad.net/openlp/+bug/693202 Bug #712252 in OpenLP: "Service song already in database detection logic broken" https://bugs.launchpad.net/openlp/+bug/712252 Bug #733271 in OpenLP: "Traceback with Song Usage Extract " https://bugs.launchpad.net/openlp/+bug/733271
For more details, see: https://code.launchpad.net/~trb143/openlp/bugs/+merge/53071 Stop report generation with no path set -- https://code.launchpad.net/~trb143/openlp/bugs/+merge/53071 Your team OpenLP Core is subscribed to branch lp:openlp.
=== modified file 'openlp/plugins/songusage/forms/songusagedetailform.py' --- openlp/plugins/songusage/forms/songusagedetailform.py 2011-02-24 05:47:38 +0000 +++ openlp/plugins/songusage/forms/songusagedetailform.py 2011-03-11 19:26:17 +0000 @@ -88,6 +88,15 @@ """ log.debug(u'accept') path = unicode(self.fileLineEdit.text()) + if path == u'': + Receiver.send_message(u'openlp_error_message', { + u'title': translate('SongUsagePlugin.SongUsageDetailForm', + 'Output Path Not Selected'), + u'message': unicode(translate( + 'SongUsagePlugin.SongUsageDetailForm', 'You have not set a ' + 'valid output location for your song usage report. Please ' + 'select an existing path on your computer.'))}) + return check_directory_exists(path) filename = unicode(translate('SongUsagePlugin.SongUsageDetailForm', 'usage_detail_%s_%s.txt')) % (
_______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : openlp-core@lists.launchpad.net Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp