minor changes, implemented 'doesn't untap' and facedown cards

cockatrice/src/carddatabasemodel.cpp

@@ -65,7 +65,7 @@ private:
 	Qt::SortOrder order;
 public:
 	CardInfoCompare(int _column, Qt::SortOrder _order) : column(_column), order(_order) { }
-	bool operator()(CardInfo *a, CardInfo *b) const
+	inline bool operator()(CardInfo *a, CardInfo *b) const
 	{
 		bool result;
 		switch (column) {

cockatrice/src/carddragitem.cpp

@@ -2,8 +2,8 @@
 #include "cardzone.h"
 #include <QtGui>
 
-CardDragItem::CardDragItem(QGraphicsScene *scene, CardZone *_startZone, QPixmap *_image, int _id, const QPointF &_hotSpot, QGraphicsItem *parent)
-	: QGraphicsItem(parent), image(_image), id(_id), hotSpot(_hotSpot), startZone(_startZone)
+CardDragItem::CardDragItem(QGraphicsScene *scene, CardZone *_startZone, QPixmap *_image, int _id, const QPointF &_hotSpot, bool _faceDown, QGraphicsItem *parent)
+	: QGraphicsItem(parent), image(_image), id(_id), hotSpot(_hotSpot), startZone(_startZone), faceDown(_faceDown)
 {
 	if ((hotSpot.x() < 0) || (hotSpot.y() < 0)) {
 		qDebug(QString("CardDragItem: coordinate overflow: x = %1, y = %2").arg(hotSpot.x()).arg(hotSpot.y()).toLatin1());
@@ -16,6 +16,7 @@ CardDragItem::CardDragItem(QGraphicsScene *scene, CardZone *_startZone, QPixmap
 	setZValue(2000000000);
 	setCacheMode(DeviceCoordinateCache);
 
+	if (!parent)
 		scene->addItem(this);
 }
 
@@ -82,16 +83,13 @@ void CardDragItem::mouseReleaseEvent(QGraphicsSceneMouseEvent *event)
 	}
 
 	if (dropZone) {
-		dropZone->handleDropEvent(id, startZone, (sp - dropZone->scenePos()).toPoint());
+		dropZone->handleDropEvent(id, startZone, (sp - dropZone->scenePos()).toPoint(), faceDown);
 		QList<QGraphicsItem *> childList = childItems();
 		for (int i = 0; i < childList.size(); i++) {
 			CardDragItem *c = qgraphicsitem_cast<CardDragItem *>(childList.at(i));
-			if (!c)
-				QMessageBox::critical(0, "fehler", "null");
-			dropZone->handleDropEvent(c->id, startZone, (sp - dropZone->scenePos() + c->pos()).toPoint());
+			dropZone->handleDropEvent(c->id, startZone, (sp - dropZone->scenePos() + c->pos()).toPoint(), faceDown);
+		}
 	}
-	} else
-		QMessageBox::critical(0, "fehler", "fehler");
 
 	event->accept();
 }

cockatrice/src/carddragitem.h

@@ -12,10 +12,11 @@ private:
 	int id;
 	QPointF hotSpot;
 	CardZone *startZone;
+	bool faceDown;
 public:
 	enum { Type = typeCardDrag };
 	int type() const { return Type; }
-	CardDragItem(QGraphicsScene *scene, CardZone *_startZone, QPixmap *_image, int _id, const QPointF &_hotSpot, QGraphicsItem *parent = 0);
+	CardDragItem(QGraphicsScene *scene, CardZone *_startZone, QPixmap *_image, int _id, const QPointF &_hotSpot, bool _faceDown, QGraphicsItem *parent = 0);
 	~CardDragItem();
 	QRectF boundingRect() const;
 	void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget);

cockatrice/src/carditem.cpp

@@ -8,7 +8,7 @@
 #include "player.h"
 
 CardItem::CardItem(CardDatabase *_db, const QString &_name, int _cardid, QGraphicsItem *parent)
