<p dir="ltr">+1 fir mediadatabase split.<br>
I will be good for history, mrl history and network favorites too.</p>
<div class="gmail_quote">Le 24 mars 2015 00:49, "Edward Wang" <<a href="mailto:edward.c.wang@compdigitec.com">edward.c.wang@compdigitec.com</a>> a écrit :<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Geoffrey,<br>
<br>
On Mon, Mar 23, 2015 at 4:46 PM, Geoffrey Métais<br>
<<a href="mailto:geoffrey.metais@gmail.com">geoffrey.metais@gmail.com</a>> wrote:<br>
> Thing is, MediaDatabase.java aggregates all db accesses.<br>
><br>
> I'd rather split different db helpers, so I'm not against this idea but for<br>
> now all is done in this class.<br>
><br>
> Thus, I already use present playlistDelete, playlistExists,<br>
> playlistInsertItem and playlistAdd methods in this very class.<br>
<br>
Indeed. Perhaps we can leave them like this for now but ideally it<br>
would be nicer to do it like this in the future:<br>
<br>
UI classes/fragments <====> PlaylistDatabase <====> MediaDatabase<br>
<====> SQLiteDatabase<br>
<br>
If so we can add a TODO in the code or a low-priority enhancement<br>
ticket on the trac.<br>
<br>
Regards,<br>
Edward<br>
_______________________________________________<br>
Android mailing list<br>
<a href="mailto:Android@videolan.org">Android@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/android" target="_blank">https://mailman.videolan.org/listinfo/android</a><br>
</blockquote></div>