fix: plugin exposed flags default value (#6332)

* plugin creation fix

* isExposed

* boolean pointer

* migrate

Author: Shivam-nagar23

pkg/plugin/GlobalPluginService.go

@@ -1941,7 +1941,7 @@ func (impl *GlobalPluginServiceImpl) MigratePluginDataToParentPluginMetadata(plu
 				continue
 			}
 			parentMetadata := repository.NewPluginParentMetadata()
-			parentMetadata.SetParentPluginMetadata(pluginMetadata).CreateAuditLog(bean.SystemUserId)
+			parentMetadata.SetParentPluginMetadata(pluginMetadata).CreateAuditLog(bean.SystemUserId).WithIsExposed(true)
 			parentMetadata.Identifier = identifier
 			parentMetadata, err = impl.globalPluginRepository.SavePluginParentMetadata(tx, parentMetadata)
 			if err != nil {

pkg/plugin/adaptor/adaptor.go

@@ -8,12 +8,12 @@ import (
 
 func GetPluginParentMetadataDbObject(pluginDto *pluginBean.PluginParentMetadataDto, userId int32) *repository.PluginParentMetadata {
 	return repository.NewPluginParentMetadata().CreateAuditLog(userId).
-		WithBasicMetadata(pluginDto.Name, pluginDto.PluginIdentifier, pluginDto.Description, pluginDto.Icon, repository.PLUGIN_TYPE_SHARED)
+		WithBasicMetadata(pluginDto.Name, pluginDto.PluginIdentifier, pluginDto.Description, pluginDto.Icon, repository.PLUGIN_TYPE_SHARED, pluginDto.IsExposed)
 }
 
 func GetPluginVersionMetadataDbObject(pluginDto *pluginBean.PluginParentMetadataDto, userId int32) *repository.PluginMetadata {
 	versionDto := pluginDto.Versions.DetailedPluginVersionData[0]
-	return repository.NewPluginVersionMetadata().CreateAuditLog(userId).WithBasicMetadata(pluginDto.Name, versionDto.Description, versionDto.Version, versionDto.DocLink)
+	return repository.NewPluginVersionMetadata().CreateAuditLog(userId).WithBasicMetadata(pluginDto.Name, versionDto.Description, versionDto.Version, versionDto.DocLink, versionDto.IsExposed)
 }
 
 func GetPluginStepDbObject(pluginStepDto *pluginBean.PluginStepsDto, pluginVersionMetadataId int, userId int32) *repository.PluginStep {

pkg/plugin/bean/bean.go

@@ -54,6 +54,7 @@ type PluginMetadataDto struct {
 	PluginStage       string            `json:"pluginStage,omitempty"`
 	PluginSteps       []*PluginStepsDto `json:"pluginSteps,omitempty"`
 	AreNewTagsPresent bool              `json:"areNewTagsPresent,omitempty"`
+	IsExposed         *bool             `json:"-"`
 }
 
 type PluginMinDto struct {
@@ -127,6 +128,7 @@ type PluginParentMetadataDto struct {
 	Icon             string          `json:"icon,omitempty"`
 	PluginStageType  string          `json:"pluginStageType,omitempty"`
 	Versions         *PluginVersions `json:"pluginVersions"`
+	IsExposed        *bool           `json:"-"`
 }
 
 func NewPluginParentMetadataDto() *PluginParentMetadataDto {

pkg/plugin/repository/GlobalPluginRepository.go

@@ -129,14 +129,24 @@ func (r *PluginParentMetadata) CreateAuditLog(userId int32) *PluginParentMetadat
 	r.UpdatedOn = time.Now()
 	return r
 }
+func (r *PluginParentMetadata) WithIsExposed(isExposed bool) *PluginParentMetadata {
+	r.IsExposed = isExposed
+	return r
+}
 
-func (r *PluginParentMetadata) WithBasicMetadata(name, identifier, description, icon string, pluginType PluginType) *PluginParentMetadata {
+func (r *PluginParentMetadata) WithBasicMetadata(name, identifier, description, icon string, pluginType PluginType, isExposed *bool) *PluginParentMetadata {
 	r.Name = name
 	r.Identifier = identifier
 	r.Description = description
 	r.Icon = icon
 	r.Type = pluginType
 	r.Deleted = false
+	if isExposed != nil {
+		r.IsExposed = *isExposed
+	} else {
+		//default set true
+		r.IsExposed = true
+	}
 	return r
 }
 
@@ -189,13 +199,19 @@ func (r *PluginMetadata) CreateAuditLog(userId int32) *PluginMetadata {
 	return r
 }
 
-func (r *PluginMetadata) WithBasicMetadata(name, description, pluginVersion, docLink string) *PluginMetadata {
+func (r *PluginMetadata) WithBasicMetadata(name, description, pluginVersion, docLink string, isExposed *bool) *PluginMetadata {
 	r.Name = name
 	r.PluginVersion = pluginVersion
 	r.Description = description
 	r.DocLink = docLink
 	r.Deleted = false
 	r.IsDeprecated = false
+	if isExposed != nil {
+		r.IsExposed = *isExposed
+	} else {
+		//default value is true
+		r.IsExposed = true
+	}
 	return r
 }