-	: QGraphicsItem(parent), db(_db), name(_name), id(_cardid), tapped(false), attacking(false), facedown(false), counters(0), dragItem(NULL)
+	: QGraphicsItem(parent), db(_db), name(_name), id(_cardid), tapped(false), attacking(false), facedown(false), counters(0), doesntUntap(false), dragItem(NULL)
 {
 	width = CARD_WIDTH;
 	height = CARD_HEIGHT;
@@ -21,6 +21,7 @@ CardItem::CardItem(CardDatabase *_db, const QString &_name, int _cardid, QGraphi
 
 CardItem::~CardItem()
 {
+	deleteDragItem();
 	qDebug(QString("CardItem destructor: %1").arg(name).toLatin1());
 }
 
@@ -80,7 +81,7 @@ void CardItem::setAttacking(bool _attacking)
 	update(boundingRect());
 }
 
-void CardItem::setFacedown(bool _facedown)
+void CardItem::setFaceDown(bool _facedown)
 {
 	facedown = _facedown;
 	update(boundingRect());
@@ -98,6 +99,11 @@ void CardItem::setAnnotation(const QString &_annotation)
 	update(boundingRect());
 }
 
+void CardItem::setDoesntUntap(bool _doesntUntap)
+{
+	doesntUntap = _doesntUntap;
+}
+
 void CardItem::resetState()
 {
 	attacking = false;
@@ -105,12 +111,14 @@ void CardItem::resetState()
 	counters = 0;
 	annotation = QString();
 	setTapped(false);
+	setDoesntUntap(false);
 	update(boundingRect());
 }
 
-CardDragItem *CardItem::createDragItem(CardZone *startZone, int _id, const QPointF &_pos, const QPointF &_scenePos)
+CardDragItem *CardItem::createDragItem(CardZone *startZone, int _id, const QPointF &_pos, const QPointF &_scenePos, bool faceDown)
 {
-	dragItem = new CardDragItem(scene(), startZone, image, _id, _pos);
+	deleteDragItem();
+	dragItem = new CardDragItem(scene(), startZone, image, _id, _pos, faceDown);
 	dragItem->setPos(_scenePos - dragItem->getHotSpot());
 
 	return dragItem;
@@ -140,8 +148,9 @@ void CardItem::mouseMoveEvent(QGraphicsSceneMouseEvent *event)
 {
 	if ((event->screenPos() - event->buttonDownScreenPos(Qt::LeftButton)).manhattanLength() < QApplication::startDragDistance())
 		return;
+	bool faceDown = event->modifiers().testFlag(Qt::ShiftModifier);
 
-	createDragItem((CardZone *) parentItem(), id, event->pos(), event->scenePos());
+	createDragItem((CardZone *) parentItem(), id, event->pos(), event->scenePos(), faceDown);
 	dragItem->grabMouse();
 
 	QList<QGraphicsItem *> sel = scene()->selectedItems();
@@ -149,7 +158,7 @@ void CardItem::mouseMoveEvent(QGraphicsSceneMouseEvent *event)
 		CardItem *c = (CardItem *) sel.at(i);
 		if (c == this)
 			continue;
-		CardDragItem *drag = new CardDragItem(scene(), (CardZone *) parentItem(), c->getImage(), c->getId(), QPointF(), dragItem);
+		CardDragItem *drag = new CardDragItem(scene(), (CardZone *) parentItem(), c->getImage(), c->getId(), QPointF(), false, dragItem);
 		drag->setPos(c->pos() - pos());
 	}
 

cockatrice/src/carditem.h

@@ -41,6 +41,7 @@ private:
 	bool facedown;
 	int counters;
 	QString annotation;
+	bool doesntUntap;
 	CardDragItem *dragItem;
 public:
 	enum { Type = typeCard };
@@ -58,15 +59,17 @@ public:
 	void setTapped(bool _tapped);
 	bool getAttacking() const { return attacking; }
 	void setAttacking(bool _attacking);
-	bool getFacedown() const { return facedown; }
-	void setFacedown(bool _facedown);
+	bool getFaceDown() const { return facedown; }
+	void setFaceDown(bool _facedown);
 	int getCounters() const { return counters; }
 	void setCounters(int _counters);
 	QString getAnnotation() const { return annotation; }
 	void setAnnotation(const QString &_annotation);
+	bool getDoesntUntap() const { return doesntUntap; }
+	void setDoesntUntap(bool _doesntUntap);
 	void resetState();
 
-	CardDragItem *createDragItem(CardZone *startZone, int _id, const QPointF &_pos, const QPointF &_scenePos);
+	CardDragItem *createDragItem(CardZone *startZone, int _id, const QPointF &_pos, const QPointF &_scenePos, bool faceDown);
 	void deleteDragItem();
 protected:
 	void mousePressEvent(QGraphicsSceneMouseEvent *event);

cockatrice/src/cardzone.cpp

@@ -56,14 +56,17 @@ CardItem *CardZone::getCard(int cardId, const QString &cardName)
 
 CardItem *CardZone::takeCard(int position, int cardId, const QString &cardName)
 {
+	if (position >= cards->size())
+		return NULL;
+		
 	CardItem *c = cards->takeAt(position);
 	for (int i = 0; i < views.size(); i++)
 		views[i]->removeCard(position);
 
-	if (c->getId() == -1) {
+//	if (c->getId() == -1) {
 		c->setId(cardId);
 		c->setName(cardName);
-	}
+//	}
 	reorganizeCards();
 	return c;
 }

cockatrice/src/cardzone.h

@@ -21,7 +21,7 @@ protected:
 public:
 	enum { Type = typeZone };
 	int type() const { return Type; }
-	virtual void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint) = 0;
+	virtual void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint, bool faceDown) = 0;
 	CardZone(Player *_player, const QString &_name, QGraphicsItem *parent = 0, bool isView = false);
 	~CardZone();
 	void clearContents();

