Review: Needs Fixing 253 + def __init__(self, manager, **kwargs): 254 + """ 255 + Initialise the PowerSong importer. 256 + """ 257 + SongImport.__init__(self, manager, **kwargs)
If you're not actually doing something in the constructor, don't bother adding it in. Otherwise it looks good. I haven't actually tested it out, but I'll do that later. -- https://code.launchpad.net/~sfindlay/openlp/songs-import-powersong/+merge/104102 Your team OpenLP Core is subscribed to branch lp:openlp. _______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : [email protected] Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp
