mirror of
https://gitlab.com/mbugroup/lti-api.git
synced 2026-05-23 06:45:43 +00:00
Revert "Merge branch 'fix/implement-fifo-v2' into 'dev/fifo-v2'"
This reverts merge request !340
This commit is contained in:
@@ -41,151 +41,6 @@ func normalizeProductFlags(raw []string) ([]string, error) {
|
||||
return utils.FlagTypesToStrings(normalized), nil
|
||||
}
|
||||
|
||||
func productMainFlagOptionsString() []string {
|
||||
mainFlags := utils.ProductMainFlags()
|
||||
result := make([]string, len(mainFlags))
|
||||
for i, flag := range mainFlags {
|
||||
result[i] = string(flag)
|
||||
}
|
||||
return result
|
||||
}
|
||||
|
||||
func productSubFlagOptionsString(flag utils.FlagType) []string {
|
||||
subFlagsByFlag := utils.ProductSubFlagsByFlag()
|
||||
subFlags := subFlagsByFlag[flag]
|
||||
result := make([]string, len(subFlags))
|
||||
for i, subFlag := range subFlags {
|
||||
result[i] = string(subFlag)
|
||||
}
|
||||
return result
|
||||
}
|
||||
|
||||
func normalizeStructuredSubFlagsInput(subFlagRaw *string, subFlagsRaw []string, hasSubFlagsField bool) ([]utils.FlagType, error) {
|
||||
values := make([]string, 0, len(subFlagsRaw)+1)
|
||||
|
||||
if subFlagRaw != nil {
|
||||
single := strings.TrimSpace(*subFlagRaw)
|
||||
if single == "" {
|
||||
return nil, fiber.NewError(fiber.StatusBadRequest, "sub_flag cannot be empty")
|
||||
}
|
||||
values = append(values, single)
|
||||
}
|
||||
|
||||
if hasSubFlagsField {
|
||||
for _, raw := range subFlagsRaw {
|
||||
item := strings.TrimSpace(raw)
|
||||
if item == "" {
|
||||
return nil, fiber.NewError(fiber.StatusBadRequest, "sub_flags cannot contain empty value")
|
||||
}
|
||||
values = append(values, item)
|
||||
}
|
||||
}
|
||||
|
||||
if len(values) == 0 {
|
||||
return nil, nil
|
||||
}
|
||||
|
||||
return utils.NormalizeFlagTypes(values), nil
|
||||
}
|
||||
|
||||
func resolveProductFlagsFromFlagInput(flagRaw *string, subFlagRaw *string, subFlagsRaw []string, hasSubFlagsField bool) ([]string, bool, error) {
|
||||
if flagRaw == nil && subFlagRaw == nil && !hasSubFlagsField {
|
||||
return nil, false, nil
|
||||
}
|
||||
|
||||
if flagRaw == nil && (subFlagRaw != nil || hasSubFlagsField) {
|
||||
return nil, false, fiber.NewError(fiber.StatusBadRequest, "flag is required when sub_flag/sub_flags is provided")
|
||||
}
|
||||
|
||||
flagText := strings.TrimSpace(*flagRaw)
|
||||
if flagText == "" {
|
||||
return nil, false, fiber.NewError(fiber.StatusBadRequest, "flag cannot be empty")
|
||||
}
|
||||
|
||||
flag := utils.CanonicalFlagType(flagText)
|
||||
if !utils.IsProductMainFlag(flag) {
|
||||
return nil, false, fiber.NewError(
|
||||
fiber.StatusBadRequest,
|
||||
fmt.Sprintf("Invalid product flag: %s. Allowed flags: %s", flagText, strings.Join(productMainFlagOptionsString(), ", ")),
|
||||
)
|
||||
}
|
||||
|
||||
out := []string{string(flag)}
|
||||
|
||||
normalizedSubFlags, err := normalizeStructuredSubFlagsInput(subFlagRaw, subFlagsRaw, hasSubFlagsField)
|
||||
if err != nil {
|
||||
return nil, false, err
|
||||
}
|
||||
|
||||
if len(normalizedSubFlags) == 0 {
|
||||
if !utils.ProductFlagAllowWithoutSubFlag(flag) {
|
||||
return nil, false, fiber.NewError(
|
||||
fiber.StatusBadRequest,
|
||||
fmt.Sprintf("sub_flag/sub_flags is required for flag %s", string(flag)),
|
||||
)
|
||||
}
|
||||
return out, true, nil
|
||||
}
|
||||
|
||||
invalidSubFlags := make([]string, 0)
|
||||
for _, subFlag := range normalizedSubFlags {
|
||||
if !utils.IsValidProductSubFlag(flag, subFlag) {
|
||||
invalidSubFlags = append(invalidSubFlags, string(subFlag))
|
||||
}
|
||||
}
|
||||
if len(invalidSubFlags) > 0 {
|
||||
return nil, false, fiber.NewError(
|
||||
fiber.StatusBadRequest,
|
||||
fmt.Sprintf("Invalid sub_flags %s for flag %s. Allowed sub_flags: %s", strings.Join(invalidSubFlags, ", "), string(flag), strings.Join(productSubFlagOptionsString(flag), ", ")),
|
||||
)
|
||||
}
|
||||
|
||||
out = append(out, utils.FlagTypesToStrings(normalizedSubFlags)...)
|
||||
return out, true, nil
|
||||
}
|
||||
|
||||
func resolveCreateProductFlags(req *validation.Create) ([]string, error) {
|
||||
hasStructuredInput := req.Flag != nil || req.SubFlag != nil || req.SubFlags != nil
|
||||
if len(req.Flags) > 0 && hasStructuredInput {
|
||||
return nil, fiber.NewError(fiber.StatusBadRequest, "Use either flags or flag/sub_flag/sub_flags, not both")
|
||||
}
|
||||
|
||||
if len(req.Flags) > 0 {
|
||||
return normalizeProductFlags(req.Flags)
|
||||
}
|
||||
|
||||
flags, _, err := resolveProductFlagsFromFlagInput(req.Flag, req.SubFlag, req.SubFlags, req.SubFlags != nil)
|
||||
return flags, err
|
||||
}
|
||||
|
||||
func resolveUpdateProductFlags(req *validation.Update) (bool, []string, error) {
|
||||
hasStructuredInput := req.Flag != nil || req.SubFlag != nil || req.SubFlags != nil
|
||||
|
||||
if req.Flags != nil {
|
||||
if hasStructuredInput {
|
||||
if len(*req.Flags) > 0 {
|
||||
return false, nil, fiber.NewError(fiber.StatusBadRequest, "Use either flags or flag/sub_flag/sub_flags, not both")
|
||||
}
|
||||
} else {
|
||||
flags, err := normalizeProductFlags(*req.Flags)
|
||||
if err != nil {
|
||||
return false, nil, err
|
||||
}
|
||||
return true, flags, nil
|
||||
}
|
||||
}
|
||||
|
||||
subFlagsRaw := make([]string, 0)
|
||||
if req.SubFlags != nil {
|
||||
subFlagsRaw = *req.SubFlags
|
||||
}
|
||||
flags, provided, err := resolveProductFlagsFromFlagInput(req.Flag, req.SubFlag, subFlagsRaw, req.SubFlags != nil)
|
||||
if err != nil {
|
||||
return false, nil, err
|
||||
}
|
||||
return provided, flags, nil
|
||||
}
|
||||
|
||||
func NewProductService(repo repository.ProductRepository, validate *validator.Validate) ProductService {
|
||||
return &productService{
|
||||
Log: utils.Log,
|
||||
@@ -322,7 +177,7 @@ func (s *productService) CreateOne(c *fiber.Ctx, req *validation.Create) (*entit
|
||||
}
|
||||
}
|
||||
|
||||
productFlags, flagErr := resolveCreateProductFlags(req)
|
||||
productFlags, flagErr := normalizeProductFlags(req.Flags)
|
||||
if flagErr != nil {
|
||||
return nil, flagErr
|
||||
}
|
||||
@@ -482,10 +337,13 @@ func (s productService) UpdateOne(c *fiber.Ctx, req *validation.Update, id uint)
|
||||
flagUpdate bool
|
||||
flagValues []string
|
||||
)
|
||||
var flagErr error
|
||||
flagUpdate, flagValues, flagErr = resolveUpdateProductFlags(req)
|
||||
if flagErr != nil {
|
||||
return nil, flagErr
|
||||
if req.Flags != nil {
|
||||
flagUpdate = true
|
||||
var flagErr error
|
||||
flagValues, flagErr = normalizeProductFlags(*req.Flags)
|
||||
if flagErr != nil {
|
||||
return nil, flagErr
|
||||
}
|
||||
}
|
||||
|
||||
if len(updateBody) == 0 && !supplierUpdate && !flagUpdate {
|
||||
|
||||
Reference in New Issue
Block a user