add update note call

Signed-off-by: alperozturk <[email protected]>

Author: alperozturk96

app/src/main/java/it/niedermann/owncloud/notes/edit/BaseNoteFragment.java

@@ -248,7 +248,7 @@ public boolean onOptionsItemSelected(MenuItem item) {
             return true;
         } else if (itemId == R.id.menu_favorite) {
             note.setFavorite(!note.getFavorite());
-            repo.toggleFavoriteAndSync(localAccount, note.getId());
+            repo.toggleFavoriteAndSync(localAccount, note);
             listener.onNoteUpdated(note);
             prepareFavoriteOption(item);
             return true;

app/src/main/java/it/niedermann/owncloud/notes/main/MainActivity.java

@@ -783,7 +783,7 @@ public void onNoteClick(int position, View v) {
 
     @Override
     public void onNoteFavoriteClick(int position, View view) {
-        final var toggleLiveData = mainViewModel.toggleFavoriteAndSync(((Note) adapter.getItem(position)).getId());
+        final var toggleLiveData = mainViewModel.toggleFavoriteAndSync(((Note) adapter.getItem(position)));
         toggleLiveData.observe(this, (next) -> toggleLiveData.removeObservers(this));
     }
 

app/src/main/java/it/niedermann/owncloud/notes/main/MainViewModel.java

@@ -513,15 +513,14 @@ public LiveData<Note> moveNoteToAnotherAccount(Account account, long noteId) {
         });
     }
 