cockatrice/src/client.cpp

@@ -254,11 +254,11 @@ int Client::drawCards(unsigned int number)
 	return cmd(QString("draw_cards|%1").arg(number));
 }
 
-int Client::moveCard(int cardid, const QString &startzone, const QString &targetzone, int x, int y)
+int Client::moveCard(int cardid, const QString &startzone, const QString &targetzone, int x, int y, bool faceDown)
 {
 	// if startzone is public: cardid is the card's id
 	// else: cardid is the position of the card in the zone (e.g. deck)
-	return cmd(QString("move_card|%1|%2|%3|%4|%5").arg(cardid).arg(startzone).arg(targetzone).arg(x).arg(y));
+	return cmd(QString("move_card|%1|%2|%3|%4|%5|%6").arg(cardid).arg(startzone).arg(targetzone).arg(x).arg(y).arg(faceDown ? 1 : 0));
 }
 
 int Client::createToken(const QString &zone, const QString &name, const QString &powtough, int x, int y)

cockatrice/src/client.h

@@ -70,7 +70,7 @@ public:
 	int shuffle();
 	int rollDice(unsigned int sides);
 	int drawCards(unsigned int number);
-	int moveCard(int cardid, const QString &startzone, const QString &targetzone, int x, int y = 0);
+	int moveCard(int cardid, const QString &startzone, const QString &targetzone, int x, int y = 0, bool faceDown = false);
 	int createToken(const QString &zone, const QString &name, const QString &powtough, int x, int y);
 	int setCardAttr(const QString &zone, int cardid, const QString &aname, const QString &avalue);
 	int readyStart();

cockatrice/src/game.cpp

@@ -66,6 +66,8 @@ Game::Game(CardDatabase *_db, Client *_client, QGraphicsScene *_scene, QMenu *_a
 	connect(aTap, SIGNAL(triggered()), this, SLOT(actTap()));
 	aUntap = new QAction(tr("&Untap"), this);
 	connect(aUntap, SIGNAL(triggered()), this, SLOT(actUntap()));
+	aDoesntUntap = new QAction(tr("Toggle &normal untapping"), this);
+	connect(aDoesntUntap, SIGNAL(triggered()), this, SLOT(actDoesntUntap()));
 	aAddCounter = new QAction(tr("&Add counter"), this);
 	connect(aAddCounter, SIGNAL(triggered()), this, SLOT(actAddCounter()));
 	aRemoveCounter = new QAction(tr("&Remove counter"), this);
@@ -77,6 +79,8 @@ Game::Game(CardDatabase *_db, Client *_client, QGraphicsScene *_scene, QMenu *_a
 
 	cardMenu->addAction(aTap);
 	cardMenu->addAction(aUntap);
+	cardMenu->addAction(aDoesntUntap);
+	cardMenu->addSeparator();
 	cardMenu->addAction(aAddCounter);
 	cardMenu->addAction(aRemoveCounter);
 	cardMenu->addAction(aSetCounters);
@@ -108,6 +112,7 @@ Player *Game::addPlayer(int playerId, const QString &playerName, QPointF base, b
 	connect(newPlayer, SIGNAL(logSetCardCounters(QString, QString, int, int)), this, SIGNAL(logSetCardCounters(QString, QString, int, int)));
 	connect(newPlayer, SIGNAL(logSetTapped(QString, QString, bool)), this, SIGNAL(logSetTapped(QString, QString, bool)));
 	connect(newPlayer, SIGNAL(logSetCounter(QString, QString, int, int)), this, SIGNAL(logSetCounter(QString, QString, int, int)));
+	connect(newPlayer, SIGNAL(logSetDoesntUntap(QString, QString, bool)), this, SIGNAL(logSetDoesntUntap(QString, QString, bool)));
 
 	players << newPlayer;
 	emit playerAdded(newPlayer);
@@ -147,6 +152,7 @@ void Game::restartGameDialog()
 
 void Game::gameEvent(ServerEventData *msg)
 {
+	qDebug(QString("game::gameEvent: public=%1, player=%2, name=%3, type=%4, data=%5").arg(msg->getPublic()).arg(msg->getPlayerId()).arg(msg->getPlayerName()).arg(msg->getEventType()).arg(msg->getEventData().join("/")).toLatin1());
 	if (!msg->getPublic())
 		localPlayer->gameEvent(msg);
 	else {
@@ -231,9 +237,7 @@ void Game::gameEvent(ServerEventData *msg)
 
 void Game::actUntapAll()
 {
-	CardList *const cards = localPlayer->getZones()->findZone("table")->getCards();
-	for (int i = 0; i < cards->size(); i++)
-		client->setCardAttr("table", cards->at(i)->getId(), "tapped", "false");
+	client->setCardAttr("table", -1, "tapped", "false");
 }
 
 void Game::actIncLife()
@@ -308,6 +312,15 @@ void Game::actUntap()
 	}
 }
 
+void Game::actDoesntUntap()
+{
+	QListIterator<QGraphicsItem *> i(scene->selectedItems());
+	while (i.hasNext()) {
+		CardItem *temp = (CardItem *) i.next();
+		client->setCardAttr(qgraphicsitem_cast<CardZone *>(temp->parentItem())->getName(), temp->getId(), "doesnt_untap", QString::number(!temp->getDoesntUntap()));
+	}
+}
+
 void Game::actAddCounter()
 {
 	QListIterator<QGraphicsItem *> i(scene->selectedItems());

cockatrice/src/game.h

@@ -15,7 +15,7 @@ class Game : public QObject {
 	Q_OBJECT
 private:
 	QMenu *actionsMenu, *cardMenu;
-	QAction *aTap, *aUntap, *aAddCounter, *aRemoveCounter, *aSetCounters, *aRearrange,
+	QAction *aTap, *aUntap, *aDoesntUntap, *aAddCounter, *aRemoveCounter, *aSetCounters, *aRearrange,
 		*aUntapAll, *aDecLife, *aIncLife, *aSetLife, *aShuffle, *aDraw, *aDrawCards, *aRollDice, *aCreateToken;
 	DlgStartGame *dlgStartGame;
 	
@@ -40,6 +40,7 @@ private slots:
 	void showCardMenu(QPoint p);
 	void actTap();
 	void actUntap();
+	void actDoesntUntap();
 	void actAddCounter();
 	void actRemoveCounter();
 	void actSetCounters();
@@ -69,6 +70,7 @@ signals:
 	void logSetCardCounters(QString playerName, QString cardName, int value, int oldValue);
 	void logSetTapped(QString playerName, QString cardName, bool tapped);
 	void logSetCounter(QString playerName, QString counterName, int value, int oldValue);
+	void logSetDoesntUntap(QString playerName, QString cardName, bool doesntUntap);
 	void logDumpZone(QString playerName, QString zoneName, QString zoneOwner, int numberCards);
 public slots:
 	void restartGameDialog();

cockatrice/src/gravezone.cpp

@@ -48,7 +48,7 @@ void GraveZone::addCard(CardItem *card, bool reorganize, int x, int y)
 		reorganizeCards();
 }
 
-void GraveZone::handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint)
+void GraveZone::handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint, bool faceDown)
 {
 	player->client->moveCard(cardId, startZone->getName(), getName(), 0, 0);
 }
@@ -79,8 +79,9 @@ void GraveZone::mouseMoveEvent(QGraphicsSceneMouseEvent *event)
 	if (cards->empty())
 		return;
 		
+	bool faceDown = event->modifiers().testFlag(Qt::ShiftModifier);
 	CardItem *card = cards->at(0);
-	CardDragItem *drag = card->createDragItem(this, card->getId(), event->pos(), event->scenePos());
+	CardDragItem *drag = card->createDragItem(this, card->getId(), event->pos(), event->scenePos(), faceDown);
 	drag->grabMouse();
 	setCursor(Qt::OpenHandCursor);
 }

cockatrice/src/gravezone.h

@@ -11,7 +11,7 @@ public:
 	void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget);
 	void addCard(CardItem *card, bool reorganize = true, int x = 0, int y = -1);
 	void reorganizeCards();
