From 5cb3760b063e6515544288f62d0fd047f7952bc0 Mon Sep 17 00:00:00 2001 From: Slick Daddy <129640104+slick-daddy@users.noreply.github.com> Date: Fri, 8 May 2026 04:32:41 +0300 Subject: [PATCH] Merge pull request #4 from slick-daddy/codex/create-new-branch-patch-4 Add 'aliases' filter to Group (GraphQL, backend, tests, and UI) --- graphql/schema/types/filters.graphql | 2 ++ pkg/models/group.go | 1 + pkg/sqlite/group_filter.go | 1 + pkg/sqlite/group_test.go | 46 ++++++++++++++++++++++++ ui/v2.5/src/models/list-filter/groups.ts | 1 + 5 files changed, 51 insertions(+) diff --git a/graphql/schema/types/filters.graphql b/graphql/schema/types/filters.graphql index c7d880266..593ea7790 100644 --- a/graphql/schema/types/filters.graphql +++ b/graphql/schema/types/filters.graphql @@ -423,6 +423,8 @@ input GroupFilterType { NOT: GroupFilterType name: StringCriterionInput + "Filter by aliases" + aliases: StringCriterionInput director: StringCriterionInput synopsis: StringCriterionInput diff --git a/pkg/models/group.go b/pkg/models/group.go index 396384b51..3d4a37a3d 100644 --- a/pkg/models/group.go +++ b/pkg/models/group.go @@ -3,6 +3,7 @@ package models type GroupFilterType struct { OperatorFilter[GroupFilterType] Name *StringCriterionInput `json:"name"` + Aliases *StringCriterionInput `json:"aliases"` Director *StringCriterionInput `json:"director"` Synopsis *StringCriterionInput `json:"synopsis"` // Filter by duration (in seconds) diff --git a/pkg/sqlite/group_filter.go b/pkg/sqlite/group_filter.go index 63d056679..d5e5ce571 100644 --- a/pkg/sqlite/group_filter.go +++ b/pkg/sqlite/group_filter.go @@ -64,6 +64,7 @@ func (qb *groupFilterHandler) criterionHandler() criterionHandler { groupFilter := qb.groupFilter return compoundHandler{ stringCriterionHandler(groupFilter.Name, "groups.name"), + stringCriterionHandler(groupFilter.Aliases, "groups.aliases"), stringCriterionHandler(groupFilter.Director, "groups.director"), stringCriterionHandler(groupFilter.Synopsis, "groups.description"), intCriterionHandler(groupFilter.Rating100, "groups.rating", nil), diff --git a/pkg/sqlite/group_test.go b/pkg/sqlite/group_test.go index 22b551e02..fb0255fe3 100644 --- a/pkg/sqlite/group_test.go +++ b/pkg/sqlite/group_test.go @@ -885,6 +885,52 @@ func TestGroupQueryURL(t *testing.T) { verifyGroupQuery(t, filter, verifyFn) } +func TestGroupQueryAliases(t *testing.T) { + const groupIdx = 1 + groupAliases := getGroupStringValue(groupIdx, aliasesField) + + aliasesCriterion := models.StringCriterionInput{ + Value: groupAliases, + Modifier: models.CriterionModifierEquals, + } + + filter := models.GroupFilterType{ + Aliases: &aliasesCriterion, + } + + verifyFn := func(n *models.Group) { + t.Helper() + verifyString(t, n.Aliases, aliasesCriterion) + } + + verifyGroupQuery(t, filter, verifyFn) + + aliasesCriterion.Modifier = models.CriterionModifierNotEquals + verifyGroupQuery(t, filter, verifyFn) + + aliasesCriterion.Modifier = models.CriterionModifierIncludes + aliasesCriterion.Value = "oup_1_alia" + verifyGroupQuery(t, filter, verifyFn) + + aliasesCriterion.Modifier = models.CriterionModifierExcludes + aliasesCriterion.Value = "not-present-alias-substring" + verifyGroupQuery(t, filter, verifyFn) + + aliasesCriterion.Modifier = models.CriterionModifierMatchesRegex + aliasesCriterion.Value = "group_.*1_aliases" + verifyGroupQuery(t, filter, verifyFn) + + aliasesCriterion.Modifier = models.CriterionModifierNotMatchesRegex + verifyGroupQuery(t, filter, verifyFn) + + aliasesCriterion.Modifier = models.CriterionModifierIsNull + aliasesCriterion.Value = "" + verifyGroupQuery(t, filter, verifyFn) + + aliasesCriterion.Modifier = models.CriterionModifierNotNull + verifyGroupQuery(t, filter, verifyFn) +} + func TestGroupQueryURLExcludes(t *testing.T) { withRollbackTxn(func(ctx context.Context) error { mqb := db.Group diff --git a/ui/v2.5/src/models/list-filter/groups.ts b/ui/v2.5/src/models/list-filter/groups.ts index 9c5b3f2d4..7b03ca874 100644 --- a/ui/v2.5/src/models/list-filter/groups.ts +++ b/ui/v2.5/src/models/list-filter/groups.ts @@ -49,6 +49,7 @@ const criterionOptions = [ GroupIsMissingCriterionOption, createStringCriterionOption("url"), createStringCriterionOption("name"), + createStringCriterionOption("aliases"), createStringCriterionOption("director"), createStringCriterionOption("synopsis"), createDurationCriterionOption("duration"),