tenkuma/jamfish
tenkuma/jamfish
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed kabouzeid/phonograph-issue-tracker#50, playlist name changes were not directly updated in the list.

Browse source
This commit is contained in:
Karim Abou Zeid 2015-09-22 16:57:46 +02:00
commit 4b49bb82a3
1 changed files with 8 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

13
app/src/main/java/com/kabouzeid/gramophone/util/PlaylistsUtil.java
View file

@ -19,6 +19,8 @@ import com.kabouzeid.gramophone.model.Song;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
import static android.provider.MediaStore.Audio.Playlists.EXTERNAL_CONTENT_URI;
/** /**
* @author Karim Abou Zeid (kabouzeid) * @author Karim Abou Zeid (kabouzeid)
*/ */
@ -28,13 +30,13 @@ public class PlaylistsUtil {
int id = -1; int id = -1;
if (name != null && name.length() > 0) { if (name != null && name.length() > 0) {
try { try {
Cursor cursor = context.getContentResolver().query(MediaStore.Audio.Playlists.EXTERNAL_CONTENT_URI, Cursor cursor = context.getContentResolver().query(EXTERNAL_CONTENT_URI,
new String[]{MediaStore.Audio.Playlists._ID}, MediaStore.Audio.PlaylistsColumns.NAME + "=?", new String[]{name}, null); new String[]{MediaStore.Audio.Playlists._ID}, MediaStore.Audio.PlaylistsColumns.NAME + "=?", new String[]{name}, null);
if (cursor == null || cursor.getCount() < 1) { if (cursor == null || cursor.getCount() < 1) {
final ContentValues values = new ContentValues(1); final ContentValues values = new ContentValues(1);
values.put(MediaStore.Audio.PlaylistsColumns.NAME, name); values.put(MediaStore.Audio.PlaylistsColumns.NAME, name);
final Uri uri = context.getContentResolver().insert( final Uri uri = context.getContentResolver().insert(
MediaStore.Audio.Playlists.EXTERNAL_CONTENT_URI, EXTERNAL_CONTENT_URI,
values); values);
if (uri != null) { if (uri != null) {
// necessary because somehow the MediaStoreObserver is not notified when adding a playlist // necessary because somehow the MediaStoreObserver is not notified when adding a playlist
@ -72,7 +74,7 @@ public class PlaylistsUtil {
} }
selection.append(")"); selection.append(")");
try { try {
context.getContentResolver().delete(MediaStore.Audio.Playlists.EXTERNAL_CONTENT_URI, selection.toString(), null); context.getContentResolver().delete(EXTERNAL_CONTENT_URI, selection.toString(), null);
} catch (SecurityException ignored) { } catch (SecurityException ignored) {
} }
} }
@ -192,10 +194,11 @@ public class PlaylistsUtil {
ContentValues contentValues = new ContentValues(); ContentValues contentValues = new ContentValues();
contentValues.put(MediaStore.Audio.PlaylistsColumns.NAME, newName); contentValues.put(MediaStore.Audio.PlaylistsColumns.NAME, newName);
try { try {
context.getContentResolver().update(MediaStore.Audio.Playlists.EXTERNAL_CONTENT_URI, context.getContentResolver().update(EXTERNAL_CONTENT_URI,
contentValues, contentValues,
MediaStore.Audio.Playlists._ID + "=?", MediaStore.Audio.Playlists._ID + "=?",
new String[]{String.valueOf(id)}); new String[]{String.valueOf(id)});
context.getContentResolver().notifyChange(Uri.parse("content://media"), null);
} catch (SecurityException ignored) { } catch (SecurityException ignored) {
} }
} }
@ -203,7 +206,7 @@ public class PlaylistsUtil {
public static String getNameForPlaylist(@NonNull final Context context, final long id) { public static String getNameForPlaylist(@NonNull final Context context, final long id) {
try { try {
Cursor cursor = context.getContentResolver().query( Cursor cursor = context.getContentResolver().query(
MediaStore.Audio.Playlists.EXTERNAL_CONTENT_URI, EXTERNAL_CONTENT_URI,
new String[]{MediaStore.Audio.PlaylistsColumns.NAME}, new String[]{MediaStore.Audio.PlaylistsColumns.NAME},
BaseColumns._ID + "=?", BaseColumns._ID + "=?",
new String[]{String.valueOf(id)}, new String[]{String.valueOf(id)},