-	void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint);
+	void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint, bool faceDown);
 protected:
 	void mousePressEvent(QGraphicsSceneMouseEvent *event);
 	void mouseMoveEvent(QGraphicsSceneMouseEvent *event);

cockatrice/src/handzone.cpp

@@ -66,7 +66,7 @@ void HandZone::addCard(CardItem *card, bool reorganize, int x, int y)
 		reorganizeCards();
 }
 
-void HandZone::handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint)
+void HandZone::handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint, bool faceDown)
 {
 	Q_UNUSED(dropPoint);
 	player->client->moveCard(cardId, startZone->getName(), getName(), cards->size(), 0);

cockatrice/src/handzone.h

@@ -11,7 +11,7 @@ public:
 	void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget);
 	void addCard(CardItem *card, bool reorganize = true, int x = -1, int y = -1);
 	void reorganizeCards();
-	void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint);
+	void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint, bool faceDown);
 };
 
 #endif

cockatrice/src/libraryzone.cpp

@@ -57,7 +57,7 @@ void LibraryZone::addCard(CardItem *card, bool reorganize, int x, int y)
 		reorganizeCards();
 }
 
-void LibraryZone::handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint)
+void LibraryZone::handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint, bool faceDown)
 {
 	player->client->moveCard(cardId, startZone->getName(), getName(), 0, 0);
 }
@@ -88,8 +88,9 @@ void LibraryZone::mouseMoveEvent(QGraphicsSceneMouseEvent *event)
 	if (cards->empty())
 		return;
 		
+	bool faceDown = event->modifiers().testFlag(Qt::ShiftModifier);
 	CardItem *card = cards->at(0);
-	CardDragItem *drag = card->createDragItem(this, 0, event->pos(), event->scenePos());
+	CardDragItem *drag = card->createDragItem(this, 0, event->pos(), event->scenePos(), faceDown);
 	drag->grabMouse();
 	setCursor(Qt::OpenHandCursor);
 }

cockatrice/src/libraryzone.h

@@ -11,7 +11,7 @@ public:
 	void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget);
 	void addCard(CardItem *card, bool reorganize = true, int x = 0, int y = -1);
 	void reorganizeCards();
-	void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint);
+	void handleDropEvent(int cardId, CardZone *startZone, const QPoint &dropPoint, bool faceDown);
 protected:
 	void mousePressEvent(QGraphicsSceneMouseEvent *event);
 	void mouseMoveEvent(QGraphicsSceneMouseEvent *event);

cockatrice/src/messagelogwidget.cpp

@@ -112,6 +112,14 @@ void MessageLogWidget::logSetCounter(QString playerName, QString counterName, in
 	appendAndScroll(tr("%1 sets counter \"%2\" to %3 (%4%5)").arg(playerName).arg(counterName).arg(value).arg(value > oldValue ? "+" : "").arg(value - oldValue));
 }
 
+void MessageLogWidget::logSetDoesntUntap(QString playerName, QString cardName, bool doesntUntap)
+{
+	if (doesntUntap)
+		appendAndScroll(tr("%1 sets %2 to not untap normally.").arg(playerName).arg(cardName));
+	else
+		appendAndScroll(tr("%1 sets %2 to untap normally.").arg(playerName).arg(cardName));
+}
+
 void MessageLogWidget::logDumpZone(QString playerName, QString zoneName, QString zoneOwner, int numberCards)
 {
 	if (numberCards)
@@ -137,6 +145,7 @@ void MessageLogWidget::connectToGame(Game *game)
 	connect(game, SIGNAL(logSetCardCounters(QString, QString, int, int)), this, SLOT(logSetCardCounters(QString, QString, int, int)));
 	connect(game, SIGNAL(logSetTapped(QString, QString, bool)), this, SLOT(logSetTapped(QString, QString, bool)));
 	connect(game, SIGNAL(logSetCounter(QString, QString, int, int)), this, SLOT(logSetCounter(QString, QString, int, int)));
+	connect(game, SIGNAL(logSetDoesntUntap(QString, QString, bool)), this, SLOT(logSetDoesntUntap(QString, QString, bool)));
 	connect(game, SIGNAL(logDumpZone(QString, QString, QString, int)), this, SLOT(logDumpZone(QString, QString, QString, int)));
 }
 

