mirror of
https://github.com/hanzoai/team.git
synced 2026-09-30 01:57:35 +00:00
Jesse HallamandGitHub
edf2075381
* MM-68149: upgrade to Go 1.26.2 Update go directive in go.mod and .go-version. * MM-68149: replace pointer helpers with Go 1.26 new() Go 1.26 extends the built-in new() to accept an initial value expression, making typed-pointer helpers like model.NewPointer(x), bToP(x), and boolPtr(x) redundant. Replace every call site with new(x) and remove the now-unused helper functions and their //go:fix inline directives. * MM-68149: apply go fix for reflect API and format-string changes - reflect.Ptr → reflect.Pointer (renamed in Go 1.18, deprecated alias removed in 1.26) - reflect range-over-struct: for i := 0; i < t.NumField(); i++ → for field := range t.Fields() and the equivalent for Methods() and interface types - Fix format-string concatenation and variadic-arg mismatches flagged by go vet * MM-68149: update JPEG fixtures and test infrastructure for Go 1.26 encoder Go 1.26 ships a new image/jpeg encoder that produces slightly different output. Regenerate all JPEG fixture files and switch the comparison helpers from byte-equality to pixel-level comparison with a small per-channel tolerance, so minor encoder drift across patch versions is handled automatically. Add -update-fixtures flag to make it easy to regenerate fixtures after future major Go upgrades. Document the update procedure in tests/README.md. * MM-68149: CI check that go fix ./... produces no changes * Fix real bugs flagged by CodeRabbit review - group.go: set newGroup.MemberCount not group.MemberCount (member count was populated on the wrong variable and lost before publish/return) - file_test.go: guard compareImage(GetFilePreview) on the preview slice length, not the thumbnail slice length (copy-paste error) - config_test.go: remove duplicate MinimumLength assignment * fixup! Fix real bugs flagged by CodeRabbit review
59 lines
1.6 KiB
Go
59 lines
1.6 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package model
|
|
|
|
import (
|
|
"encoding/json"
|
|
"errors"
|
|
)
|
|
|
|
type OrphanedRecord struct {
|
|
ParentId *string `json:"parent_id"`
|
|
ChildId *string `json:"child_id"`
|
|
}
|
|
|
|
type RelationalIntegrityCheckData struct {
|
|
ParentName string `json:"parent_name"`
|
|
ChildName string `json:"child_name"`
|
|
ParentIdAttr string `json:"parent_id_attr"`
|
|
ChildIdAttr string `json:"child_id_attr"`
|
|
Records []OrphanedRecord `json:"records"`
|
|
}
|
|
|
|
type IntegrityCheckResult struct {
|
|
Data any `json:"data"`
|
|
Err error `json:"err"`
|
|
}
|
|
|
|
func (r *IntegrityCheckResult) UnmarshalJSON(b []byte) error {
|
|
var data map[string]any
|
|
if err := json.Unmarshal(b, &data); err != nil {
|
|
return err
|
|
}
|
|
if d, ok := data["data"]; ok && d != nil {
|
|
var rdata RelationalIntegrityCheckData
|
|
m := d.(map[string]any)
|
|
rdata.ParentName = m["parent_name"].(string)
|
|
rdata.ChildName = m["child_name"].(string)
|
|
rdata.ParentIdAttr = m["parent_id_attr"].(string)
|
|
rdata.ChildIdAttr = m["child_id_attr"].(string)
|
|
for _, recData := range m["records"].([]any) {
|
|
var record OrphanedRecord
|
|
m := recData.(map[string]any)
|
|
if val := m["parent_id"]; val != nil {
|
|
record.ParentId = new(val.(string))
|
|
}
|
|
if val := m["child_id"]; val != nil {
|
|
record.ChildId = new(val.(string))
|
|
}
|
|
rdata.Records = append(rdata.Records, record)
|
|
}
|
|
r.Data = rdata
|
|
}
|
|
if err, ok := data["err"]; ok && err != nil {
|
|
r.Err = errors.New(data["err"].(string))
|
|
}
|
|
return nil
|
|
}
|