← Back to team overview

openlp-core team mailing list archive

  1. openlp-core team
  2. Mailing list archive
  3. Message #17957

Re: [Merge] lp:~phill-ridout/openlp/1065420 into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Needs Fixing

You need to have the is_valid flag.

Please read the servicemanager code to see where it is used.  It is used in 4 places and this code will be a waste of time without setting it.
-- 
https://code.launchpad.net/~phill-ridout/openlp/1065420/+merge/133348
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next