Add fallback urls for cards without a muId set
We only have a good HQ source for them; any idea on a low-quality source?
This commit is contained in:
parent
6b1df247d1
commit
fc1d06e13b
3 changed files with 30 additions and 6 deletions
|
@ -224,16 +224,18 @@ QString PictureLoader::getPicUrl()
|
||||||
return picUrl;
|
return picUrl;
|
||||||
}
|
}
|
||||||
|
|
||||||
// otherwise, fallback to the default url
|
// if a card has a muid, use the default url; if not, use the fallback
|
||||||
picUrl = picDownloadHq ? settingsCache->getPicUrlHq() : settingsCache->getPicUrl();
|
int muid = set ? muid = card->getMuId(set->getShortName()) : 0;
|
||||||
picUrl.replace("!name!", QUrl::toPercentEncoding(card->getCorrectedName()));
|
if(muid)
|
||||||
|
picUrl = picDownloadHq ? settingsCache->getPicUrlHq() : settingsCache->getPicUrl();
|
||||||
|
else
|
||||||
|
picUrl = picDownloadHq ? settingsCache->getPicUrlHqFallback() : settingsCache->getPicUrlFallback();
|
||||||
|
|
||||||
|
picUrl.replace("!name!", QUrl::toPercentEncoding(card->getCorrectedName()));
|
||||||
|
picUrl.replace("!cardid!", QUrl::toPercentEncoding(QString::number(muid)));
|
||||||
if (set) {
|
if (set) {
|
||||||
picUrl.replace("!setcode!", QUrl::toPercentEncoding(set->getShortName()));
|
picUrl.replace("!setcode!", QUrl::toPercentEncoding(set->getShortName()));
|
||||||
picUrl.replace("!setname!", QUrl::toPercentEncoding(set->getLongName()));
|
picUrl.replace("!setname!", QUrl::toPercentEncoding(set->getLongName()));
|
||||||
int muid = card->getMuId(set->getShortName());
|
|
||||||
if (muid)
|
|
||||||
picUrl.replace("!cardid!", QUrl::toPercentEncoding(QString::number(muid)));
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (picUrl.contains("!name!") ||
|
if (picUrl.contains("!name!") ||
|
||||||
|
|
|
@ -25,6 +25,8 @@ SettingsCache::SettingsCache()
|
||||||
picDownloadHq = settings->value("personal/picturedownloadhq", false).toBool();
|
picDownloadHq = settings->value("personal/picturedownloadhq", false).toBool();
|
||||||
picUrl = settings->value("personal/picUrl", PIC_URL_DEFAULT).toString();
|
picUrl = settings->value("personal/picUrl", PIC_URL_DEFAULT).toString();
|
||||||
picUrlHq = settings->value("personal/picUrlHq", PIC_URL_HQ_DEFAULT).toString();
|
picUrlHq = settings->value("personal/picUrlHq", PIC_URL_HQ_DEFAULT).toString();
|
||||||
|
picUrlFallback = settings->value("personal/picUrlFallback", PIC_URL_FALLBACK).toString();
|
||||||
|
picUrlHqFallback = settings->value("personal/picUrlHqFallback", PIC_URL_HQ_FALLBACK).toString();
|
||||||
|
|
||||||
mainWindowGeometry = settings->value("interface/main_window_geometry").toByteArray();
|
mainWindowGeometry = settings->value("interface/main_window_geometry").toByteArray();
|
||||||
notificationsEnabled = settings->value("interface/notificationsenabled", true).toBool();
|
notificationsEnabled = settings->value("interface/notificationsenabled", true).toBool();
|
||||||
|
@ -153,6 +155,18 @@ void SettingsCache::setPicUrlHq(const QString &_picUrlHq)
|
||||||
settings->setValue("personal/picUrlHq", picUrlHq);
|
settings->setValue("personal/picUrlHq", picUrlHq);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void SettingsCache::setPicUrlFallback(const QString &_picUrlFallback)
|
||||||
|
{
|
||||||
|
picUrlFallback = _picUrlFallback;
|
||||||
|
settings->setValue("personal/picUrlFallback", picUrlFallback);
|
||||||
|
}
|
||||||
|
|
||||||
|
void SettingsCache::setPicUrlHqFallback(const QString &_picUrlHqFallback)
|
||||||
|
{
|
||||||
|
picUrlHqFallback = _picUrlHqFallback;
|
||||||
|
settings->setValue("personal/picUrlHqFallback", picUrlHqFallback);
|
||||||
|
}
|
||||||
|
|
||||||
void SettingsCache::setNotificationsEnabled(int _notificationsEnabled)
|
void SettingsCache::setNotificationsEnabled(int _notificationsEnabled)
|
||||||
{
|
{
|
||||||
notificationsEnabled = _notificationsEnabled;
|
notificationsEnabled = _notificationsEnabled;
|
||||||
|
|
|
@ -4,7 +4,9 @@
|
||||||
#include <QObject>
|
#include <QObject>
|
||||||
|
|
||||||
#define PIC_URL_DEFAULT "http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=!cardid!&type=card"
|
#define PIC_URL_DEFAULT "http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=!cardid!&type=card"
|
||||||
|
#define PIC_URL_FALLBACK "http://mtgimage.com/set/!setcode!/!name!.jpg"
|
||||||
#define PIC_URL_HQ_DEFAULT "http://mtgimage.com/multiverseid/!cardid!.jpg"
|
#define PIC_URL_HQ_DEFAULT "http://mtgimage.com/multiverseid/!cardid!.jpg"
|
||||||
|
#define PIC_URL_HQ_FALLBACK "http://mtgimage.com/set/!setcode!/!name!.jpg"
|
||||||
|
|
||||||
class QSettings;
|
class QSettings;
|
||||||
|
|
||||||
|
@ -57,6 +59,8 @@ private:
|
||||||
bool ignoreUnregisteredUsers;
|
bool ignoreUnregisteredUsers;
|
||||||
QString picUrl;
|
QString picUrl;
|
||||||
QString picUrlHq;
|
QString picUrlHq;
|
||||||
|
QString picUrlFallback;
|
||||||
|
QString picUrlHqFallback;
|
||||||
bool attemptAutoConnect;
|
bool attemptAutoConnect;
|
||||||
public:
|
public:
|
||||||
SettingsCache();
|
SettingsCache();
|
||||||
|
@ -93,6 +97,8 @@ public:
|
||||||
bool getIgnoreUnregisteredUsers() const { return ignoreUnregisteredUsers; }
|
bool getIgnoreUnregisteredUsers() const { return ignoreUnregisteredUsers; }
|
||||||
QString getPicUrl() const { return picUrl; }
|
QString getPicUrl() const { return picUrl; }
|
||||||
QString getPicUrlHq() const { return picUrlHq; }
|
QString getPicUrlHq() const { return picUrlHq; }
|
||||||
|
QString getPicUrlFallback() const { return picUrlFallback; }
|
||||||
|
QString getPicUrlHqFallback() const { return picUrlHqFallback; }
|
||||||
void copyPath(const QString &src, const QString &dst);
|
void copyPath(const QString &src, const QString &dst);
|
||||||
bool getAutoConnect() const { return attemptAutoConnect; }
|
bool getAutoConnect() const { return attemptAutoConnect; }
|
||||||
public slots:
|
public slots:
|
||||||
|
@ -129,6 +135,8 @@ public slots:
|
||||||
void setIgnoreUnregisteredUsers(bool _ignoreUnregisteredUsers);
|
void setIgnoreUnregisteredUsers(bool _ignoreUnregisteredUsers);
|
||||||
void setPicUrl(const QString &_picUrl);
|
void setPicUrl(const QString &_picUrl);
|
||||||
void setPicUrlHq(const QString &_picUrlHq);
|
void setPicUrlHq(const QString &_picUrlHq);
|
||||||
|
void setPicUrlFallback(const QString &_picUrlFallback);
|
||||||
|
void setPicUrlHqFallback(const QString &_picUrlHqFallback);
|
||||||
void setAutoConnect(const bool &_autoConnect);
|
void setAutoConnect(const bool &_autoConnect);
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue