Cleanup/Refactor:Remove followSetNotes property from User, and refactor accordingly. Make getFollowSetNotes() return the notes.

This commit is contained in:
KotlinGeekDev
2025-08-10 02:18:03 +01:00
parent 9e7d3146fa
commit 653494e922
4 changed files with 6 additions and 14 deletions
@@ -844,12 +844,11 @@ class Account(
suspend fun getFollowSetNotes() = suspend fun getFollowSetNotes() =
withContext(Dispatchers.Default) { withContext(Dispatchers.Default) {
val followSetNotes = LocalCache.getFollowSetNotesFor(userProfile()) val followSetNotes = LocalCache.getFollowSetNotesFor(userProfile())
userProfile().updateFollowSetNotes(followSetNotes) Log.d(this@Account.javaClass.simpleName, "Number of follow sets: ${followSetNotes.size}")
// userProfile().followSets = followSetNotes return@withContext followSetNotes
println("Number of follow sets: ${followSetNotes.size}")
} }
suspend fun mapNoteToFollowSet(note: Note): FollowSet = fun mapNoteToFollowSet(note: Note): FollowSet =
FollowSet FollowSet
.mapEventToSet( .mapEventToSet(
event = note.event as PeopleListEvent, event = note.event as PeopleListEvent,
@@ -56,8 +56,6 @@ class User(
var latestMetadata: MetadataEvent? = null var latestMetadata: MetadataEvent? = null
var latestMetadataRelay: NormalizedRelayUrl? = null var latestMetadataRelay: NormalizedRelayUrl? = null
var latestContactList: ContactListEvent? = null var latestContactList: ContactListEvent? = null
var followSetNotes: Set<AddressableNote> = setOf()
private set
var reports = mapOf<User, Set<Note>>() var reports = mapOf<User, Set<Note>>()
private set private set
@@ -145,10 +143,6 @@ class User(
flowSet?.relays?.invalidateData() flowSet?.relays?.invalidateData()
} }
fun updateFollowSetNotes(setNotes: List<AddressableNote>) {
followSetNotes = followSetNotes + setNotes
}
fun addReport(note: Note) { fun addReport(note: Note) {
val author = note.author ?: return val author = note.author ?: return
@@ -188,14 +188,13 @@ class AccountViewModel(
.followSets.stateFlow .followSets.stateFlow
.map { followSetsState -> .map { followSetsState ->
checkNotInMainThread() checkNotInMainThread()
account.getFollowSetNotes() account.getFollowSetNotes().map {
followSetsState.user.followSetNotes.map {
account.mapNoteToFollowSet(it) account.mapNoteToFollowSet(it)
} }
}.flowOn(Dispatchers.Default) }.flowOn(Dispatchers.Default)
.stateIn( .stateIn(
viewModelScope, viewModelScope,
SharingStarted.WhileSubscribed(10000, 10000), SharingStarted.WhileSubscribed(5000, 5000),
emptyList(), emptyList(),
) )
@@ -161,7 +161,7 @@ private fun FollowSetListView(
Row { Row {
IconButton( IconButton(
onClick = { onClick = {
onDeleteUser(item.info?.name.toString()) onDeleteUser(item.pubkeyHex)
}, },
) { ) {
Icon( Icon(