Compare commits
2 Commits
4536be4d10
...
v1.0.0
| Author | SHA1 | Date | |
|---|---|---|---|
| b37121a218 | |||
| ffbeccc4fd |
@@ -1,9 +1,9 @@
|
||||
package main
|
||||
|
||||
import (
|
||||
"github.com/kuzgoga/gormlint/nullSafetyCheck"
|
||||
"github.com/kuzgoga/gormlint/relationsCheck"
|
||||
"golang.org/x/tools/go/analysis/multichecker"
|
||||
"gormlint/nullSafetyCheck"
|
||||
"gormlint/relationsCheck"
|
||||
)
|
||||
|
||||
func main() {
|
||||
|
||||
@@ -1,9 +1,9 @@
|
||||
package nullSafetyCheck
|
||||
|
||||
import (
|
||||
"github.com/kuzgoga/gormlint/common"
|
||||
"go/ast"
|
||||
"golang.org/x/tools/go/analysis"
|
||||
"gormlint/common"
|
||||
"strings"
|
||||
)
|
||||
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
package relationsCheck
|
||||
|
||||
import (
|
||||
"gormlint/common"
|
||||
"github.com/kuzgoga/gormlint/common"
|
||||
)
|
||||
|
||||
func IsBelongsTo(field common.Field, model common.Model, relatedModel common.Model) bool {
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
package relationsCheck
|
||||
|
||||
import "gormlint/common"
|
||||
import "github.com/kuzgoga/gormlint/common"
|
||||
|
||||
func IsHasOne(field common.Field, model common.Model, relatedModel common.Model) bool {
|
||||
foreignKey := field.Tags.GetParamOr("foreignKey", model.Name+"Id")
|
||||
|
||||
@@ -1,10 +1,10 @@
|
||||
package relationsCheck
|
||||
|
||||
import (
|
||||
"github.com/kuzgoga/gormlint/common"
|
||||
"go/token"
|
||||
"go/types"
|
||||
"golang.org/x/tools/go/analysis"
|
||||
"gormlint/common"
|
||||
"strings"
|
||||
)
|
||||
|
||||
|
||||
@@ -2,9 +2,9 @@ package relationsCheck
|
||||
|
||||
import (
|
||||
"fmt"
|
||||
"github.com/kuzgoga/gormlint/common"
|
||||
"go/types"
|
||||
"golang.org/x/tools/go/analysis"
|
||||
"gormlint/common"
|
||||
)
|
||||
|
||||
func findBackReferenceInOneToMany(model common.Model, relatedModel common.Model) *common.Field {
|
||||
|
||||
@@ -1,14 +1,14 @@
|
||||
package relationsCheck
|
||||
|
||||
import (
|
||||
"github.com/kuzgoga/gormlint/common"
|
||||
"golang.org/x/tools/go/analysis"
|
||||
"gormlint/common"
|
||||
"strings"
|
||||
)
|
||||
|
||||
func CheckCascadeDelete(pass *analysis.Pass, field common.Field) bool {
|
||||
if !field.Tags.HasParam("constraint") {
|
||||
pass.Reportf(field.Pos, "field %s should have a constraint", field.Name)
|
||||
pass.Reportf(field.Pos, "field %s should have a delete constraint", field.Name)
|
||||
return true
|
||||
}
|
||||
constraintValue := field.Tags.GetParam("constraint").Value
|
||||
|
||||
@@ -2,8 +2,8 @@ package relationsCheck
|
||||
|
||||
import (
|
||||
"fmt"
|
||||
"github.com/kuzgoga/gormlint/common"
|
||||
"golang.org/x/tools/go/analysis"
|
||||
"gormlint/common"
|
||||
"slices"
|
||||
)
|
||||
|
||||
@@ -123,8 +123,14 @@ func CheckOneToMany(pass *analysis.Pass, models map[string]common.Model) {
|
||||
if !foundOneToMany {
|
||||
if foundBelongsTo {
|
||||
fmt.Printf("`%s` belongs `%s`\n", *baseType, model.Name)
|
||||
if CheckCascadeDelete(pass, field) {
|
||||
return
|
||||
}
|
||||
} else if hasOne {
|
||||
fmt.Printf("`%s` has one `%s` \n", model.Name, relatedModel.Name)
|
||||
if CheckCascadeDelete(pass, field) {
|
||||
return
|
||||
}
|
||||
} else {
|
||||
pass.Reportf(field.Pos, "Invalid relation in field `%s`", field.Name)
|
||||
}
|
||||
|
||||
@@ -1,8 +1,8 @@
|
||||
package tests
|
||||
|
||||
import (
|
||||
"github.com/kuzgoga/gormlint/nullSafetyCheck"
|
||||
"golang.org/x/tools/go/analysis/analysistest"
|
||||
"gormlint/nullSafetyCheck"
|
||||
"testing"
|
||||
)
|
||||
|
||||
|
||||
@@ -1,8 +1,8 @@
|
||||
package tests
|
||||
|
||||
import (
|
||||
"github.com/kuzgoga/gormlint/relationsCheck"
|
||||
"golang.org/x/tools/go/analysis/analysistest"
|
||||
"gormlint/relationsCheck"
|
||||
"testing"
|
||||
)
|
||||
|
||||
|
||||
@@ -65,8 +65,8 @@ type ShoppingCart struct {
|
||||
// Has one
|
||||
|
||||
type Hotel struct {
|
||||
Id uint `gorm:"primaryKey"`
|
||||
Office
|
||||
Id uint `gorm:"primaryKey"`
|
||||
Office // want "field Office should have a delete constraint"
|
||||
}
|
||||
|
||||
type Office struct {
|
||||
|
||||
@@ -45,7 +45,7 @@ type Owner struct {
|
||||
Id uint `gorm:"primaryKey"`
|
||||
Name string
|
||||
CompanyId int
|
||||
Company Company
|
||||
Company Company `gorm:"constraint:OnDelete:CASCADE;"`
|
||||
}
|
||||
|
||||
type Company struct {
|
||||
|
||||
Reference in New Issue
Block a user