Re-implemented getting tasks with the new seperate endpoint #39

Open
konrad wants to merge 34 commits from feature/task-list into master
5 changed files with 7 additions and 7 deletions
Showing only changes of commit 972b194f56 - Show all commits

View File

@ -28,7 +28,7 @@ class Client {
'Content-Type': 'application/json'
};
Future<dynamic> get(String url, [Map<String, String> queryParameters]) {
Future<dynamic> get(String url, [Map<String, List<String>> queryParameters]) {
// TODO: This could be moved to a seperate function
var uri = Uri.parse('${this.base}$url');
// Because these are all final values, we can't just add the queryParameters and must instead build a new Uri Object every time this method is called.

View File

@ -28,7 +28,7 @@ class TaskAPIService extends APIService implements TaskService {
}
@override
Future<List<Task>> getAll(int listId, [Map<String, String> queryParameters]) {
Future<List<Task>> getAll(int listId, [Map<String, List<String>> queryParameters]) {
return client
.get('/lists/$listId/tasks', queryParameters)
.then((list) => convertList(list, (result) => Task.fromJson(result)));

View File

@ -106,9 +106,9 @@ class _ListPageState extends State<ListPage> {
Future<void> _loadTasksForPage(int page) {
return VikunjaGlobal.of(context).taskService.getAll(_list.id, {
"sort_by": "done",
"order_by": "asc",
"page": page.toString()
"sort_by": ["done", "id"],
"order_by": ["asc", "desc"],
"page": [page.toString()]
}).then((tasks) {
setState(() {
konrad marked this conversation as resolved Outdated

resolve provider

resolve provider

Done.

Done.
_loading = false;

View File

@ -137,7 +137,7 @@ class MockedTaskService implements TaskService {
}
@override
Future<List<Task>> getAll(int listId, [Map<String, String> queryParameters]) {
Future<List<Task>> getAll(int listId, [Map<String, List<String>> queryParameters]) {
return Future.value(_tasks.values.toList());
}

View File

@ -26,7 +26,7 @@ abstract class TaskService {
Future<Task> update(Task task);
Future delete(int taskId);
Future<Task> add(int listId, Task task);
Future<List<Task>> getAll(int listId, [Map<String, String> queryParameters]);
Future<List<Task>> getAll(int listId, [Map<String, List<String>> queryParameters]);
// TODO: Avoid having to add this to each abstract class
int get maxPages;
}