Another style lint on a sunday night

Took 5 minutes
This commit is contained in:
Lukas Brübach 2025-11-09 21:53:07 +01:00
parent 599d20c61d
commit 348b0479be

View file

@ -106,43 +106,35 @@ QVariant DeckListModel::data(const QModelIndex &index, int role) const
case Qt::DisplayRole: case Qt::DisplayRole:
case Qt::EditRole: { case Qt::EditRole: {
switch (index.column()) { switch (index.column()) {
case CARD_AMOUNT: { case CARD_AMOUNT:
return group->recursiveCount(true); return group->recursiveCount(true);
} case CARD_NAME:
case CARD_NAME: { if (role == Qt::DisplayRole) {
if (role == Qt::DisplayRole)
return group->getVisibleName(); return group->getVisibleName();
}
return group->getName(); return group->getName();
} case CARD_SET:
case CARD_SET: {
return group->getCardSetShortName(); return group->getCardSetShortName();
} case CARD_COLLECTOR_NUMBER:
case CARD_COLLECTOR_NUMBER: {
return group->getCardCollectorNumber(); return group->getCardCollectorNumber();
} case CARD_PROVIDER_ID:
case CARD_PROVIDER_ID: {
return group->getCardProviderId(); return group->getCardProviderId();
}
default: default:
return {}; return {};
} }
} }
case DeckRoles::IsCardRole: { case DeckRoles::IsCardRole:
return false; return false;
}
case DeckRoles::DepthRole: { case DeckRoles::DepthRole:
return group->depth(); return group->depth();
}
// legality does not apply to group nodes // legality does not apply to group nodes
case DeckRoles::IsLegalRole: { case DeckRoles::IsLegalRole:
return true; return true;
}
default: { default:
return {}; return {};
}
} }
} }
@ -151,24 +143,18 @@ QVariant DeckListModel::data(const QModelIndex &index, int role) const
case Qt::DisplayRole: case Qt::DisplayRole:
case Qt::EditRole: case Qt::EditRole:
switch (index.column()) { switch (index.column()) {
case CARD_AMOUNT: { case CARD_AMOUNT:
return card->getNumber(); return card->getNumber();
} case CARD_NAME:
case CARD_NAME: {
return card->getName(); return card->getName();
} case CARD_SET:
case CARD_SET: {
return card->getCardSetShortName(); return card->getCardSetShortName();
} case CARD_COLLECTOR_NUMBER:
case CARD_COLLECTOR_NUMBER: {
return card->getCardCollectorNumber(); return card->getCardCollectorNumber();
} case CARD_PROVIDER_ID:
case CARD_PROVIDER_ID: {
return card->getCardProviderId(); return card->getCardProviderId();
} default:
default: {
return {}; return {};
}
} }
case DeckRoles::IsCardRole: { case DeckRoles::IsCardRole: {