-    public LiveData<Void> toggleFavoriteAndSync(long noteId) {
+    public LiveData<Void> toggleFavoriteAndSync(Note note) {
         return switchMap(getCurrentAccount(), currentAccount -> {
-            if (currentAccount == null) {
-                return new MutableLiveData<>(null);
-            } else {
+            if (currentAccount != null) {
                 Log.v(TAG, "[toggleFavoriteAndSync] - currentAccount: " + currentAccount.getAccountName());
-                repo.toggleFavoriteAndSync(currentAccount, noteId);
-                return new MutableLiveData<>(null);
+                repo.toggleFavoriteAndSync(currentAccount, note);
             }
+
+            return new MutableLiveData<>(null);
         });
     }
 

app/src/main/java/it/niedermann/owncloud/notes/main/items/list/NotesListViewItemTouchHelper.java

@@ -101,7 +101,7 @@ public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int direction)
                     case ItemTouchHelper.RIGHT -> {
                         viewHolder.setIsRecyclable(false);
                         final var adapterNote = (Note) adapter.getItem(viewHolder.getLayoutPosition());
-                        final var toggleLiveData = mainViewModel.toggleFavoriteAndSync(adapterNote.getId());
+                        final var toggleLiveData = mainViewModel.toggleFavoriteAndSync(adapterNote);
                         toggleLiveData.observe(lifecycleOwner, (next) -> toggleLiveData.removeObservers(lifecycleOwner));
                     }
                     default -> {

app/src/main/java/it/niedermann/owncloud/notes/persistence/NotesRepository.java

@@ -12,6 +12,7 @@
 import static androidx.lifecycle.Transformations.map;
 import static java.util.stream.Collectors.toMap;
 import static it.niedermann.owncloud.notes.edit.EditNoteActivity.ACTION_SHORTCUT;
+import static it.niedermann.owncloud.notes.shared.util.ApiVersionUtil.getPreferredApiVersion;
 import static it.niedermann.owncloud.notes.shared.util.NoteUtil.generateNoteExcerpt;
 import static it.niedermann.owncloud.notes.widget.notelist.NoteListWidget.updateNoteListWidgets;
 import static it.niedermann.owncloud.notes.widget.singlenote.SingleNoteWidget.updateSingleNoteWidgets;
@@ -47,6 +48,7 @@
 import com.nextcloud.android.sso.exceptions.NoCurrentAccountSelectedException;
 import com.nextcloud.android.sso.helper.SingleAccountHelper;
 import com.nextcloud.android.sso.model.SingleSignOnAccount;
+import com.owncloud.android.lib.common.utils.Log_OC;
 
 import java.util.ArrayList;
 import java.util.Calendar;
@@ -538,11 +540,26 @@ public Map<Long, Long> getIdMap(long accountId) {
                 .collect(toMap(Note::getRemoteId, Note::getId));
     }
 
+    // FIXME: RACE CONDITION
     @AnyThread
-    public void toggleFavoriteAndSync(Account account, long noteId) {
+    public void toggleFavoriteAndSync(Account account, Note note) {
         executor.submit(() -> {
-            db.getNoteDao().toggleFavorite(noteId);
-            scheduleSync(account, true);
+            try {
+                final var ssoAccount = AccountImporter.getSingleSignOnAccount(context, account.getAccountName());
+                final var notesAPI = apiProvider.getNotesAPI(context, ssoAccount, getPreferredApiVersion(account.getApiVersion()));
+                note.setFavorite(!note.getFavorite());
+                final var result = notesAPI.updateNote(note);
+                final var response = result.execute();
+                if (response.isSuccessful()) {
+                    final var updatedNote = response.body();
+                    if (updatedNote != null) {
+                        db.getNoteDao().updateNote(updatedNote);
+                        scheduleSync(account, true);
+                    }
+                }
+            } catch (Exception e) {
+                Log_OC.e(TAG, "toggleFavoriteAndSync: " + e);
+            }
         });
     }
 

app/src/main/java/it/niedermann/owncloud/notes/persistence/sync/NotesAPI.java

@@ -17,7 +17,9 @@
 import com.nextcloud.android.sso.api.ParsedResponse;
 
 import java.util.Calendar;
+import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 import java.util.stream.Collectors;
 
 import io.reactivex.Observable;
@@ -106,6 +108,21 @@ public Call<Note> createNote(Note note) {
         }
     }
 
+    public Call<Note> updateNote(@NonNull Note note) {
+        final Long remoteId = note.getRemoteId();
+        if (remoteId == null) {
+            return null;
+        }
+
+        if (ApiVersion.API_VERSION_1_0.equals(usedApiVersion)) {
+            return notesAPI_1_0.updateNote(remoteId, note);
+        } else if (ApiVersion.API_VERSION_0_2.equals(usedApiVersion)) {
+            return notesAPI_0_2.updateNote(remoteId, new Note_0_2(note));
+        } else {
+            return null;
+        }
+    }
+
     public Call<Note> editNote(@NonNull Note note) {
         final Long remoteId = note.getRemoteId();
         if (remoteId == null) {

app/src/main/java/it/niedermann/owncloud/notes/persistence/sync/NotesAPI_0_2.java

@@ -10,6 +10,7 @@
 import com.nextcloud.android.sso.api.ParsedResponse;
 
 import java.util.List;
+import java.util.Map;
 
 import io.reactivex.Observable;
 import it.niedermann.owncloud.notes.persistence.entity.Note;
@@ -46,4 +47,7 @@ public interface NotesAPI_0_2 {
 
     @DELETE("notes/{remoteId}")
     Call<EmptyResponse> deleteNote(@Path("remoteId") long noteId);
+
+    @PUT("notes/{id}")
+    Call<Note> updateNote(@Path("id") long id, @Body NotesAPI.Note_0_2 note);
 }

app/src/main/java/it/niedermann/owncloud/notes/persistence/sync/NotesAPI_1_0.java

@@ -10,6 +10,7 @@
 import com.nextcloud.android.sso.api.ParsedResponse;
 
 import java.util.List;
+import java.util.Map;
 
 import io.reactivex.Observable;
 import it.niedermann.owncloud.notes.persistence.entity.Note;
@@ -52,4 +53,7 @@ public interface NotesAPI_1_0 {
 
     @PUT("settings")
     Call<NotesSettings> putSettings(@Body NotesSettings settings);
+
+    @PUT("notes/{id}")
+    Call<Note> updateNote(@Path("id") long id, @Body Note note);
 }