cockatrice/src/messagelogwidget.h

@@ -28,6 +28,7 @@ private slots:
 	void logSetCardCounters(QString playerName, QString cardName, int value, int oldValue);
 	void logSetTapped(QString playerName, QString cardName, bool tapped);
 	void logSetCounter(QString playerName, QString counterName, int value, int oldValue);
+	void logSetDoesntUntap(QString playerName, QString cardName, bool doesntUntap);
 	void logDumpZone(QString playerName, QString zoneName, QString zoneOwner, int numberCards);
 private:
 	void appendAndScroll(const QString &s);

cockatrice/src/player.cpp

@@ -128,6 +128,31 @@ void Player::addCounter(Counter *c)
 	counters << c;
 }
 
+void Player::setCardAttrHelper(CardItem *card, const QString &aname, const QString &avalue, bool allCards)
+{
+	if (aname == "tapped") {
+		bool tapped = avalue == "1";
+		if (!(!tapped && card->getDoesntUntap() && allCards)) {
+			emit logSetTapped(name, card->getName(), tapped);
+			card->setTapped(tapped);
+		}
+	} else if (aname == "attacking")
+		card->setAttacking(avalue == "1");
+	else if (aname == "facedown")
+		card->setFaceDown(avalue == "1");
+	else if (aname == "counters") {
+		int value = avalue.toInt();
+		emit logSetCardCounters(name, card->getName(), value, card->getCounters());
+		card->setCounters(value);
+	} else if (aname == "annotation")
+		card->setAnnotation(avalue);
+	else if (aname == "doesnt_untap") {
+		bool value = (avalue == "1");
+		emit logSetDoesntUntap(name, card->getName(), value);
+		card->setDoesntUntap(value);
+	}
+}
+
 void Player::gameEvent(ServerEventData *event)
 {
 	QStringList data = event->getEventData();
@@ -171,7 +196,7 @@ void Player::gameEvent(ServerEventData *event)
 			break;
 		}
 		case eventMoveCard: {
-			if (data.size() != 7) {
+			if (data.size() != 8) {
 				qDebug("error");
 				// XXX
 			}
@@ -186,14 +211,17 @@ void Player::gameEvent(ServerEventData *event)
 				qDebug(QString("target zone invalid: %1").arg(data[4]).toLatin1());
 			int x = data[5].toInt();
 			int y = data[6].toInt();
+			bool facedown = data[7].toInt();
 			// XXX Mehr Fehlerbehandlung
 
 			CardItem *card = startZone->takeCard(position, cardId, cardName);
 			if (!card) // XXX
-				qDebug("null");
+				qDebug("moveCard: card not found");
 
 			card->deleteDragItem();
 			
+			card->setFaceDown(facedown);
+
 			// The log event has to be sent before the card is added to the target zone
 			// because the addCard function can modify the card object.
 			emit logMoveCard(name, card->getName(), startZone->getName(), targetZone->getName());
@@ -227,25 +255,18 @@ void Player::gameEvent(ServerEventData *event)
 			}
 			CardZone *zone = zones.findZone(data[0]);
 			int cardId = data[1].toInt();
-			CardItem *card = zone->getCard(cardId, "");
 			QString aname = data[2];
 			QString avalue = data[3];
 			// XXX Fehlerbehandlung
 
-			if (aname == "tapped") {
-				bool tapped = avalue == "1";
-				emit logSetTapped(name, card->getName(), tapped);
-				card->setTapped(tapped);
-			} else if (aname == "attacking")
-				card->setAttacking(avalue == "1");
-			else if (aname == "facedown")
-				card->setFacedown(avalue == "1");
-			else if (aname == "counters") {
-				int value = avalue.toInt();
-				emit logSetCardCounters(name, card->getName(), value, card->getCounters());
-				card->setCounters(value);
-			} else if (aname == "annotation")
-				card->setAnnotation(avalue);
+			if (cardId == -1) {
+				CardList *const cards = zone->getCards();
+				for (int i = 0; i < cards->size(); i++)
+					setCardAttrHelper(cards->at(i), aname, avalue, true);
+			} else {
+				CardItem *card = zone->getCard(cardId, "");
+				setCardAttrHelper(card, aname, avalue, false);
+			}
 			break;
 		}
 		case eventSetCounter: {