Skip to content

Commit

Permalink
Fix bug while reading memberships (#18)
Browse files Browse the repository at this point in the history
  • Loading branch information
raj-saxena authored Mar 4, 2022
1 parent a5d130f commit c6d9f48
Showing 1 changed file with 12 additions and 7 deletions.
19 changes: 12 additions & 7 deletions metabase/resource_membership.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,11 @@ package metabase

import (
"context"
"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
"strconv"
"terraform-provider-metabase/client"

"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
)

func resourceMembership() *schema.Resource {
Expand Down Expand Up @@ -79,12 +80,16 @@ func resourceMembershipRead(_ context.Context, d *schema.ResourceData, meta inte

c := meta.(*client.Client)
var m client.Membership
if memberships, err := c.GetMemberships(); err != nil {
m = findMatchingMembership(memberships, membershipId)

if m == (client.Membership{}) {
return diag.Errorf("Could not find Membership by id [%d] in memberships[%+v]", membershipId, memberships)
}
memberships, err := c.GetMemberships()
if err != nil {
return diag.FromErr(err)
}

m = findMatchingMembership(memberships, membershipId)

if m == (client.Membership{}) {
return diag.Errorf("Could not find Membership by id [%d] in memberships[%+v]", membershipId, memberships)
}

if err := d.Set("user_id", m.UserId); err != nil {
Expand Down

0 comments on commit c6d9f48

Please sign in to comment.