cache: consistently use "ref" to fetch a repository

Michael Muré created 6 years ago

Change summary

cache/multi_repo_cache.go     | 2 +-
graphql/resolvers/mutation.go | 6 +++---
graphql/resolvers/query.go    | 4 ++--
3 files changed, 6 insertions(+), 6 deletions(-)

Detailed changes

cache/multi_repo_cache.go

@@ -54,7 +54,7 @@ func (c *MultiRepoCache) DefaultRepo() (*RepoCache, error) {
 	panic("unreachable")
 }
 
-// ResolveRepo retrieve a repository by name
+// ResolveRepo retrieve a repository with a reference
 func (c *MultiRepoCache) ResolveRepo(ref string) (*RepoCache, error) {
 	r, ok := c.repos[ref]
 	if !ok {

graphql/resolvers/mutation.go

@@ -15,9 +15,9 @@ type mutationResolver struct {
 	cache *cache.MultiRepoCache
 }
 
-func (r mutationResolver) getRepo(repoRef *string) (*cache.RepoCache, error) {
-	if repoRef != nil {
-		return r.cache.ResolveRepo(*repoRef)
+func (r mutationResolver) getRepo(ref *string) (*cache.RepoCache, error) {
+	if ref != nil {
+		return r.cache.ResolveRepo(*ref)
 	}
 
 	return r.cache.DefaultRepo()

graphql/resolvers/query.go

@@ -27,8 +27,8 @@ func (r rootQueryResolver) DefaultRepository(ctx context.Context) (*models.Repos
 	}, nil
 }
 
-func (r rootQueryResolver) Repository(ctx context.Context, id string) (*models.Repository, error) {
-	repo, err := r.cache.ResolveRepo(id)
+func (r rootQueryResolver) Repository(ctx context.Context, ref string) (*models.Repository, error) {
+	repo, err := r.cache.ResolveRepo(ref)
 
 	if err != nil {
 		return nil, err