Skip to content
This repository was archived by the owner on Sep 8, 2019. It is now read-only.

Commit 55d693f

Browse files
committed
- deleting an image in the ItemActivity no longer takes the user back to AlbumActivity (#132)
1 parent 9ca847e commit 55d693f

File tree

3 files changed

+24
-4
lines changed

3 files changed

+24
-4
lines changed

app/src/main/java/us/koller/cameraroll/data/provider/Provider.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ public static boolean isAlbumPinned(String albumPath, ArrayList<String> pinnedPa
153153
return true;
154154
} else {
155155
for (int i = 0; i < pinnedPaths.size(); i++) {
156-
if (albumPath.contains(pinnedPaths.get(i))) {
156+
if (albumPath.equals(pinnedPaths.get(i))) {
157157
return true;
158158
}
159159
}

app/src/main/java/us/koller/cameraroll/ui/AlbumActivity.java

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1161,6 +1161,7 @@ public void onAlbumLoaded(Album album) {
11611161
}
11621162

11631163
private void removeAlbumItem(String path) {
1164+
Log.d("AlbumActivity", "removeAlbumItem() called with: path = [" + path + "]");
11641165
int index = -1;
11651166
for (int i = 0; i < album.getAlbumItems().size(); i++) {
11661167
AlbumItem albumItem = album.getAlbumItems().get(i);
@@ -1169,9 +1170,14 @@ private void removeAlbumItem(String path) {
11691170
break;
11701171
}
11711172
}
1173+
Log.d("AlbumActivity", "removeAlbumItem: " + index);
11721174
if (index > -1) {
11731175
album.getAlbumItems().remove(index);
1174-
recyclerViewAdapter.notifyDataSetChanged();
1176+
}
1177+
recyclerViewAdapter.notifyDataSetChanged();
1178+
1179+
if (album.getAlbumItems().size() == 0) {
1180+
finish();
11751181
}
11761182
}
11771183
}

app/src/main/java/us/koller/cameraroll/ui/ItemActivity.java

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -572,8 +572,22 @@ public void onReceive(Context context, Intent intent) {
572572
.putExtra(ALBUM_ITEM_PATH, path);
573573
//notify AlbumActivity
574574
LocalBroadcastManager.getInstance(ItemActivity.this).sendBroadcast(i);
575-
ItemActivity.this.setResult(RESULT_OK);
576-
finish();
575+
/*ItemActivity.this.setResult(RESULT_OK);
576+
finish();*/
577+
578+
album.getAlbumItems().remove(albumItem);
579+
viewPager.getAdapter().notifyDataSetChanged();
580+
581+
if (album.getAlbumItems().size() == 0) {
582+
ItemActivity.this.setResult(RESULT_OK);
583+
finish();
584+
return;
585+
}
586+
587+
albumItem = album.getAlbumItems().get(viewPager.getCurrentItem());
588+
ItemAdapter adapter = (ItemAdapter) viewPager.getAdapter();
589+
ViewHolder viewHolder = adapter.findViewHolderByTag(albumItem.getPath());
590+
onShowViewHolder(viewHolder);
577591
break;
578592
case FileOperation.FAILED:
579593
//onBackPressed();

0 commit comments

Comments
 (0)