diff --git a/app/src/main/java/org/xbmc/kore/Settings.java b/app/src/main/java/org/xbmc/kore/Settings.java index 387ced5..434b2a1 100644 --- a/app/src/main/java/org/xbmc/kore/Settings.java +++ b/app/src/main/java/org/xbmc/kore/Settings.java @@ -45,7 +45,10 @@ public class Settings { SORT_BY_DATE_ADDED = 1, SORT_BY_RATING = 2, SORT_BY_YEAR = 3, - SORT_BY_LENGTH = 4; + SORT_BY_LENGTH = 4, + SORT_BY_ALBUM = 5, + SORT_BY_ARTIST = 6, + SORT_BY_ARTIST_YEAR = 7 ; /** * Preferences keys. @@ -88,6 +91,10 @@ public class Settings { public static final String KEY_PREF_MOVIES_SORT_ORDER = "movies_sort_order"; public static final int DEFAULT_PREF_MOVIES_SORT_ORDER = SORT_BY_NAME; + // Sort order on albums + public static final String KEY_PREF_ALBUMS_SORT_ORDER = "albums_sort_order"; + public static final int DEFAULT_PREF_ALBUMS_SORT_ORDER = SORT_BY_ALBUM; + // Ignore articles on movie sorting public static final String KEY_PREF_MOVIES_IGNORE_PREFIXES = "movies_ignore_prefixes"; public static final boolean DEFAULT_PREF_MOVIES_IGNORE_PREFIXES = false; diff --git a/app/src/main/java/org/xbmc/kore/ui/AlbumListFragment.java b/app/src/main/java/org/xbmc/kore/ui/AlbumListFragment.java index 0d56d90..6d0122f 100644 --- a/app/src/main/java/org/xbmc/kore/ui/AlbumListFragment.java +++ b/app/src/main/java/org/xbmc/kore/ui/AlbumListFragment.java @@ -18,14 +18,18 @@ package org.xbmc.kore.ui; import android.annotation.TargetApi; import android.app.Activity; import android.content.Context; +import android.content.SharedPreferences; import android.content.res.Resources; import android.database.Cursor; import android.net.Uri; import android.os.Bundle; +import android.preference.PreferenceManager; import android.provider.BaseColumns; import android.support.v4.content.CursorLoader; import android.text.TextUtils; import android.view.LayoutInflater; +import android.view.Menu; +import android.view.MenuInflater; import android.view.MenuItem; import android.view.View; import android.view.ViewGroup; @@ -35,6 +39,7 @@ import android.widget.PopupMenu; import android.widget.TextView; import org.xbmc.kore.R; +import org.xbmc.kore.Settings; import org.xbmc.kore.host.HostInfo; import org.xbmc.kore.host.HostManager; import org.xbmc.kore.jsonrpc.type.PlaylistType; @@ -92,6 +97,64 @@ public class AlbumListFragment extends AbstractCursorListFragment { return fragment; } + @Override + public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { + inflater.inflate(R.menu.album_list, menu); + + MenuItem sortByAlbum = menu.findItem(R.id.action_sort_by_album), + sortByArtist = menu.findItem(R.id.action_sort_by_artist), + sortByArtistYear = menu.findItem(R.id.action_sort_by_artist_year); + + + SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(getActivity()); + + int sortOrder = preferences.getInt(Settings.KEY_PREF_ALBUMS_SORT_ORDER, Settings.DEFAULT_PREF_ALBUMS_SORT_ORDER); + switch (sortOrder) { + case Settings.SORT_BY_ALBUM: + sortByAlbum.setChecked(true); + break; + case Settings.SORT_BY_ARTIST: + sortByArtist.setChecked(true); + break; + case Settings.SORT_BY_ARTIST_YEAR: + sortByArtistYear.setChecked(true); + break; + } + super.onCreateOptionsMenu(menu, inflater); + } + + @Override + public boolean onOptionsItemSelected(MenuItem item) { + SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(getActivity()); + switch (item.getItemId()) { + case R.id.action_sort_by_album: + item.setChecked(!item.isChecked()); + preferences.edit() + .putInt(Settings.KEY_PREF_ALBUMS_SORT_ORDER, Settings.SORT_BY_ALBUM) + .apply(); + refreshList(); + break; + case R.id.action_sort_by_artist: + item.setChecked(!item.isChecked()); + preferences.edit() + .putInt(Settings.KEY_PREF_ALBUMS_SORT_ORDER, Settings.SORT_BY_ARTIST) + .apply(); + refreshList(); + break; + case R.id.action_sort_by_artist_year: + item.setChecked(!item.isChecked()); + preferences.edit() + .putInt(Settings.KEY_PREF_ALBUMS_SORT_ORDER, Settings.SORT_BY_ARTIST_YEAR) + .apply(); + refreshList(); + break; + default: + break; + } + + return super.onOptionsItemSelected(item); + } + @Override protected void onListItemClicked(View view) { // Get the movie id from the tag @@ -127,8 +190,20 @@ public class AlbumListFragment extends AbstractCursorListFragment { selectionArgs = new String[] {"%" + searchFilter + "%"}; } + SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(getActivity()); + + String sortOrderStr; + int sortOrder = preferences.getInt(Settings.KEY_PREF_ALBUMS_SORT_ORDER, Settings.DEFAULT_PREF_ALBUMS_SORT_ORDER); + if (sortOrder == Settings.SORT_BY_ARTIST) { + sortOrderStr = AlbumListQuery.SORT_BY_ARTIST; + } else if (sortOrder == Settings.SORT_BY_ARTIST_YEAR) { + sortOrderStr = AlbumListQuery.SORT_BY_ARTIST_YEAR; + } else { + sortOrderStr = AlbumListQuery.SORT_BY_ALBUM; + } + return new CursorLoader(getActivity(), uri, - AlbumListQuery.PROJECTION, selection, selectionArgs, AlbumListQuery.SORT); + AlbumListQuery.PROJECTION, selection, selectionArgs, sortOrderStr); } @Override @@ -175,7 +250,10 @@ public class AlbumListFragment extends AbstractCursorListFragment { MediaContract.Albums.RATING, }; - String SORT = MediaDatabase.sortCommonTokens(MediaContract.Albums.TITLE) + " ASC"; + String SORT_BY_ALBUM = MediaDatabase.sortCommonTokens(MediaContract.Albums.TITLE) + " ASC"; + String SORT_BY_ARTIST = MediaDatabase.sortCommonTokens(MediaContract.Albums.DISPLAYARTIST) + " ASC"; + String SORT_BY_ARTIST_YEAR = MediaDatabase.sortCommonTokens(MediaContract.Albums.DISPLAYARTIST) + + " ASC, " + MediaContract.Albums.YEAR + " ASC"; final int ID = 0; final int ALBUMID = 1; diff --git a/app/src/main/res/menu/album_list.xml b/app/src/main/res/menu/album_list.xml new file mode 100644 index 0000000..f9f927c --- /dev/null +++ b/app/src/main/res/menu/album_list.xml @@ -0,0 +1,35 @@ + + + + + + + + + + + + + diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index b3a4476..a349851 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -388,5 +388,8 @@ Permission denied. Won\'t be able to download files. No songs to download + By album + By artist + By artist and year