[Android] [PATCH 2/2] Playlists in database

[Geoffrey Métais]

+1 fir mediadatabase split.
I will be good for history, mrl history and network favorites too.
Le 24 mars 2015 00:49, "Edward Wang" <edward.c.wang at compdigitec.com> a
écrit :

> Hi Geoffrey,
>
> On Mon, Mar 23, 2015 at 4:46 PM, Geoffrey Métais
> <geoffrey.metais at gmail.com> wrote:
> > Thing is, MediaDatabase.java aggregates all db accesses.
> >
> > I'd rather split different db helpers, so I'm not against this idea but
> for
> > now all is done in this class.
> >
> > Thus, I already use present playlistDelete, playlistExists,
> > playlistInsertItem and playlistAdd methods in this very class.
>
> Indeed. Perhaps we can leave them like this for now but ideally it
> would be nicer to do it like this in the future:
>
> UI classes/fragments <====> PlaylistDatabase <====> MediaDatabase
> <====> SQLiteDatabase
>
> If so we can add a TODO in the code or a low-priority enhancement
> ticket on the trac.
>
> Regards,
> Edward
> _______________________________________________
> Android mailing list
> Android at videolan.org
> https://mailman.videolan.org/listinfo/android
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/android/attachments/20150324/fee0d7c9/attachment.html>