fix: not checking if player moves with cards onto market field

Author: maxblan

src/plugin/action/card.rs

@@ -75,8 +75,8 @@ impl Card {
                         "You can only play this card if you are behind the other player",
                     ));
                 }
-                self.move_to_field(current, state, other.position + 1, remaining_cards)?;
                 // saturating add is here unnecessary because the board is finite and never larger than usize::MAX
+                self.move_to_field(current, state, other.position + 1, remaining_cards)?;
             }
             Card::EatSalad => current.eat_salad(state)?,
             Card::SwapCarrots => {

src/plugin/rules_engine.rs

@@ -113,8 +113,8 @@ impl RulesEngine {
             Field::Salad => Err(HUIError::new_err("Field is occupied by opponent")),
             Field::Hare if !cards.is_empty() => Ok(()),
             Field::Hare => Err(HUIError::new_err("No card to play")),
-            Field::Market if player.carrots >= 10 => Ok(()),
-            Field::Market => Err(HUIError::new_err("Not enough carrots")),
+            Field::Market if player.carrots >= 10 && !cards.is_empty() => Ok(()),
+            Field::Market => Err(HUIError::new_err("Not enough carrots or no card to play")),
             Field::Goal if player.carrots <= 10 && player.salads == 0 => Ok(()),
             Field::Goal => Err(HUIError::new_err("Too many carrots or salads")),
             _ => Ok(()),