diff options
author | Petr Mrázek <peterix@gmail.com> | 2019-05-31 21:52:58 +0200 |
---|---|---|
committer | Petr Mrázek <peterix@gmail.com> | 2019-05-31 21:53:58 +0200 |
commit | 3470a3df966100d4f1ea6488892ac048118a3131 (patch) | |
tree | fc0f649f4bba0859f7f38d2b81876fd52a8266a2 /api | |
parent | 61913daaf3b21960de13340a6069bc3daca8744a (diff) | |
download | MultiMC-3470a3df966100d4f1ea6488892ac048118a3131.tar MultiMC-3470a3df966100d4f1ea6488892ac048118a3131.tar.gz MultiMC-3470a3df966100d4f1ea6488892ac048118a3131.tar.lz MultiMC-3470a3df966100d4f1ea6488892ac048118a3131.tar.xz MultiMC-3470a3df966100d4f1ea6488892ac048118a3131.zip |
NOISSUE improve icon handling while importing and exporting instances
Now it handles formats other than png.
Diffstat (limited to 'api')
-rw-r--r-- | api/gui/icons/MMCIcon.cpp | 14 | ||||
-rw-r--r-- | api/gui/icons/MMCIcon.h | 2 | ||||
-rw-r--r-- | api/logic/CMakeLists.txt | 4 | ||||
-rw-r--r-- | api/logic/InstanceImportTask.cpp | 6 | ||||
-rw-r--r-- | api/logic/icons/IconUtils.cpp | 62 | ||||
-rw-r--r-- | api/logic/icons/IconUtils.h | 14 |
6 files changed, 99 insertions, 3 deletions
diff --git a/api/gui/icons/MMCIcon.cpp b/api/gui/icons/MMCIcon.cpp index 2a2d6e6f..5058717a 100644 --- a/api/gui/icons/MMCIcon.cpp +++ b/api/gui/icons/MMCIcon.cpp @@ -102,3 +102,17 @@ void MMCIcon::replace(IconType new_type, const QString& key) m_images[new_type].filename = QString(); m_images[new_type].key = key; } + +QString MMCIcon::getFilePath() const +{ + if(m_current_type == IconType::ToBeDeleted){ + return QString(); + } + return m_images[m_current_type].filename; +} + + +bool MMCIcon::isBuiltIn() const +{ + return m_current_type == IconType::Builtin; +} diff --git a/api/gui/icons/MMCIcon.h b/api/gui/icons/MMCIcon.h index c0eb5570..9fd1b9a2 100644 --- a/api/gui/icons/MMCIcon.h +++ b/api/gui/icons/MMCIcon.h @@ -46,4 +46,6 @@ struct MULTIMC_GUI_EXPORT MMCIcon void remove(IconType rm_type); void replace(IconType new_type, QIcon icon, QString path = QString()); void replace(IconType new_type, const QString &key); + bool isBuiltIn() const; + QString getFilePath() const; }; diff --git a/api/logic/CMakeLists.txt b/api/logic/CMakeLists.txt index 39850163..94da2682 100644 --- a/api/logic/CMakeLists.txt +++ b/api/logic/CMakeLists.txt @@ -182,9 +182,11 @@ set(NEWS_SOURCES # Icon interface set(ICONS_SOURCES - # News System + # Icons System and related code icons/IIconList.h icons/IIconList.cpp + icons/IconUtils.h + icons/IconUtils.cpp ) # Minecraft services status checker diff --git a/api/logic/InstanceImportTask.cpp b/api/logic/InstanceImportTask.cpp index bbab557a..1e93174c 100644 --- a/api/logic/InstanceImportTask.cpp +++ b/api/logic/InstanceImportTask.cpp @@ -6,6 +6,7 @@ #include "NullInstance.h" #include "settings/INISettingsObject.h" #include "icons/IIconList.h" +#include "icons/IconUtils.h" #include <QtConcurrentRun> // FIXME: this does not belong here, it's Minecraft/Flame specific @@ -393,8 +394,9 @@ void InstanceImportTask::processMultiMC() else { m_instIcon = instance.iconKey(); - auto importIconPath = FS::PathCombine(instance.instanceRoot(), m_instIcon + ".png"); - if (QFile::exists(importIconPath)) + + auto importIconPath = IconUtils::findBestIconIn(instance.instanceRoot(), m_instIcon); + if (!importIconPath.isNull() && QFile::exists(importIconPath)) { // import icon auto iconList = ENV.icons(); diff --git a/api/logic/icons/IconUtils.cpp b/api/logic/icons/IconUtils.cpp new file mode 100644 index 00000000..1b6d7553 --- /dev/null +++ b/api/logic/icons/IconUtils.cpp @@ -0,0 +1,62 @@ +#include "IconUtils.h" + +#include "FileSystem.h" +#include <QDirIterator> + +#include <array> + +namespace { +std::array<const char *, 6> validIconExtensions = { + "svg", + "png", + "ico", + "gif", + "jpg", + "jpeg" +}; +} + +namespace IconUtils{ + +QString findBestIconIn(const QString &folder, const QString & iconKey) { + int best_found = validIconExtensions.size(); + QString best_filename; + + QDirIterator it(folder, QDir::NoDotAndDotDot | QDir::Files, QDirIterator::NoIteratorFlags); + while (it.hasNext()) { + it.next(); + auto fileInfo = it.fileInfo(); + + if(fileInfo.completeBaseName() != iconKey) + continue; + + auto extension = fileInfo.suffix(); + + for(int i = 0; i < best_found; i++) { + if(extension == validIconExtensions[i]) { + best_found = i; + qDebug() << i << " : " << fileInfo.fileName(); + best_filename = fileInfo.fileName(); + } + } + } + return FS::PathCombine(folder, best_filename); +} + +QString getIconFilter() { + QString out; + QTextStream stream(&out); + stream << '('; + for(size_t i = 0; i < validIconExtensions.size() - 1; i++) { + if(i > 0) { + stream << " "; + } + stream << "*." << validIconExtensions[i]; + } + stream << " *." << validIconExtensions[validIconExtensions.size() - 1]; + stream << ')'; + return out; +} + +} + diff --git a/api/logic/icons/IconUtils.h b/api/logic/icons/IconUtils.h new file mode 100644 index 00000000..ce236946 --- /dev/null +++ b/api/logic/icons/IconUtils.h @@ -0,0 +1,14 @@ +#pragma once + +#include <QString> +#include "multimc_logic_export.h" + +namespace IconUtils { + +// Given a folder and an icon key, find 'best' of the icons with the given key in there and return its path +MULTIMC_LOGIC_EXPORT QString findBestIconIn(const QString &folder, const QString & iconKey); + +// Get icon file type filter for file browser dialogs +MULTIMC_LOGIC_EXPORT QString getIconFilter(); + +} |