fix(activitylog): fix limit parameter

Signed-off-by: jkoberg <jkoberg@owncloud.com>

changelog/unreleased/fix-activity-limit.md

@@ -0,0 +1,5 @@
+Bugfix: Fix activity limit
+
+When requesting a limit on activities, ocis would limit first, then filter and sort. Now it filters and sorts first, then limits.
+
+https://github.com/owncloud/ocis/pull/10165

services/activitylog/pkg/service/http.go

@@ -107,10 +107,6 @@ func (s *ActivitylogService) HandleGetItemActivities(w http.ResponseWriter, r *h
 	for _, e := range evRes.GetEvents() {
 		delete(toDelete, e.GetId())
 
-		if limit != 0 && len(resp.Activities) >= limit {
-			continue
-		}
-
 		if !activityAccepted(e) {
 			continue
 		}
@@ -221,6 +217,10 @@ func (s *ActivitylogService) HandleGetItemActivities(w http.ResponseWriter, r *h
 
 	sort(resp.Activities)
 
+	if limit > 0 && limit < len(resp.Activities) {
+		resp.Activities = resp.Activities[:limit]
+	}
+
 	b, err := json.Marshal(resp)
 	if err != nil {
 		s.log.Error().Err(err).Msg("error marshalling activities")