jeirmeister/Cockatrice
1
0
Fork 0
mirror of https://github.com/Cockatrice/Cockatrice.git synced 2026-07-14 22:42:14 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Warnings.

Browse source 
Took 5 minutes
This commit is contained in:
Lukas Brübach 2025-07-10 13:26:39 +02:00
parent 5f88e874ef
commit d276cd7cce
2 changed files with 9 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
cockatrice/src/client/network/parsers/interface_json_deck_parser.h
View file

@ -67,7 +67,7 @@ public:
QString collectorNumber = cardData.value("cn").toString(); QString collectorNumber = cardData.value("cn").toString();
QString providerId = cardData.value("scryfall_id").toString(); QString providerId = cardData.value("scryfall_id").toString();
list->setBannerCard(QPair(commanderName, providerId)); list->setBannerCard(QPair<QString, QString>(commanderName, providerId));
list->addCard(commanderName, DECK_ZONE_MAIN, -1, setName, collectorNumber, providerId); list->addCard(commanderName, DECK_ZONE_MAIN, -1, setName, collectorNumber, providerId);
} }
} }

8
cockatrice/src/dialogs/dlg_load_deck_from_website.cpp
View file

@ -6,6 +6,7 @@
#include <QEventLoop> #include <QEventLoop>
#include <QJsonDocument> #include <QJsonDocument>
#include <QJsonObject> #include <QJsonObject>
#include <QMessageBox>
#include <QNetworkReply> #include <QNetworkReply>
DlgLoadDeckFromWebsite::DlgLoadDeckFromWebsite(QWidget *parent) : QDialog(parent) DlgLoadDeckFromWebsite::DlgLoadDeckFromWebsite(QWidget *parent) : QDialog(parent)
@ -58,6 +59,7 @@ void DlgLoadDeckFromWebsite::accept()
auto jsonParser = createParserForProvider(info.provider); auto jsonParser = createParserForProvider(info.provider);
if (!jsonParser && info.provider != DeckProvider::Deckstats && info.provider != DeckProvider::TappedOut) { if (!jsonParser && info.provider != DeckProvider::Deckstats && info.provider != DeckProvider::TappedOut) {
qWarning() << "No parser found for provider"; qWarning() << "No parser found for provider";
QMessageBox::warning(this, tr("Load Deck from Website"), tr("No parser available for this deck provider."));
QDialog::reject(); QDialog::reject();
return; return;
} }
@ -71,6 +73,7 @@ void DlgLoadDeckFromWebsite::accept()
if (reply->error() != QNetworkReply::NoError) { if (reply->error() != QNetworkReply::NoError) {
qWarning() << "Network error:" << reply->errorString(); qWarning() << "Network error:" << reply->errorString();
QMessageBox::warning(this, tr("Load Deck from Website"), tr("Network error: %1").arg(reply->errorString()));
reply->deleteLater(); reply->deleteLater();
QDialog::reject(); QDialog::reject();
return; return;
@ -84,6 +87,7 @@ void DlgLoadDeckFromWebsite::accept()
QString deckText = QString::fromUtf8(responseData); QString deckText = QString::fromUtf8(responseData);
if (deckText.isEmpty()) { if (deckText.isEmpty()) {
qWarning() << "Response is empty"; qWarning() << "Response is empty";
QMessageBox::warning(this, tr("Load Deck from Website"), tr("Received empty deck data."));
QDialog::reject(); QDialog::reject();
return; return;
} }
@ -104,6 +108,8 @@ void DlgLoadDeckFromWebsite::accept()
QJsonDocument doc = QJsonDocument::fromJson(responseData, &parseError); QJsonDocument doc = QJsonDocument::fromJson(responseData, &parseError);
if (parseError.error != QJsonParseError::NoError) { if (parseError.error != QJsonParseError::NoError) {
qWarning() << "JSON parse error:" << parseError.errorString(); qWarning() << "JSON parse error:" << parseError.errorString();
QMessageBox::warning(this, tr("Load Deck from Website"),
tr("Failed to parse deck data: %1").arg(parseError.errorString()));
QDialog::reject(); QDialog::reject();
return; return;
} }
@ -113,6 +119,8 @@ void DlgLoadDeckFromWebsite::accept()
} else { } else {
qInfo() << "URL not recognized"; qInfo() << "URL not recognized";
QMessageBox::warning(this, tr("Load Deck from Website"),
tr("The provided URL is not recognized as a valid deck URL."));
QDialog::reject(); QDialog::reject();
} }
} }