Skip to content
Snippets Groups Projects
Commit b90695fe authored by Geoffrey Métais's avatar Geoffrey Métais
Browse files

ViewModel saves sorting preference instead of View

parent 81d5510c
No related branches found
No related tags found
No related merge requests found
......@@ -295,13 +295,7 @@ public abstract class MediaBrowserFragment<T extends SortableModel> extends Frag
}
protected void sortBy(int sort) {
final T model = getViewModel();
model.sort(sort);
final String key = model.getKey();
Settings.INSTANCE.getInstance(requireContext()).edit()
.putInt(key, sort)
.putBoolean(key+"_desc", model.getDesc())
.apply();
getViewModel().sort(sort);
}
public Menu getMenu() {
......
......@@ -3,6 +3,7 @@ package org.videolan.vlc.viewmodels
import android.content.Context
import org.videolan.medialibrary.Medialibrary
import org.videolan.vlc.util.RefreshModel
import org.videolan.vlc.util.Settings
import org.videolan.vlc.util.canSortBy
abstract class SortableModel(protected val context: Context): ScopedModel(), RefreshModel {
......@@ -35,6 +36,10 @@ abstract class SortableModel(protected val context: Context): ScopedModel(), Ref
}
this.sort = sort
refresh()
Settings.getInstance(context).edit()
.putInt(sortKey, sort)
.putBoolean("${sortKey}_desc", desc)
.apply()
}
}
......
......@@ -9,6 +9,7 @@ import androidx.paging.PositionalDataSource
import kotlinx.coroutines.launch
import org.videolan.medialibrary.Medialibrary
import org.videolan.medialibrary.media.MediaLibraryItem
import org.videolan.vlc.util.Settings
import org.videolan.vlc.viewmodels.SortableModel
abstract class MLPagedModel<T : MediaLibraryItem>(context: Context) : SortableModel(context), Medialibrary.OnMedialibraryReadyListener, Medialibrary.OnDeviceChangeListener {
......@@ -63,6 +64,10 @@ abstract class MLPagedModel<T : MediaLibraryItem>(context: Context) : SortableMo
desc = false
} else desc = !desc
refresh()
Settings.getInstance(context).edit()
.putInt(sortKey, sort)
.putBoolean("${sortKey}_desc", desc)
.apply()
}
fun isFiltering() = filter != null
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment