← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~crichter/openlp/media_rewrite into lp:openlp

 

Review: Needs Fixing
The plugin to core code is wrong and I think it is because we have the placement wrong.
A lot of the plumbing is because the players are controlled by the plugin and core needs access to them.
For presentations this as fine as the players were external but for this it's more complicated.
I think Core should provide player services (extensible and flexible so more can be added).  This should stop the needed to pass controllers around.  
The media plugin asks what players can I use and what do I want to us or just does not care and uses what core has decided.
I am not sure what will happen if this code has media deactivated.


-- 
https://code.launchpad.net/~crichter/openlp/media_rewrite/+merge/64630
Your team OpenLP Core is subscribed to branch lp:openlp.


References