← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~phill-ridout/openlp/yet-more-refactors into lp:openlp

 


Diff comments:

> === modified file 'openlp/plugins/bibles/bibleplugin.py'
> --- openlp/plugins/bibles/bibleplugin.py	2016-08-08 18:11:32 +0000
> +++ openlp/plugins/bibles/bibleplugin.py	2016-08-21 07:47:26 +0000
> @@ -140,10 +140,10 @@
>  
>      def uses_theme(self, theme):
>          """
> -        Called to find out if the bible plugin is currently using a theme. Returns ``1`` if the theme is being used,
> -        otherwise returns ``0``.
> +        Called to find out if the bible plugin is currently using a theme.
>  
>          :param theme: The theme
> +        :return: 1 if the theme is being used, otherwise returns 0

Good question, I wondered that myself. You'd have to ask trb143 about that, possibly related to a change he also made in thememanager in http://bazaar.launchpad.net/~openlp-core/openlp/trunk/revision/2566

>          """
>          if str(self.settings_tab.bible_theme) == theme:
>              return 1


-- 
https://code.launchpad.net/~phill-ridout/openlp/yet-more-refactors/+merge/303490
Your team OpenLP Core is subscribed to branch lp:openlp.


References