Clean up SongUsage so it now works again

openlp/core/ui/maindisplay.py

@@ -241,11 +241,8 @@ class MainDisplay(DisplayWidget):
             self.display_text.setPixmap(QtGui.QPixmap.fromImage(self.blankFrame))
         else:
             self.displayBlank = False
-            if self.display_text_frame:
+            if self.display_frame:
                 self.frameView(self.display_frame)
-#        if blanked != self.parent.LiveController.blankButton.isChecked():
-#            self.parent.LiveController.blankButton.setChecked(self.displayBlank)
-#        self.parent.generalConfig.set_config(u'screen blank', self.displayBlank)
 
     def displayAlert(self, text=u''):
         """

openlp/core/ui/thememanager.py

@@ -249,8 +249,9 @@ class ThemeManager(QtGui.QWidget):
         log.debug(u'Load themes from dir')
         self.themelist = []
         self.ThemeListWidget.clear()
-        for root, dirs, files in os.walk(self.path):
-            for name in files:
+        #root, dirs, files = os.walk(self.path)
+        dirList = os.listdir(self.path)
+        for name in dirList:
             if name.endswith(u'.png'):
                 #check to see file is in theme root directory
                 theme = os.path.join(self.path, name)

openlp/plugins/songusage/forms/songusagedeleteform.py

@@ -33,11 +33,11 @@ class SongUsageDeleteForm(QtGui.QDialog, Ui_SongUsageDeleteDialog):
     """
     Class documentation goes here.
     """
-    def __init__(self, auditmanager, parent=None):
+    def __init__(self, songusagemanager, parent=None):
         """
         Constructor
         """
-        self.auditmanager = auditmanager
+        self.songusagemanager = songusagemanager
         QtGui.QDialog.__init__(self, parent)
         self.setupUi(self)
 
@@ -52,5 +52,5 @@ class SongUsageDeleteForm(QtGui.QDialog, Ui_SongUsageDeleteDialog):
         if ret == QtGui.QMessageBox.Ok:
             qDeleteDate = self.DeleteCalendar.selectedDate()
             deleteDate = date(qDeleteDate.year(), qDeleteDate.month(), qDeleteDate.day())
-            self.auditmanager.delete_to_date(deleteDate)
+            self.songusagemanager.delete_to_date(deleteDate)
         self.close()

openlp/plugins/songusage/forms/songusagedetailform.py

@@ -25,10 +25,14 @@
 import os
 
 from PyQt4 import QtCore, QtGui
+import logging
 
 from songusagedetaildialog import Ui_SongUsageDetailDialog
 
 class SongUsageDetailForm(QtGui.QDialog, Ui_SongUsageDetailDialog):
+    global log
+    log = logging.getLogger(u'SongUsageDetailForm')
+    log.info(u'SongUsage Detail Form loaded')
     """
     Class documentation goes here.
     """
@@ -106,19 +110,19 @@ class SongUsageDetailForm(QtGui.QDialog, Ui_SongUsageDetailDialog):
         self.close()
 
     def detailedReport(self):
-        print "detailed"
-        filename = u'audit_det_%s_%s.txt' % \
+        log.debug(u'Detailed report generated')
+        filename = u'usage_detail_%s_%s.txt' % \
             (self.FromDateEdit.date().toString(u'ddMMyyyy'),
              self.ToDateEdit.date().toString(u'ddMMyyyy'))
-        audits = self.parent.auditmanager.get_all_audits()
+        usage = self.parent.songusagemanager.get_all_songusage()
         outname = os.path.join(unicode(self.FileLineEdit.text()), filename)
         file = None
         try:
             file = open(outname, u'w')
-            for audit in audits:
+            for instance in usage:
                 record = u'\"%s\",\"%s\",\"%s\",\"%s\",\"%s\",\"%s\"\n' % \
-                    (audit.auditdate,audit.audittime, audit.title,
-                    audit.copyright, audit.ccl_number , audit.authors)
+                    (instance.usagedate,instance.usagetime, instance.title,
+                    instance.copyright, instance.ccl_number , instance.authors)
                 file.write(record)
         except:
             log.exception(u'Failed to write out audit records')
@@ -127,8 +131,7 @@ class SongUsageDetailForm(QtGui.QDialog, Ui_SongUsageDetailDialog):
                 file.close()
 
     def summaryReport(self):
-        print "summary"
+        log.debug(u'Summary report generated')
         filename = u'audit_sum_%s_%s.txt' % \
             (self.FromDateEdit.date().toString(u'ddMMyyyy'),
              self.ToDateEdit.date().toString(u'ddMMyyyy'))
-        print filename

openlp/plugins/songusage/songusageplugin.py

@@ -141,7 +141,7 @@ class SongUsagePlugin(Plugin):
             SongUsageitem.authors = u''
             for author in SongUsageData[1]:
                 SongUsageitem.authors += author + u' '
-            self.songusagemanager.insert_SongUsage(SongUsageitem)
+            self.songusagemanager.insert_songusage(SongUsageitem)
 
     def onSongUsageDelete(self):
         self.SongUsagedeleteform.exec_()