don't remove gap when removing status

re: #898
sk22 · Oct 26, 2023 · e8c9253 · e8c9253
1 parent c8f633a
commit e8c9253
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/mastodon/src/main/java/org/joinmastodon/android/api/CacheController.java b/mastodon/src/main/java/org/joinmastodon/android/api/CacheController.java
@@ -273,6 +273,7 @@ public void putRecentSearch(SearchResult result){
 
 	public void deleteStatus(String id){
 		runOnDbThread((db)->{
+			// TODO: set previous status's hasGapAfter flag if the deleted status hasGapAfter
 			db.delete("home_timeline", "`id`=?", new String[]{id});
 		});
 	}

diff --git a/mastodon/src/main/java/org/joinmastodon/android/fragments/StatusListFragment.java b/mastodon/src/main/java/org/joinmastodon/android/fragments/StatusListFragment.java
@@ -25,6 +25,7 @@
 import org.joinmastodon.android.ui.displayitems.EmojiReactionsStatusDisplayItem;
 import org.joinmastodon.android.ui.displayitems.ExtendedFooterStatusDisplayItem;
 import org.joinmastodon.android.ui.displayitems.FooterStatusDisplayItem;
+import org.joinmastodon.android.ui.displayitems.GapStatusDisplayItem;
 import org.joinmastodon.android.ui.displayitems.StatusDisplayItem;
 import org.joinmastodon.android.ui.displayitems.TextStatusDisplayItem;
 import org.parceler.Parcels;
@@ -216,7 +217,7 @@ protected void removeStatus(Status status){
 			StatusDisplayItem item=displayItems.get(i);
 			// we found a status that the to-be-removed status replies to!
 			// storing indices to maybe update its display items
-			if(item.parentID.equals(status.inReplyToId)){
+			if(item.parentID.equals(status.inReplyToId) && !(item instanceof GapStatusDisplayItem)){
 				if(ancestorFirstIndex==-1) ancestorFirstIndex=i;
 				ancestorLastIndex=i;
 			}