← Back to team overview

openlp-core team mailing list archive

[Merge] lp:~googol/openlp/ticket-330 into lp:openlp

 

Andreas Preikschat has proposed merging lp:~googol/openlp/ticket-330 into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)

For more details, see:
https://code.launchpad.net/~googol/openlp/ticket-330/+merge/83338

- applied patch from http://support.openlp.org/issues/330
- not tested!
-- 
https://code.launchpad.net/~googol/openlp/ticket-330/+merge/83338
Your team OpenLP Core is requested to review the proposed merge of lp:~googol/openlp/ticket-330 into lp:openlp.
=== modified file 'openlp/plugins/songs/lib/ewimport.py'
--- openlp/plugins/songs/lib/ewimport.py	2011-09-06 05:47:33 +0000
+++ openlp/plugins/songs/lib/ewimport.py	2011-11-24 19:31:24 +0000
@@ -242,7 +242,7 @@
                 if self.stopImportFlag:
                     break
                 raw_record = db_file.read(record_size)
-                self.fields = self.record_struct.unpack(raw_record)
+                self.fields = self.recordStruct.unpack(raw_record)
                 self.setDefaults()
                 self.title = self.getField(fi_title)
                 # Get remaining fields.
@@ -321,11 +321,11 @@
         db_file.close()
         self.memoFile.close()
 
-    def find_field(self, field_name):
+    def findField(self, field_name):
         return [i for i, x in enumerate(self.fieldDescs)
             if x.name == field_name][0]
 
-    def set_record_struct(self, field_descs):
+    def setRecordStruct(self, field_descs):
         # Begin with empty field struct list
         fsl = ['>']
         for field_desc in field_descs:
@@ -355,7 +355,7 @@
         self.recordStruct = struct.Struct(''.join(fsl))
         self.fieldDescs = field_descs
 
-    def get_field(self, field_desc_index):
+    def getField(self, field_desc_index):
         field = self.fields[field_desc_index]
         field_desc = self.fieldDescs[field_desc_index]
         # Return None in case of 'blank' entries


Follow ups