Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove unneeded reference to field dependency manager
  • Loading branch information
jsoriano committed Jun 29, 2023
commit fa6a4d6721a746e8629e4897e09d1c5e5e37195b
20 changes: 3 additions & 17 deletions internal/docs/exported_fields.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,36 +106,22 @@ func collectFieldsFromDefinitions(validator *fields.Validator) ([]fieldsTableRec
root := validator.Schema
var err error
for _, f := range root {
records, err = visitFields("", f, records, validator.FieldDependencyManager)
records, err = visitFields("", f, records)
if err != nil {
return nil, fmt.Errorf("visiting fields failed: %w", err)
}
}
return uniqueTableRecords(records), nil
}

func visitFields(namePrefix string, f fields.FieldDefinition, records []fieldsTableRecord, fdm *fields.DependencyManager) ([]fieldsTableRecord, error) {
func visitFields(namePrefix string, f fields.FieldDefinition, records []fieldsTableRecord) ([]fieldsTableRecord, error) {
var name = namePrefix
if namePrefix != "" {
name += "."
}
name += f.Name

if len(f.Fields) == 0 && f.Type != "group" {
if f.External != "" {
imported, err := fdm.ImportField(f.External, name)
if err != nil {
return nil, fmt.Errorf("can't import field: %w", err)
}

// Override imported fields with the definition, except for the type and external.
var updated fields.FieldDefinition
updated.Update(imported)
updated.Update(f)
updated.Type = imported.Type
updated.External = ""
f = updated
}
records = append(records, fieldsTableRecord{
name: name,
description: f.Description,
Expand All @@ -156,7 +142,7 @@ func visitFields(namePrefix string, f fields.FieldDefinition, records []fieldsTa

var err error
for _, fieldEntry := range f.Fields {
records, err = visitFields(name, fieldEntry, records, fdm)
records, err = visitFields(name, fieldEntry, records)
if err != nil {
return nil, fmt.Errorf("recursive visiting fields failed (namePrefix: %s): %w", namePrefix, err)
}
Expand Down
4 changes: 0 additions & 4 deletions internal/fields/validate.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,6 @@ var (
type Validator struct {
// Schema contains definition records.
Schema []FieldDefinition
// FieldDependencyManager resolves references to external fields
// XXX: Remove this.
FieldDependencyManager *DependencyManager

// SpecVersion contains the version of the spec used by the package.
specVersion semver.Version
Expand Down Expand Up @@ -179,7 +176,6 @@ func createValidatorForDirectoryAndPackageRoot(fieldsParentDir string, finder pa
if err != nil {
return nil, fmt.Errorf("failed to initialize dependency management: %w", err)
}
v.FieldDependencyManager = fdm
}
}

Expand Down