IntenseWebs/mattermost
3
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2025-02-25 18:55:24 -06:00
Code Issues Packages Projects Releases Wiki Activity

refactor: Replace map[string]bool with map[string]struct{} for key existence check

Browse Source
This commit is contained in:
Julien Tant (aider)
2025-02-12 12:46:43 -07:00
parent 3263b04478
commit 43710b018a
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
server/public/model/custom_profile_attributes.go
View File

@@ -66,17 +66,17 @@ func (c CustomProfileAttributesSelectOption) IsValid() error {
type CustomProfileAttributesSelectOptions []CustomProfileAttributesSelectOption
func (c CustomProfileAttributesSelectOptions) IsValid() error {
seenNames := make(map[string]bool)
seenNames := make(map[string]struct{})
for _, option := range c {
if err := option.IsValid(); err != nil {
return err
}
if seenNames[option.Name] {
if _, exists := seenNames[option.Name]; exists {
return errors.New("duplicate option name found")
}
seenNames[option.Name] = true
seenNames[option.Name] = struct{}{}
}
return nil