use remote id

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

Author: alperozturk96

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

@@ -114,6 +114,10 @@ public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceStat
                         }
                         isNew = false;
                         note = originalNote = repo.getNoteById(id);
+                        if (note == null) {
+                            Log_OC.d(TAG, "remoteNoteId will be used to get note");
+                            note = repo.getNoteByRemoteId(id);
+                        }
                         requireActivity().runOnUiThread(() -> onNoteLoaded(note));
                         requireActivity().invalidateOptionsMenu();
                     } else {
@@ -278,21 +282,35 @@ public boolean onOptionsItemSelected(MenuItem item) {
             shareNote();
             return false;
         } else if (itemId == MENU_ID_PIN) {
-            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
-                final var context = requireContext();
-                if (ShortcutManagerCompat.isRequestPinShortcutSupported(context)) {
-                    final var pinShortcutInfo = new ShortcutInfoCompat.Builder(context, String.valueOf(note.getId()))
-                            .setShortLabel(note.getTitle())
-                            .setIcon(IconCompat.createWithResource(context.getApplicationContext(), TRUE.equals(note.getFavorite()) ? R.drawable.ic_star_yellow_24dp : R.drawable.ic_star_border_grey_ccc_24dp))
-                            .setIntent(new Intent(getActivity(), EditNoteActivity.class).putExtra(EditNoteActivity.PARAM_NOTE_ID, note.getId()).setAction(ACTION_SHORTCUT))
-                            .build();
-
-                    ShortcutManagerCompat.requestPinShortcut(context, pinShortcutInfo, PendingIntent.getBroadcast(context, 0, ShortcutManagerCompat.createShortcutResultIntent(context, pinShortcutInfo), pendingIntentFlagCompat(0)).getIntentSender());
-                } else {
-                    Log.i(TAG, "RequestPinShortcut is not supported");
-                }
+            if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O) {
+                return true;
             }
 
+            if (!ShortcutManagerCompat.isRequestPinShortcutSupported(requireContext())) {
+                Log.i(TAG, "RequestPinShortcut is not supported");
+                return true;
+            }
+
+            final var iconId = note.getFavorite() ? R.drawable.ic_star_yellow_24dp : R.drawable.ic_star_border_grey_ccc_24dp;
+            final var icon = IconCompat.createWithResource(requireContext().getApplicationContext(), iconId);
+            final var intent = new Intent(getActivity(), EditNoteActivity.class)
+                    .putExtra(EditNoteActivity.PARAM_NOTE_ID, note.getRemoteId())
+                    .setAction(ACTION_SHORTCUT);
+            final var noteId = String.valueOf(note.getRemoteId());
+
+            final var pinShortcutInfo = new ShortcutInfoCompat.Builder(requireContext(), noteId)
+                    .setShortLabel(note.getTitle())
+                    .setIcon(icon)
+                    .setIntent(intent)
+                    .build();
+
+            final var broadcastIntent = ShortcutManagerCompat.createShortcutResultIntent(requireContext(), pinShortcutInfo);
+            final var intentFlag = pendingIntentFlagCompat(0);
+            final var intentSender = PendingIntent
+                    .getBroadcast(requireContext(), 0, broadcastIntent, intentFlag)
+                    .getIntentSender();
+            ShortcutManagerCompat.requestPinShortcut(requireContext(), pinShortcutInfo, intentSender);
+
             return true;
         }
         return super.onOptionsItemSelected(item);

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

@@ -355,6 +355,10 @@ public Note getNoteById(long id) {
         return db.getNoteDao().getNoteById(id);
     }
 
+    public Note getNoteByRemoteId(long id) {
+        return db.getNoteDao().getNoteByRemoteId(id);
+    }
+
     public LiveData<Integer> count$(long accountId) {
         return db.getNoteDao().count$(accountId);
     }

app/src/main/java/it/niedermann/owncloud/notes/persistence/dao/NoteDao.java

@@ -53,6 +53,9 @@ public interface NoteDao {
     @Query(getNoteById)
     Note getNoteById(long id);
 
+    @Query("SELECT * FROM NOTE WHERE remoteId = :id")
+    Note getNoteByRemoteId(long id);
+
     @Query("SELECT remoteId FROM NOTE WHERE id = :id")
     Long getRemoteId(long id);