Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions graphql/schema/types/filters.graphql
Original file line number Diff line number Diff line change
Expand Up @@ -423,6 +423,8 @@ input GroupFilterType {
NOT: GroupFilterType

name: StringCriterionInput
"Filter by aliases"
aliases: StringCriterionInput
director: StringCriterionInput
synopsis: StringCriterionInput

Expand Down
1 change: 1 addition & 0 deletions pkg/models/group.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
1 change: 1 addition & 0 deletions pkg/sqlite/group_filter.go
Original file line number Diff line number Diff line change
Expand Up @@ -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),
Expand Down
60 changes: 56 additions & 4 deletions pkg/sqlite/group_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -328,6 +328,7 @@ func emptyGroup(idx int) models.Group {
return models.Group{
ID: groupIDs[idx],
Name: groupNames[idx],
Aliases: "",
TagIDs: models.NewRelatedIDs([]int{}),
ContainingGroups: models.NewRelatedGroupDescriptions([]models.GroupIDDescription{}),
SubGroups: models.NewRelatedGroupDescriptions([]models.GroupIDDescription{}),
Expand Down Expand Up @@ -452,8 +453,9 @@ func Test_groupQueryBuilder_UpdatePartial(t *testing.T) {
},
},
models.Group{
ID: groupIDs[groupIdxWithParent],
Name: groupNames[groupIdxWithParent],
ID: groupIDs[groupIdxWithParent],
Name: groupNames[groupIdxWithParent],
Aliases: getGroupEmptyString(groupIdxWithParent, "Aliases"),
ContainingGroups: models.NewRelatedGroupDescriptions([]models.GroupIDDescription{
{GroupID: groupIDs[groupIdxWithChild]},
{GroupID: groupIDs[groupIdxWithScene], Description: containingGroupDescription},
Expand All @@ -473,8 +475,9 @@ func Test_groupQueryBuilder_UpdatePartial(t *testing.T) {
},
},
models.Group{
ID: groupIDs[groupIdxWithChild],
Name: groupNames[groupIdxWithChild],
ID: groupIDs[groupIdxWithChild],
Name: groupNames[groupIdxWithChild],
Aliases: getGroupEmptyString(groupIdxWithChild, "Aliases"),
SubGroups: models.NewRelatedGroupDescriptions([]models.GroupIDDescription{
{GroupID: groupIDs[groupIdxWithParent]},
{GroupID: groupIDs[groupIdxWithScene], Description: subGroupDescription},
Expand All @@ -496,6 +499,7 @@ func Test_groupQueryBuilder_UpdatePartial(t *testing.T) {
models.Group{
ID: groupIDs[groupIdxWithParent],
Name: groupNames[groupIdxWithParent],
Aliases: getGroupEmptyString(groupIdxWithParent, "Aliases"),
ContainingGroups: models.NewRelatedGroupDescriptions([]models.GroupIDDescription{}),
},
false,
Expand All @@ -514,6 +518,7 @@ func Test_groupQueryBuilder_UpdatePartial(t *testing.T) {
models.Group{
ID: groupIDs[groupIdxWithChild],
Name: groupNames[groupIdxWithChild],
Aliases: getGroupEmptyString(groupIdxWithChild, "Aliases"),
SubGroups: models.NewRelatedGroupDescriptions([]models.GroupIDDescription{}),
},
false,
Expand Down Expand Up @@ -885,6 +890,53 @@ func TestGroupQueryURL(t *testing.T) {
verifyGroupQuery(t, filter, verifyFn)
}

func TestGroupQueryAliases(t *testing.T) {
const groupIdx = 1
const aliasesField = "Aliases"
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 = "0001_Ali"
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
Expand Down
4 changes: 4 additions & 0 deletions pkg/sqlite/scene_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2787,6 +2787,10 @@ func verifyString(t *testing.T, value string, criterion models.StringCriterionIn
assert.Equal(criterion.Value, value)
case models.CriterionModifierNotEquals:
assert.NotEqual(criterion.Value, value)
case models.CriterionModifierIncludes:
assert.Contains(value, criterion.Value)
case models.CriterionModifierExcludes:
assert.NotContains(value, criterion.Value)
case models.CriterionModifierMatchesRegex:
assert.Regexp(regexp.MustCompile(criterion.Value), value)
case models.CriterionModifierNotMatchesRegex:
Expand Down
4 changes: 3 additions & 1 deletion pkg/sqlite/setup_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1527,8 +1527,10 @@ func createGroups(ctx context.Context, mqb models.GroupReaderWriter, n int, o in
// groups [ i ] and [ n + o - i - 1 ] should have similar names with only the Name!=NaMe part different

name = getGroupStringValue(index, name)
aliases := getGroupEmptyString(i, "Aliases")
group := models.Group{
Name: name,
Name: name,
Aliases: aliases,
URLs: models.NewRelatedStrings([]string{
getGroupEmptyString(i, urlField),
}),
Expand Down
1 change: 1 addition & 0 deletions ui/v2.5/src/models/list-filter/groups.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ const criterionOptions = [
GroupIsMissingCriterionOption,
createStringCriterionOption("url"),
createStringCriterionOption("name"),
createStringCriterionOption("aliases"),
createStringCriterionOption("director"),
createStringCriterionOption("synopsis"),
createDurationCriterionOption("duration"),
Expand Down
Loading