Alexander Schremmer committed f2d56e8

OMG, fix JackOfAllTrades, it was so broken.

Comments (0)

Files changed (1)


             "You may trash a card from your hand that is not a Treasure.")
     def activate_action(self, game, player):
-        silver_cards =["Silver"]
-        if silver_cards:
-            player.discard_pile.append(silver_cards.pop())
-            new_card = silver_cards.pop()
+        with fetch_card_from_supply(game, Silver) as new_card:
+            player.discard_pile.append(new_card)
             for info_player in game.following_participants(player):
                 yield InfoRequest(game, info_player,
                         _("%s gains:", (, )), [new_card])
-            for val in game.check_empty_pile("Silver"):
-                yield val
         drawn, player.hand = player.hand[-1:], player.hand[:-1]
-        if (yield YesNoQuestion(game, player, _("Do you want to keep this %s?"%drawn[0].name, {}))):
+        if (yield YesNoQuestion(game, player, _("Do you want to keep the card '%s' on your hand?" (drawn[0].name,) ))):
+        while len(player.hand) < 5:
+            if player.draw_cards(1) is None:
+                break
         #FIXME only cards that are no treasure
         cards = yield SelectHandCards(game, player,
-                    count_lower=0, count_upper=1,
+                    count_lower=0, count_upper=1, not_selectable=[c for c in player.hand if isinstance(c, TreasureCard)],
                     msg=_("Select a card you want to trash."))
         if cards:
             card = cards[0]
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.