Fix incorrect unsubscription behaviour in the public 'Manage' flow. Closes #1407.

knadh · knadh · commit eafae46409ee · 2023-08-06T11:38:13.000+05:30
diff --git a/cmd/public.go b/cmd/public.go
@@ -313,7 +313,7 @@ func handleSubscriptionPrefs(c echo.Context) error {
 
 	unsubUUIDs := make([]string, 0, len(req.ListUUIDs))
 	for _, s := range subs {
-		if _, ok := reqUUIDs[s.UUID]; !ok {
+		if _, ok := reqUUIDs[s.UUID]; ok {
 			unsubUUIDs = append(unsubUUIDs, s.UUID)
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -313,7 +313,7 @@ func handleSubscriptionPrefs(c echo.Context) error {`
`313`	`313`
`314`	`314`	`unsubUUIDs := make([]string, 0, len(req.ListUUIDs))`
`315`	`315`	`for _, s := range subs {`
`316`		`- if _, ok := reqUUIDs[s.UUID]; !ok {`
	`316`	`+ if _, ok := reqUUIDs[s.UUID]; ok {`
`317`	`317`	`unsubUUIDs = append(unsubUUIDs, s.UUID)`
`318`	`318`	`}`
`319`	`319`	`}`