From 7ebcfd7d76b5058c9a384b0f9e158a4d90ecbe40 Mon Sep 17 00:00:00 2001 From: Carl Henrik Lunde Date: Thu, 8 Jul 2021 16:18:52 +0200 Subject: [PATCH] s3: consistent resolver function name Signed-off-by: Carl Henrik Lunde --- apis/s3/v1beta1/referencers.go | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/apis/s3/v1beta1/referencers.go b/apis/s3/v1beta1/referencers.go index 435be0ae84..d7729fb66a 100644 --- a/apis/s3/v1beta1/referencers.go +++ b/apis/s3/v1beta1/referencers.go @@ -51,17 +51,6 @@ func BucketARN() reference.ExtractValueFn { } } -// BucketARN2 Why not like this? :shrug: -func BucketARN2(mg resource.Managed) string { - r, ok := mg.(*Bucket) - if !ok { - return "" - } - return r.Status.AtProvider.ARN -} - -var _ reference.ExtractValueFn = BucketARN2 - // ResolveReferences of this Bucket func (mg *Bucket) ResolveReferences(ctx context.Context, c client.Reader) error { // nolint:gocyclo r := reference.NewAPIResolver(c, mg) @@ -125,7 +114,7 @@ func (mg *Bucket) ResolveReferences(ctx context.Context, c client.Reader) error Reference: v.Destination.BucketRef, Selector: v.Destination.BucketSelector, To: reference.To{Managed: &Bucket{}, List: &BucketList{}}, - Extract: BucketARN2, + Extract: BucketARN(), }) if err != nil { return errors.Wrapf(err, "spec.forProvider.replicationConfiguration.rules[%d].bucket", i)