diff options
| author |  Daenney <daenney@users.noreply.github.com> | 2023-08-06 12:22:40 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2023-08-06 12:22:40 +0200 |
| commit | c1375ca5c1d5a8a645151cedfced5a336f6b7467 (patch) | |
| tree | c3cc68d992811de674458e646af16f4659926ec7 /internal/gtsmodel/report.go | |
| parent | [bugfix] update cache library with nil ptr fix (#2070) (diff) | |
| download | gotosocial-c1375ca5c1d5a8a645151cedfced5a336f6b7467.tar.xz | |
[chore] Remove go-playground/validator (#2069)
* [chore] Remove go-playground/validator
It turns out we're not actually using the validator code. This is a
remnant from when we intended to use it, but the presence of it and its
struct tags creates the illusion we're validating a lot of things we're
not. It resulted in some confusion when we were trying to figure out
language valdiation.
Remove all this code, so that only the validation functions from the
validate package we actually use remain. I'm not touching the struct
tags in the migrations in order to avoid things potentially thinking
migrations need to be re-run.
* [chore] Bring back a struct tag on api
The validate on internal/api is Gin doing form validation, not the
validator from go-playground/validator.
Diffstat (limited to 'internal/gtsmodel/report.go')
| -rw-r--r-- | internal/gtsmodel/report.go | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/internal/gtsmodel/report.go b/internal/gtsmodel/report.go index eb45c820c..e5b942563 100644 --- a/internal/gtsmodel/report.go +++ b/internal/gtsmodel/report.go @@ -26,20 +26,20 @@ import "time" // or another instance, OR a report that was created remotely (on another instance) // about a user on this instance, and received via the federated (s2s) API. type Report struct { - ID string `validate:"required,ulid" bun:"type:CHAR(26),pk,nullzero,notnull,unique"` // id of this item in the database - CreatedAt time.Time `validate:"-" bun:"type:timestamptz,nullzero,notnull,default:current_timestamp"` // when was item created - UpdatedAt time.Time `validate:"-" bun:"type:timestamptz,nullzero,notnull,default:current_timestamp"` // when was item last updated - URI string `validate:"required,url" bun:",unique,nullzero,notnull"` // activitypub URI of this report - AccountID string `validate:"required,ulid" bun:"type:CHAR(26),nullzero,notnull"` // which account created this report - Account *Account `validate:"-" bun:"-"` // account corresponding to AccountID - TargetAccountID string `validate:"required,ulid" bun:"type:CHAR(26),nullzero,notnull"` // which account is targeted by this report - TargetAccount *Account `validate:"-" bun:"-"` // account corresponding to TargetAccountID - Comment string `validate:"-" bun:",nullzero"` // comment / explanation for this report, by the reporter - StatusIDs []string `validate:"dive,ulid" bun:"statuses,array"` // database IDs of any statuses referenced by this report - Statuses []*Status `validate:"-" bun:"-"` // statuses corresponding to StatusIDs - Forwarded *bool `validate:"-" bun:",nullzero,notnull,default:false"` // flag to indicate report should be forwarded to remote instance - ActionTaken string `validate:"-" bun:",nullzero"` // string description of what action was taken in response to this report - ActionTakenAt time.Time `validate:"-" bun:"type:timestamptz,nullzero"` // time at which action was taken, if any - ActionTakenByAccountID string `validate:",omitempty,ulid" bun:"type:CHAR(26),nullzero"` // database ID of account which took action, if any - ActionTakenByAccount *Account `validate:"-" bun:"-"` // account corresponding to ActionTakenByID, if any + ID string `bun:"type:CHAR(26),pk,nullzero,notnull,unique"` // id of this item in the database + CreatedAt time.Time `bun:"type:timestamptz,nullzero,notnull,default:current_timestamp"` // when was item created + UpdatedAt time.Time `bun:"type:timestamptz,nullzero,notnull,default:current_timestamp"` // when was item last updated + URI string `bun:",unique,nullzero,notnull"` // activitypub URI of this report + AccountID string `bun:"type:CHAR(26),nullzero,notnull"` // which account created this report + Account *Account `bun:"-"` // account corresponding to AccountID + TargetAccountID string `bun:"type:CHAR(26),nullzero,notnull"` // which account is targeted by this report + TargetAccount *Account `bun:"-"` // account corresponding to TargetAccountID + Comment string `bun:",nullzero"` // comment / explanation for this report, by the reporter + StatusIDs []string `bun:"statuses,array"` // database IDs of any statuses referenced by this report + Statuses []*Status `bun:"-"` // statuses corresponding to StatusIDs + Forwarded *bool `bun:",nullzero,notnull,default:false"` // flag to indicate report should be forwarded to remote instance + ActionTaken string `bun:",nullzero"` // string description of what action was taken in response to this report + ActionTakenAt time.Time `bun:"type:timestamptz,nullzero"` // time at which action was taken, if any + ActionTakenByAccountID string `bun:"type:CHAR(26),nullzero"` // database ID of account which took action, if any + ActionTakenByAccount *Account `bun:"-"` // account corresponding to ActionTakenByID, if any } |