优化发布等各种状态

backend/internal/modules/listing/service.go

@@ -2,6 +2,7 @@ package listing
 
 import (
 	"errors"
+	"strconv"
 	"strings"
 )
 
@@ -9,14 +10,27 @@ var (
 	ErrDependencyUnavailable = errors.New("dependency unavailable")
 	ErrInvalidInput          = errors.New("invalid listing input")
 	ErrListingLocked         = errors.New("listing locked")
+	ErrMissingTitle          = errors.New("missing listing title")
+	ErrMissingServerRegion   = errors.New("missing server region")
+	ErrInvalidPrice          = errors.New("invalid listing price")
+	ErrInvalidDeposit        = errors.New("invalid listing deposit")
+	ErrInvalidHafCoin        = errors.New("invalid haf coin amount")
+	ErrMissingScreenshot     = errors.New("missing screenshot")
 )
 
 type Service struct {
-	repo *Repository
+	repo   *Repository
+	config ConfigReader
 }
 
-func NewService(repo *Repository) *Service {
-	return &Service{repo: repo}
+type ConfigReader interface {
+	FindValue(key string) (string, error)
+}
+
+const reviewRequiredConfigKey = "listing.review_required"
+
+func NewService(repo *Repository, config ConfigReader) *Service {
+	return &Service{repo: repo, config: config}
 }
 
 func (s *Service) Create(ownerID uint64, req CreateRequest) (*ListingDTO, error) {
@@ -26,7 +40,11 @@ func (s *Service) Create(ownerID uint64, req CreateRequest) (*ListingDTO, error)
 	if err := validateRequest(req); err != nil {
 		return nil, err
 	}
-	return s.repo.Create(ownerID, req)
+	reviewRequired, err := s.reviewRequired()
+	if err != nil {
+		return nil, err
+	}
+	return s.repo.Create(ownerID, req, reviewRequired)
 }
 
 func (s *Service) Update(ownerID uint64, id uint64, req UpdateRequest) (*ListingDTO, error) {
@@ -36,14 +54,22 @@ func (s *Service) Update(ownerID uint64, id uint64, req UpdateRequest) (*Listing
 	if err := validateRequest(req); err != nil {
 		return nil, err
 	}
-	return s.repo.Update(ownerID, id, req)
+	reviewRequired, err := s.reviewRequired()
+	if err != nil {
+		return nil, err
+	}
+	return s.repo.Update(ownerID, id, req, reviewRequired)
 }
 
 func (s *Service) SubmitReview(ownerID uint64, id uint64) (*ListingDTO, error) {
 	if s.repo == nil {
 		return nil, ErrDependencyUnavailable
 	}
-	return s.repo.SubmitReview(ownerID, id)
+	reviewRequired, err := s.reviewRequired()
+	if err != nil {
+		return nil, err
+	}
+	return s.repo.SubmitReview(ownerID, id, reviewRequired)
 }
 
 func (s *Service) ListPendingReview() ([]ListingDTO, error) {
@@ -140,17 +166,23 @@ func (s *Service) FindMine(ownerID uint64, id uint64) (*ListingDTO, error) {
 }
 
 func validateRequest(req CreateRequest) error {
-	if req.Title == "" || req.ServerRegion == "" {
-		return ErrInvalidInput
+	if strings.TrimSpace(req.Title) == "" {
+		return ErrMissingTitle
 	}
-	if req.PriceHourly <= 0 || req.DepositAmount < 0 {
-		return ErrInvalidInput
+	if strings.TrimSpace(req.ServerRegion) == "" {
+		return ErrMissingServerRegion
+	}
+	if req.PriceHourly <= 0 {
+		return ErrInvalidPrice
+	}
+	if req.DepositAmount < 0 {
+		return ErrInvalidDeposit
 	}
 	if req.HafCoinAmount < 0 {
-		return ErrInvalidInput
+		return ErrInvalidHafCoin
 	}
 	if !hasScreenshotURL(req.ScreenshotURLS) {
-		return ErrInvalidInput
+		return ErrMissingScreenshot
 	}
 	return nil
 }
@@ -163,3 +195,18 @@ func hasScreenshotURL(urls []string) bool {
 	}
 	return false
 }
+
+func (s *Service) reviewRequired() (bool, error) {
+	if s.config == nil {
+		return false, nil
+	}
+	value, err := s.config.FindValue(reviewRequiredConfigKey)
+	if err != nil {
+		return false, err
+	}
+	required, err := strconv.ParseBool(strings.TrimSpace(value))
+	if err != nil {
+		return false, nil
+	}
+	return required, nil
+}