← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~orangeshirt/openlp/bibles into lp:openlp

 

Review: Needs Fixing
5	return [
6	    {
7	    u'name': book.name,
8	  - u'chapters': self.db_cache[bible].get_chapter_count(
9	  - book)
10	  + u'book_reference_id': book.book_reference_id,
11	  + u'chapters': self.db_cache[bible].get_chapter_count(book)
12	    }
13	    for book in self.db_cache[bible].get_books()
14	]

This is already indented incorrectly. Please fix it properly, it needs to look like this:

return [
    {
        u'name': book.name,
        u'book_reference_id': book.book_reference_id,
        u'chapters': self.db_cache[bible].get_chapter_count(book)
    }
    for book in self.db_cache[bible].get_books()
]

-- 
https://code.launchpad.net/~orangeshirt/openlp/bibles/+merge/62798
Your team OpenLP Core is subscribed to branch lp:openlp.


References