diff --git a/Source/Core/DolphinQt2/GameList/GameListModel.cpp b/Source/Core/DolphinQt2/GameList/GameListModel.cpp index 30aaa9d77b..4d00a2e3df 100644 --- a/Source/Core/DolphinQt2/GameList/GameListModel.cpp +++ b/Source/Core/DolphinQt2/GameList/GameListModel.cpp @@ -21,17 +21,24 @@ QVariant GameListModel::data(const QModelIndex& index, int role) const return QVariant(); QSharedPointer game = m_games[index.row()]; - if (role == Qt::DecorationRole) + + switch (index.column()) { - switch (index.column()) - { - case COL_PLATFORM: + case COL_PLATFORM: + if (role == Qt::DecorationRole) return Resources::GetPlatform(static_cast(game->GetPlatformID())); - case COL_COUNTRY: + break; + case COL_COUNTRY: + if (role == Qt::DecorationRole) return Resources::GetCountry(static_cast(game->GetCountryID())); - case COL_RATING: + break; + case COL_RATING: + if (role == Qt::DecorationRole) return Resources::GetRating(game->GetRating()); - case COL_BANNER: + break; + case COL_BANNER: + if (role == Qt::DecorationRole) + { // GameCube banners are 96x32, but Wii banners are 192x64. // TODO: use custom banners from rom directory like DolphinWX? QPixmap banner = game->GetBanner(); @@ -39,23 +46,29 @@ QVariant GameListModel::data(const QModelIndex& index, int role) const banner.height() / GAMECUBE_BANNER_SIZE.height())); return banner; } - } - if (role == Qt::DisplayRole) - { - switch (index.column()) - { - case COL_TITLE: + break; + case COL_TITLE: + if (role == Qt::DisplayRole) return game->GetLongName(); - case COL_ID: + break; + case COL_ID: + if (role == Qt::DisplayRole) return game->GetGameID(); - case COL_DESCRIPTION: + break; + case COL_DESCRIPTION: + if (role == Qt::DisplayRole) return game->GetDescription(); - case COL_MAKER: + break; + case COL_MAKER: + if (role == Qt::DisplayRole) return game->GetMaker(); - case COL_SIZE: + break; + case COL_SIZE: + if (role == Qt::DisplayRole) return FormatSize(game->GetFileSize()); - } + break; } + return QVariant(); }