Unused method receiver RVV-B0013
Bug risk
Major
a month ago
—
a year old
Method receiver 'e' is not referenced in method's body, consider removing it
100 return false, nil
101}
102
103func (e *TestEnumResolver) SchemaResolver(fieldInfo shared.AttributeInfo, castName string) (*openapi3.Schema, bool) {104 switch castName {
105 case "datetime":
106 return openapi3.NewDateTimeSchema(), falseMethod receiver 'e' is not referenced in method's body, consider removing it
89var _ shared.EnumResolverInterface = &TestEnumResolver{}
90var _ shared.SchemaResolverInterface = &TestEnumResolver{}
91
92func (e *TestEnumResolver) EnumResolver(t reflect.Type) (bool, shared.Enum) { 93 if t.Name() == "UserGender" {
94 return true, []shared.EnumEntry{
95 {Title: "NOT_SET", Value: 0},Method receiver 'f' is not referenced in method's body, consider removing it
72 return true, nil
73}
74
75func (f *TestFilter) SchemaResolver(fieldInfo shared.AttributeInfo, castName string) (*openapi3.Schema, bool) { 76 switch castName {
77 case "datetime":
78 return openapi3.NewDateTimeSchema(), falseMethod receiver 'r' is not referenced in method's body, consider removing it
28
29type sharedDecoder struct{}
30
31func (r *sharedDecoder) DecodeBody(body io.ReadCloser, target interface{}, obj interface{}) error { 32 data, ok := target.(*someData)
33 if !ok {
34 return fmt.Errorf("invalid type: %T", target)Method receiver 's' is not referenced in method's body, consider removing it
32 return s.generateSchemaFor(ctx, doc, t, 0, shared.AttributeInfo{}, callbacksObject)
33}
34
35func (s *Schema) generateSchemaWithCastedType(ctx context.Context, doc *openapi3.T, castName string, inlineLevel int, fieldInfo shared.AttributeInfo, callbacksObject shared.ChipiCallbacks) (*openapi3.SchemaRef, error) { 36 if !fieldInfo.Empty() {
37 filter, err := callbacksObject.FilterField(ctx, fieldInfo)
38 if err != nil {
Description
Methods with unused receivers can be a symptom of unfinished refactoring or a bug. To keep the same method signature, omit the receiver name or '_' as it is unused.
Bad practice
func (f *Unix) Name() string {
return "unix"
}
func (_ *Unix) Name() string {
return "unix"
}
Recommended
func (*Unix) Name() string {
return "unix"
}