jeirmeister/Cockatrice
1
0
Fork 0
mirror of https://github.com/Cockatrice/Cockatrice.git synced 2026-06-10 16:24:45 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Refactor: rename and consolidate getSpectator (#6067)

Browse source
This commit is contained in:
RickyRister 2025-08-15 02:13:53 -03:00 • committed by GitHub
parent 03bebbe4c2
commit 46d65f0b7e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 6 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

5
cockatrice/src/client/tabs/tab_game.cpp
View file

@ -415,11 +415,6 @@ void TabGame::adminLockChanged(bool lock)
sayEdit->setVisible(v);
}
bool TabGame::isSpectator()
{
return spectator;
}
void TabGame::actGameInfo()
{
DlgCreateGame dlg(gameInfo, roomGameTypes, this);

5
cockatrice/src/client/tabs/tab_game.h
View file

@ -235,15 +235,14 @@ public:
return gameInfo.game_id();
}
QString getTabText() const override;
bool getSpectator() const
bool isSpectator() const
{
return spectator;
}
bool getSpectatorsSeeEverything() const
bool isSpectatorsOmniscient() const
{
return gameInfo.spectators_omniscient();
}
bool isSpectator();
Player *getActiveLocalPlayer() const;
AbstractClient *getClientForPlayer(int playerId) const;

4
cockatrice/src/game/board/card_item.cpp
View file

@ -259,7 +259,7 @@ void CardItem::deleteDragItem()
void CardItem::drawArrow(const QColor &arrowColor)
{
if (static_cast<TabGame *>(owner->parent())->getSpectator())
if (static_cast<TabGame *>(owner->parent())->isSpectator())
return;
Player *arrowOwner = static_cast<TabGame *>(owner->parent())->getActiveLocalPlayer();
@ -282,7 +282,7 @@ void CardItem::drawArrow(const QColor &arrowColor)
void CardItem::drawAttachArrow()
{
if (static_cast<TabGame *>(owner->parent())->getSpectator())
if (static_cast<TabGame *>(owner->parent())->isSpectator())
return;
auto *arrow = new ArrowAttachItem(this);

5
cockatrice/src/game/player/player.cpp
View file

@ -150,8 +150,7 @@ Player::Player(const ServerInfo_User &info, int _id, bool _local, bool _judge, T
stack = addZone(new StackZone(this, (int)table->boundingRect().height(), this));
hand = addZone(new HandZone(this,
_local || _judge || (_parent->getSpectator() && _parent->getSpectatorsSeeEverything()),
hand = addZone(new HandZone(this, _local || _judge || (_parent->isSpectator() && _parent->isSpectatorsOmniscient()),
(int)table->boundingRect().height(), this));
connect(hand, &HandZone::cardCountChanged, handCounter, &HandCounter::updateNumber);
connect(handCounter, &HandCounter::showContextMenu, hand, &HandZone::showContextMenu);
@ -2782,7 +2781,7 @@ void Player::processPlayerInfo(const ServerInfo_Player &info)
switch (zoneInfo.type()) {
case ServerInfo_Zone::PrivateZone:
contentsKnown = local || judge || (game->getSpectator() && game->getSpectatorsSeeEverything());
contentsKnown = local || judge || (game->isSpectator() && game->isSpectatorsOmniscient());
break;
case ServerInfo_Zone::PublicZone: