Skip to content

Commit d20920e

Browse files
committed
Removed function and moved inline
1 parent c5dd37f commit d20920e

1 file changed

Lines changed: 20 additions & 24 deletions

File tree

rest-api/api/pkg/api/handler/tenantaccount.go

Lines changed: 20 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -569,10 +569,16 @@ func (gtah GetTenantAccountHandler) Handle(c echo.Context) error {
569569
return cutil.NewAPIErrorResponse(c, http.StatusInternalServerError, "Failed to retrieve Status Details for TenantAccount", nil)
570570
}
571571

572-
tenantSites, err := loadTenantSitesForTenantAccount(ctx, gtah.dbSession, ta)
573-
if err != nil {
574-
logger.Error().Err(err).Msg("error retrieving Tenant Sites for Tenant Account")
575-
return cutil.NewAPIErrorResponse(c, http.StatusInternalServerError, "Failed to retrieve Tenant Sites for Tenant Account", nil)
572+
var tenantSites []cdbm.TenantSite
573+
if ta != nil && ta.TenantID != nil {
574+
tsDAO := cdbm.NewTenantSiteDAO(gtah.dbSession)
575+
tenantSites, _, err = tsDAO.GetAll(ctx, nil, cdbm.TenantSiteFilterInput{
576+
TenantIDs: []uuid.UUID{*ta.TenantID},
577+
}, cdbp.PageInput{Limit: cutil.GetPtr(cdbp.TotalLimit)}, []string{"Site"})
578+
if err != nil {
579+
logger.Error().Err(err).Msg("error retrieving Tenant Sites for Tenant Account")
580+
return cutil.NewAPIErrorResponse(c, http.StatusInternalServerError, "Failed to retrieve Tenant Sites for Tenant Account", nil)
581+
}
576582
}
577583

578584
// Create response
@@ -691,22 +697,6 @@ func (utah UpdateTenantAccountHandler) Handle(c echo.Context) error {
691697
return utah.handleTenantInviteAcceptance(c, ctx, logger, org, dbUser, taID, apiRequest)
692698
}
693699

694-
func loadTenantSitesForTenantAccount(ctx context.Context, dbSession *cdb.Session, ta *cdbm.TenantAccount) ([]cdbm.TenantSite, error) {
695-
if ta == nil || ta.TenantID == nil {
696-
return nil, nil
697-
}
698-
699-
tsDAO := cdbm.NewTenantSiteDAO(dbSession)
700-
tenantSites, _, err := tsDAO.GetAll(ctx, nil, cdbm.TenantSiteFilterInput{
701-
TenantIDs: []uuid.UUID{*ta.TenantID},
702-
}, cdbp.PageInput{Limit: cutil.GetPtr(cdbp.TotalLimit)}, []string{"Site"})
703-
if err != nil {
704-
return nil, err
705-
}
706-
707-
return tenantSites, nil
708-
}
709-
710700
func (utah UpdateTenantAccountHandler) handleTenantInviteAcceptance(c echo.Context, ctx context.Context, logger zerolog.Logger, org string, dbUser *cdbm.User, taID uuid.UUID, apiRequest model.APITenantAccountUpdateRequest) error {
711701
taDAO := cdbm.NewTenantAccountDAO(utah.dbSession)
712702

@@ -912,10 +902,16 @@ func (utah UpdateTenantAccountHandler) handleProviderSiteCapabilitiesUpdate(c ec
912902
return cutil.NewAPIErrorResponse(c, http.StatusInternalServerError, "Failed to retrieve updated Tenant Account", nil)
913903
}
914904

915-
tenantSites, err := loadTenantSitesForTenantAccount(ctx, utah.dbSession, ta)
916-
if err != nil {
917-
logger.Error().Err(err).Msg("error retrieving Tenant Sites for updated Tenant Account")
918-
return cutil.NewAPIErrorResponse(c, http.StatusInternalServerError, "Failed to retrieve Tenant Sites for Tenant Account", nil)
905+
var tenantSites []cdbm.TenantSite
906+
if ta != nil && ta.TenantID != nil {
907+
tsDAO := cdbm.NewTenantSiteDAO(utah.dbSession)
908+
tenantSites, _, err = tsDAO.GetAll(ctx, nil, cdbm.TenantSiteFilterInput{
909+
TenantIDs: []uuid.UUID{*ta.TenantID},
910+
}, cdbp.PageInput{Limit: cutil.GetPtr(cdbp.TotalLimit)}, []string{"Site"})
911+
if err != nil {
912+
logger.Error().Err(err).Msg("error retrieving Tenant Sites for updated Tenant Account")
913+
return cutil.NewAPIErrorResponse(c, http.StatusInternalServerError, "Failed to retrieve Tenant Sites for Tenant Account", nil)
914+
}
919915
}
920916

921917
sdDAO := cdbm.NewStatusDetailDAO(utah.dbSession)

0 commit comments

Comments
 (0)