slightly changed the code to have less mutations, added comments

This commit is contained in:
Henry Jameson 2018-12-14 17:33:48 +03:00
parent d7973b0b80
commit 13fbef4550

View file

@ -90,43 +90,54 @@ export default {
store.state.api.backendInteractor.followUser(this.user.id) store.state.api.backendInteractor.followUser(this.user.id)
.then((followedUser) => store.commit('addNewUsers', [followedUser])) .then((followedUser) => store.commit('addNewUsers', [followedUser]))
.then(() => { .then(() => {
// For locked users we just mark it that we sent the follow request
if (this.user.locked) {
this.followRequestInProgress = false
this.followRequestSent = true
return
}
if (this.user.following) { if (this.user.following) {
// If we get result immediately, just stop.
this.followRequestInProgress = false this.followRequestInProgress = false
return return
} }
if (!this.user.locked) {
let attemptsLeft = 3 // But usually we don't get result immediately, so we ask server
const fetchUser = () => new Promise((resolve, reject) => { // for updated user profile to confirm if we are following them
setTimeout(() => { // Sometimes it takes several tries. Sometimes we end up not following
store.state.api.backendInteractor.fetchUser({ id: this.user.id }) // user anyway, probably because they locked themselves and we
.then((user) => store.commit('addNewUsers', [user])) // don't know that yet.
.then(() => resolve(this.user.following)) // Recursive Promise, it will call itself up to 3 times.
.catch((e) => reject(e)) const fetchUser = (attempt) => new Promise((resolve, reject) => {
}, 500) setTimeout(() => {
}).then((confirmed) => { store.state.api.backendInteractor.fetchUser({ id: this.user.id })
if (!confirmed && attemptsLeft > 0) { .then((user) => store.commit('addNewUsers', [user]))
attemptsLeft-- .then(() => resolve([this.user.following, attempt]))
return fetchUser() .catch((e) => reject(e))
} else if (confirmed) { }, 500)
return true }).then(([following, attempt]) => {
if (!following && attempt <= 3) {
// If we BE reports that we still not following that user - retry,
// increment attempts by one
return fetchUser(++attempt)
} else {
// If we run out of attempts, just return whatever status is.
return following
}
})
return fetchUser(1)
.then((following) => {
if (following) {
// We confirmed and everything its good.
this.followRequestInProgress = false
} else { } else {
return false // If after all the tries, just treat it as if user is locked
this.followRequestInProgress = false
this.followRequestSent = true
} }
}) })
return fetchUser()
.then((successfulConfirmation) => {
if (successfulConfirmation) {
this.followRequestInProgress = false
} else {
this.followRequestInProgress = false
this.followRequestSent = true
}
})
} else {
this.followRequestInProgress = false
this.followRequestSent = true
}
}) })
}, },
unfollowUser () { unfollowUser () {