X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=capifcore%2Finternal%2Fpublishservice%2Fpublishservice.go;h=bf798999eae3f8d36d561249b5b5b2baff119870;hb=65f718afb57c1aab02fa48d34582f133ff65d5af;hp=ee3efef25acea45532afb6af041c53e339a56b80;hpb=c21e101767fbde3a091a0ee7fef7782fedbe5b37;p=nonrtric%2Fplt%2Fsme.git

diff --git a/capifcore/internal/publishservice/publishservice.go b/capifcore/internal/publishservice/publishservice.go
index ee3efef..bf79899 100644
--- a/capifcore/internal/publishservice/publishservice.go
+++ b/capifcore/internal/publishservice/publishservice.go
@@ -42,9 +42,6 @@ import (
 
 //go:generate mockery --name PublishRegister
 type PublishRegister interface {
-	// Checks if the provided APIs are published.
-	// Returns true if all provided APIs have been published, false otherwise.
-	AreAPIsPublished(serviceDescriptions *[]publishapi.ServiceAPIDescription) bool
 	// Checks if the provided API is published.
 	// Returns true if the provided API has been published, false otherwise.
 	IsAPIPublished(aefId, path string) bool
@@ -71,15 +68,6 @@ func NewPublishService(serviceRegister providermanagement.ServiceRegister, hm he
 	}
 }
 
-func (ps *PublishService) AreAPIsPublished(serviceDescriptions *[]publishapi.ServiceAPIDescription) bool {
-
-	if serviceDescriptions != nil {
-		registeredApis := ps.getAllAefIds()
-		return checkNewDescriptions(*serviceDescriptions, registeredApis)
-	}
-	return true
-}
-
 func (ps *PublishService) getAllAefIds() []string {
 	ps.lock.Lock()
 	defer ps.lock.Unlock()
@@ -87,46 +75,12 @@ func (ps *PublishService) getAllAefIds() []string {
 	allIds := []string{}
 	for _, descriptions := range ps.publishedServices {
 		for _, description := range descriptions {
-			allIds = append(allIds, getIdsFromDescription(description)...)
+			allIds = append(allIds, description.GetAefIds()...)
 		}
 	}
 	return allIds
 }
 
-func getIdsFromDescription(description publishapi.ServiceAPIDescription) []string {
-	allIds := []string{}
-	if description.AefProfiles != nil {
-		for _, aefProfile := range *description.AefProfiles {
-			allIds = append(allIds, aefProfile.AefId)
-		}
-	}
-	return allIds
-}
-
-func checkNewDescriptions(newDescriptions []publishapi.ServiceAPIDescription, registeredAefIds []string) bool {
-	registered := true
-	for _, newApi := range newDescriptions {
-		if !checkProfiles(newApi.AefProfiles, registeredAefIds) {
-			registered = false
-			break
-		}
-	}
-	return registered
-}
-
-func checkProfiles(newProfiles *[]publishapi.AefProfile, registeredAefIds []string) bool {
-	allRegistered := true
-	if newProfiles != nil {
-		for _, profile := range *newProfiles {
-			if !slices.Contains(registeredAefIds, profile.AefId) {
-				allRegistered = false
-				break
-			}
-		}
-	}
-	return allRegistered
-}
-
 func (ps *PublishService) IsAPIPublished(aefId, path string) bool {
 	return slices.Contains(ps.getAllAefIds(), aefId)
 }
@@ -181,8 +135,7 @@ func (ps *PublishService) PostApfIdServiceApis(ctx echo.Context, apfId string) e
 		}
 	}
 
-	newId := "api_id_" + newServiceAPIDescription.ApiName
-	newServiceAPIDescription.ApiId = &newId
+	newServiceAPIDescription.PrepareNewService()
 
 	shouldReturn, returnValue := ps.installHelmChart(newServiceAPIDescription, ctx)
 	if shouldReturn {
@@ -307,11 +260,11 @@ func (ps *PublishService) PutApfIdServiceApisServiceApiId(ctx echo.Context, apfI
 	if err != nil {
 		return sendCoreError(ctx, http.StatusBadRequest, fmt.Sprintf(errMsg, err))
 	}
-	ps.updateDescription(pos, apfId, &updatedServiceDescription, &publishedService)
 	err = ps.checkProfilesRegistered(apfId, *updatedServiceDescription.AefProfiles)
 	if err != nil {
 		return sendCoreError(ctx, http.StatusBadRequest, fmt.Sprintf(errMsg, err))
 	}
+	ps.updateDescription(pos, apfId, &updatedServiceDescription, &publishedService)
 	publishedService.AefProfiles = updatedServiceDescription.AefProfiles
 	ps.publishedServices[apfId][pos] = publishedService
 	err = ctx.JSON(http.StatusOK, publishedService)