Skip to content

Commit

Permalink
fix: return correct dependencies on REST endpoint
Browse files Browse the repository at this point in the history
  • Loading branch information
Vilsol committed Aug 19, 2024
1 parent bd02e8f commit b6d6d3c
Showing 1 changed file with 21 additions and 4 deletions.
25 changes: 21 additions & 4 deletions nodes/mod.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import (
"github.com/satisfactorymodding/smr-api/db"
"github.com/satisfactorymodding/smr-api/generated"
"github.com/satisfactorymodding/smr-api/generated/conv"
"github.com/satisfactorymodding/smr-api/generated/ent"
mod2 "github.com/satisfactorymodding/smr-api/generated/ent/mod"
version2 "github.com/satisfactorymodding/smr-api/generated/ent/version"
"github.com/satisfactorymodding/smr-api/generated/ent/versiontarget"
Expand Down Expand Up @@ -252,7 +253,7 @@ func getModVersions(c echo.Context) (interface{}, *ErrorResponse) {

modID := c.Param("modId")

mod, err := db.From(c.Request().Context()).Mod.Query().
mod, err := db.From(c.Request().Context()).Debug().Mod.Query().

Check warning on line 256 in nodes/mod.go

View check run for this annotation

Codecov / codecov/patch

nodes/mod.go#L256

Added line #L256 was not covered by tests
WithTags().
Where(mod2.ID(modID)).
First(c.Request().Context())
Expand All @@ -266,7 +267,9 @@ func getModVersions(c echo.Context) (interface{}, *ErrorResponse) {
}

versions, err := mod.QueryVersions().
WithDependencies().
WithVersionDependencies(func(query *ent.VersionDependencyQuery) {
query.WithMod()
}).

Check warning on line 272 in nodes/mod.go

View check run for this annotation

Codecov / codecov/patch

nodes/mod.go#L270-L272

Added lines #L270 - L272 were not covered by tests
WithTargets().
Limit(limit).
Offset(offset).
Expand All @@ -278,7 +281,13 @@ func getModVersions(c echo.Context) (interface{}, *ErrorResponse) {
return nil, &ErrorVersionNotFound
}

return (*conv.VersionImpl)(nil).ConvertSlice(versions), nil
for _, version := range versions {
for _, dependency := range version.Edges.VersionDependencies {
dependency.ModID = dependency.Edges.Mod.ModReference

Check warning on line 286 in nodes/mod.go

View check run for this annotation

Codecov / codecov/patch

nodes/mod.go#L284-L286

Added lines #L284 - L286 were not covered by tests
}
}

return (*conv.ModAllVersionsImpl)(nil).ConvertSlice(versions), nil

Check warning on line 290 in nodes/mod.go

View check run for this annotation

Codecov / codecov/patch

nodes/mod.go#L290

Added line #L290 was not covered by tests
}

// @Summary Retrieve a Mod Authors
Expand Down Expand Up @@ -478,7 +487,9 @@ func getAllModVersions(c echo.Context) (interface{}, *ErrorResponse) {
}

versions, err := mod.QueryVersions().
WithVersionDependencies().
WithVersionDependencies(func(query *ent.VersionDependencyQuery) {
query.WithMod()
}).

Check warning on line 492 in nodes/mod.go

View check run for this annotation

Codecov / codecov/patch

nodes/mod.go#L490-L492

Added lines #L490 - L492 were not covered by tests
WithTargets().
Where(version2.Approved(true), version2.Denied(false)).
Select(version2.FieldHash, version2.FieldSize, version2.FieldGameVersion, version2.FieldVersion).
Expand All @@ -488,5 +499,11 @@ func getAllModVersions(c echo.Context) (interface{}, *ErrorResponse) {
return nil, &ErrorVersionNotFound
}

for _, version := range versions {
for _, dependency := range version.Edges.VersionDependencies {
dependency.ModID = dependency.Edges.Mod.ModReference

Check warning on line 504 in nodes/mod.go

View check run for this annotation

Codecov / codecov/patch

nodes/mod.go#L502-L504

Added lines #L502 - L504 were not covered by tests
}
}

return (*conv.ModAllVersionsImpl)(nil).ConvertSlice(versions), nil
}

0 comments on commit b6d6d3c

Please sign in to comment.