norohind/vanilla
1
0
Fork 0
Code Issues 1 Pull Requests Projects Releases Wiki Activity

Switch to old manual playlist code as this is still buggy on 5.1!

Browse Source 
Android sometimes creates messed up play_orders in audio_playlists_map:

 sqlite> select audio_id,play_order from audio_playlists_map where playlist_id=829 limit 4;
 audio_id|play_order
 474|13
 479|14
 480|14 <---
 475|7

Such a database (created by android) confuses moveItem() - our own implementation is aware
of this bug and re-generates the play_order
This commit is contained in:
Adrian Ulrich 2015-08-10 20:41:48 +02:00
parent de955a96a7
commit 1be79d7ca9
1 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/ch/blinkenlights/android/vanilla/PlaylistAdapter.java
View File

@ -25,6 +25,7 @@ package ch.blinkenlights.android.vanilla;
import android.content.ContentResolver;
import android.content.ContentUris;
import android.content.ContentValues;
import android.content.Context;
import android.database.Cursor;
import android.net.Uri;
@ -172,15 +173,6 @@ public class PlaylistAdapter extends CursorAdapter implements Handler.Callback {
* @param from original position of item
* @param to destination of item
**/
public void moveItem(int from, int to) {
if (from == to)
return;
android.provider.MediaStore.Audio.Playlists.Members.moveItem(mContext.getContentResolver(), mPlaylistId , from, to);
mUiHandler.sendEmptyMessage(MSG_RUN_QUERY);
}
/* fixme: does the move-after-delete bug still exist in 4.x?
public void moveItem(int from, int to)
{
if (from == to)
@ -208,11 +200,11 @@ public class PlaylistAdapter extends CursorAdapter implements Handler.Callback {
order = 0;
} else {
cursor.moveToPosition(start - 1);
order = cursor.getLong(4) + 1;
order = cursor.getLong(5) + 1;
}
cursor.moveToPosition(end);
long endOrder = cursor.getLong(4);
long endOrder = cursor.getLong(5);
// clear the rows we are replacing
String[] args = new String[] { Long.toString(order), Long.toString(endOrder) };
@ -233,7 +225,7 @@ public class PlaylistAdapter extends CursorAdapter implements Handler.Callback {
changeCursor(runQuery(resolver));
}
*/
public void removeItem(int position)
{
ContentResolver resolver = mContext.getContentResolver();