reflect onQuickPermissionChanges

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

Author: alperozturk96

app/src/main/java/it/niedermann/owncloud/notes/share/NoteShareActivity.java

@@ -432,11 +432,12 @@ public void refreshSharesFromDB() {
                     return;
                 }
 
-                adapter.getShares().clear();
+                // clear adapter
+                adapter.removeAll();
+                shares.clear();
 
                 // to show share with users/groups info
                 if (note != null) {
-
                     // get shares from local DB
                     final var shareEntities = repository.getShareEntitiesForSpecificNote(note);
                     shareEntities.forEach(entity -> {
@@ -649,9 +650,10 @@ private boolean getExpDateShown() {
     public void onQuickPermissionChanged(OCShare share, int permission) {
         executorService.submit(() -> {
             final var result = repository.updateSharePermission(share.getId(), permission);
-
             runOnUiThread(() -> {
-                if (result instanceof ApiResult.Error error) {
+                if (ApiResultKt.isSuccess(result)) {
+                    NoteShareActivity.this.recreate();
+                } else if (result instanceof ApiResult.Error error) {
                     DisplayUtils.showSnackMessage(NoteShareActivity.this, error.getMessage());
                 }
             });

app/src/main/java/it/niedermann/owncloud/notes/share/adapter/ShareeListAdapter.java

@@ -47,6 +47,7 @@ public ShareeListAdapter(Activity activity,
         this.account = account;
 
         sortShares();
+        setHasStableIds(true);
     }
 
     @Override
@@ -128,6 +129,10 @@ public void onBindViewHolder(@NonNull RecyclerView.ViewHolder holder, int positi
 
     @Override
     public long getItemId(int position) {
+        if (position < 0 || position >= shares.size()) {
+            return 0;
+        }
+
         return shares.get(position).getId();
     }
 
@@ -156,6 +161,12 @@ public void remove(OCShare share) {
         notifyDataSetChanged();
     }
 
+    @SuppressLint("NotifyDataSetChanged")
+    public void removeAll() {
+        shares.clear();
+        notifyDataSetChanged();
+    }
+
     /**
      * sort all by creation time, then email/link shares on top
      */