chore: remote IsValid method from price

Author: ironman0x7b2

types/v1/price.go

@@ -181,11 +181,6 @@ func (p Price) Validate() error {
 	return nil
 }
 
-// IsValid returns true if the Price passes validation.
-func (p Price) IsValid() bool {
-	return p.Validate() == nil
-}
-
 // negative returns the negated Price (both base and quote).
 func (p Price) negative() Price {
 	return Price{
@@ -356,11 +351,6 @@ func (p Prices) Validate() error {
 	return nil
 }
 
-// IsValid checks if the Prices slice is valid.
-func (p Prices) IsValid() bool {
-	return p.Validate() == nil
-}
-
 // Len returns the number of Prices.
 func (p Prices) Len() int {
 	return len(p)

x/node/types/v3/msg.go

@@ -51,12 +51,14 @@ func (m *MsgRegisterNodeRequest) ValidateBasic() error {
 		return types.NewErrorInvalidMessage(fmt.Errorf("invalid from: %w", err))
 	}
 
-	if prices := m.GetGigabytePrices(); !prices.IsValid() {
-		return types.NewErrorInvalidMessage("gigabyte_prices must be valid")
+	gigabytePrices := m.GetGigabytePrices()
+	if err := gigabytePrices.Validate(); err != nil {
+		return types.NewErrorInvalidMessage(fmt.Errorf("invalid gigabyte_prices: %w", err))
 	}
 
-	if prices := m.GetHourlyPrices(); !prices.IsValid() {
-		return types.NewErrorInvalidMessage("hourly_prices must be valid")
+	hourlyPrices := m.GetHourlyPrices()
+	if err := hourlyPrices.Validate(); err != nil {
+		return types.NewErrorInvalidMessage(fmt.Errorf("invalid hourly_prices: %w", err))
 	}
 
 	if len(m.RemoteAddrs) == 0 {
@@ -110,12 +112,14 @@ func (m *MsgUpdateNodeDetailsRequest) ValidateBasic() error {
 		return types.NewErrorInvalidMessage(fmt.Errorf("invalid from: %w", err))
 	}
 
-	if prices := m.GetGigabytePrices(); !prices.IsValid() {
-		return types.NewErrorInvalidMessage("gigabyte_prices must be valid")
+	gigabytePrices := m.GetGigabytePrices()
+	if err := gigabytePrices.Validate(); err != nil {
+		return types.NewErrorInvalidMessage(fmt.Errorf("invalid gigabyte_prices: %w", err))
 	}
 
-	if prices := m.GetHourlyPrices(); !prices.IsValid() {
-		return types.NewErrorInvalidMessage("hourly_prices must be valid")
+	hourlyPrices := m.GetHourlyPrices()
+	if err := hourlyPrices.Validate(); err != nil {
+		return types.NewErrorInvalidMessage(fmt.Errorf("invalid hourly_prices: %w", err))
 	}
 
 	if len(m.RemoteAddrs) > 0 {

x/node/types/v3/node.go

@@ -30,12 +30,14 @@ func (m *Node) Validate() error {
 		return fmt.Errorf("invalid address: %w", err)
 	}
 
-	if prices := m.GetGigabytePrices(); !prices.IsValid() {
-		return errors.New("gigabyte_prices must be valid")
+	gigabytePrices := m.GetGigabytePrices()
+	if err := gigabytePrices.Validate(); err != nil {
+		return fmt.Errorf("invalid gigabyte_prices: %w", err)
 	}
 
-	if prices := m.GetHourlyPrices(); !prices.IsValid() {
-		return errors.New("hourly_prices must be valid")
+	hourlyPrices := m.GetHourlyPrices()
+	if err := hourlyPrices.Validate(); err != nil {
+		return fmt.Errorf("invalid hourly_prices: %w", err)
 	}
 
 	if err := validateRemoteAddrs(m.RemoteAddrs); err != nil {

x/plan/types/v3/msg.go

@@ -68,8 +68,9 @@ func (m *MsgCreatePlanRequest) ValidateBasic() error {
 		return types.NewErrorInvalidMessage("duration cannot be negative")
 	}
 
-	if prices := m.GetPrices(); !prices.IsValid() {
-		return types.NewErrorInvalidMessage("prices must be valid")
+	prices := m.GetPrices()
+	if err := prices.Validate(); err != nil {
+		return types.NewErrorInvalidMessage(fmt.Errorf("invalid prices: %w", err))
 	}
 
 	return nil

x/plan/types/v3/plan.go

@@ -62,8 +62,9 @@ func (m *Plan) Validate() error {
 		return errors.New("duration cannot be negative")
 	}
 
-	if prices := m.GetPrices(); !prices.IsValid() {
-		return errors.New("prices must be valid")
+	prices := m.GetPrices()
+	if err := prices.Validate(); err != nil {
+		return fmt.Errorf("invalid prices: %w", err)
 	}
 
 	if !m.Status.IsOneOf(v1base.StatusActive, v1base.StatusInactive) {