fix: remove domain and user correctly in DomainPolicyUsernamesWriteModel (#4850)

fix: remove Domain and User correctly in DomainPolicyUsernamesWriteModel
This commit is contained in:
Livio Spring 2022-12-08 08:11:49 +01:00 committed by GitHub
parent a3f15d8213
commit 48f535afa1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -109,23 +109,23 @@ func (wm *DomainPolicyUsernamesWriteModel) Reduce() error {
func (wm *DomainPolicyUsernamesWriteModel) removeDomain(domain string) { func (wm *DomainPolicyUsernamesWriteModel) removeDomain(domain string) {
for i, verifiedDomain := range wm.VerifiedDomains { for i, verifiedDomain := range wm.VerifiedDomains {
if verifiedDomain != domain { if verifiedDomain == domain {
continue wm.VerifiedDomains[i] = wm.VerifiedDomains[len(wm.VerifiedDomains)-1]
wm.VerifiedDomains[len(wm.VerifiedDomains)-1] = ""
wm.VerifiedDomains = wm.VerifiedDomains[:len(wm.VerifiedDomains)-1]
return
} }
wm.VerifiedDomains[i] = wm.VerifiedDomains[len(wm.VerifiedDomains)-1]
wm.VerifiedDomains[len(wm.VerifiedDomains)-1] = ""
wm.VerifiedDomains = wm.VerifiedDomains[:len(wm.VerifiedDomains)-1]
} }
} }
func (wm *DomainPolicyUsernamesWriteModel) removeUser(userID string) { func (wm *DomainPolicyUsernamesWriteModel) removeUser(userID string) {
for i, user := range wm.Users { for i, user := range wm.Users {
if user.id != userID { if user.id == userID {
continue wm.Users[i] = wm.Users[len(wm.Users)-1]
wm.Users[len(wm.Users)-1] = nil
wm.Users = wm.Users[:len(wm.Users)-1]
return
} }
wm.Users[i] = wm.Users[len(wm.Users)-1]
wm.Users[len(wm.Users)-1] = nil
wm.Users = wm.Users[len(wm.Users)-1:]
} }
} }