Added back the INIFile read function
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
This commit is contained in:
parent
703f7698c1
commit
b77fb05908
@ -45,12 +45,12 @@
|
|||||||
|
|
||||||
#include <QSettings>
|
#include <QSettings>
|
||||||
|
|
||||||
INIFile::INIFile()
|
INIFile::INIFile() {}
|
||||||
{
|
|
||||||
}
|
|
||||||
|
|
||||||
bool INIFile::saveFile(QString fileName)
|
bool INIFile::saveFile(QString fileName)
|
||||||
{
|
{
|
||||||
|
if (!contains("ConfigVersion"))
|
||||||
|
insert("ConfigVersion", "1.1");
|
||||||
QSettings _settings_obj{ fileName, QSettings::Format::IniFormat };
|
QSettings _settings_obj{ fileName, QSettings::Format::IniFormat };
|
||||||
_settings_obj.setFallbacksEnabled(false);
|
_settings_obj.setFallbacksEnabled(false);
|
||||||
|
|
||||||
@ -71,6 +71,67 @@ bool INIFile::saveFile(QString fileName)
|
|||||||
|
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
QString unescape(QString orig)
|
||||||
|
{
|
||||||
|
QString out;
|
||||||
|
QChar prev = QChar::Null;
|
||||||
|
for (auto c : orig) {
|
||||||
|
if (prev == '\\') {
|
||||||
|
if (c == 'n')
|
||||||
|
out += '\n';
|
||||||
|
else if (c == 't')
|
||||||
|
out += '\t';
|
||||||
|
else if (c == '#')
|
||||||
|
out += '#';
|
||||||
|
else
|
||||||
|
out += c;
|
||||||
|
prev = QChar::Null;
|
||||||
|
} else {
|
||||||
|
if (c == '\\') {
|
||||||
|
prev = c;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
out += c;
|
||||||
|
prev = QChar::Null;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return out;
|
||||||
|
}
|
||||||
|
bool parseOldFileFormat(QIODevice& device, QSettings::SettingsMap& map)
|
||||||
|
{
|
||||||
|
QTextStream in(device.readAll());
|
||||||
|
#if QT_VERSION <= QT_VERSION_CHECK(6, 0, 0)
|
||||||
|
in.setCodec("UTF-8");
|
||||||
|
#endif
|
||||||
|
|
||||||
|
QStringList lines = in.readAll().split('\n');
|
||||||
|
for (int i = 0; i < lines.count(); i++) {
|
||||||
|
QString& lineRaw = lines[i];
|
||||||
|
// Ignore comments.
|
||||||
|
int commentIndex = 0;
|
||||||
|
QString line = lineRaw;
|
||||||
|
// Search for comments until no more escaped # are available
|
||||||
|
while ((commentIndex = line.indexOf('#', commentIndex + 1)) != -1) {
|
||||||
|
if (commentIndex > 0 && line.at(commentIndex - 1) == '\\') {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
line = line.left(lineRaw.indexOf('#')).trimmed();
|
||||||
|
}
|
||||||
|
|
||||||
|
int eqPos = line.indexOf('=');
|
||||||
|
if (eqPos == -1)
|
||||||
|
continue;
|
||||||
|
QString key = line.left(eqPos).trimmed();
|
||||||
|
QString valueStr = line.right(line.length() - eqPos - 1).trimmed();
|
||||||
|
|
||||||
|
valueStr = unescape(valueStr);
|
||||||
|
|
||||||
|
QVariant value(valueStr);
|
||||||
|
map.insert(key, value);
|
||||||
|
}
|
||||||
|
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
bool INIFile::loadFile(QString fileName)
|
bool INIFile::loadFile(QString fileName)
|
||||||
{
|
{
|
||||||
@ -84,10 +145,19 @@ bool INIFile::loadFile(QString fileName)
|
|||||||
qCritical() << "A format error occurred (e.g. loading a malformed INI file).";
|
qCritical() << "A format error occurred (e.g. loading a malformed INI file).";
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
if (!_settings_obj.value("ConfigVersion").isValid()) {
|
||||||
|
QFile file(fileName);
|
||||||
|
if (!file.open(QIODevice::ReadOnly))
|
||||||
|
return false;
|
||||||
|
QSettings::SettingsMap map;
|
||||||
|
parseOldFileFormat(file, map);
|
||||||
|
file.close();
|
||||||
|
for (auto&& key : map.keys())
|
||||||
|
insert(key, map.value(key));
|
||||||
|
insert("ConfigVersion", "1.1");
|
||||||
|
} else
|
||||||
for (auto&& key : _settings_obj.allKeys())
|
for (auto&& key : _settings_obj.allKeys())
|
||||||
insert(key, _settings_obj.value(key));
|
insert(key, _settings_obj.value(key));
|
||||||
|
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -103,4 +173,3 @@ void INIFile::set(QString key, QVariant val)
|
|||||||
{
|
{
|
||||||
this->operator[](key) = val;
|
this->operator[](key) = val;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1,24 +1,16 @@
|
|||||||
#include <QTest>
|
#include <QTest>
|
||||||
|
|
||||||
|
#include <settings/INIFile.h>
|
||||||
#include <QList>
|
#include <QList>
|
||||||
#include <QVariant>
|
#include <QVariant>
|
||||||
#include <settings/INIFile.h>
|
|
||||||
|
|
||||||
#include <QVariantUtils.h>
|
#include <QVariantUtils.h>
|
||||||
|
|
||||||
class IniFileTest : public QObject
|
class IniFileTest : public QObject {
|
||||||
{
|
|
||||||
Q_OBJECT
|
Q_OBJECT
|
||||||
private
|
private slots:
|
||||||
slots:
|
void initTestCase() {}
|
||||||
void initTestCase()
|
void cleanupTestCase() {}
|
||||||
{
|
|
||||||
|
|
||||||
}
|
|
||||||
void cleanupTestCase()
|
|
||||||
{
|
|
||||||
|
|
||||||
}
|
|
||||||
|
|
||||||
void test_Escape_data()
|
void test_Escape_data()
|
||||||
{
|
{
|
||||||
@ -47,17 +39,17 @@ slots:
|
|||||||
// load
|
// load
|
||||||
INIFile f2;
|
INIFile f2;
|
||||||
f2.loadFile(filename);
|
f2.loadFile(filename);
|
||||||
QCOMPARE(f2.get("a","NOT SET").toString(), a);
|
QCOMPARE(f2.get("a", "NOT SET").toString(), a);
|
||||||
QCOMPARE(f2.get("b","NOT SET").toString(), b);
|
QCOMPARE(f2.get("b", "NOT SET").toString(), b);
|
||||||
}
|
}
|
||||||
|
|
||||||
void test_SaveLoadLists()
|
void test_SaveLoadLists()
|
||||||
{
|
{
|
||||||
QString slist_strings = "(\"a\",\"b\",\"c\")";
|
QString slist_strings = "(\"a\",\"b\",\"c\")";
|
||||||
QStringList list_strings = {"a", "b", "c"};
|
QStringList list_strings = { "a", "b", "c" };
|
||||||
|
|
||||||
QString slist_numbers = "(1,2,3,10)";
|
QString slist_numbers = "(1,2,3,10)";
|
||||||
QList<int> list_numbers = {1, 2, 3, 10};
|
QList<int> list_numbers = { 1, 2, 3, 10 };
|
||||||
|
|
||||||
QString filename = "test_SaveLoadLists.ini";
|
QString filename = "test_SaveLoadLists.ini";
|
||||||
|
|
||||||
@ -79,6 +71,34 @@ slots:
|
|||||||
QCOMPARE(out_list_strings, list_strings);
|
QCOMPARE(out_list_strings, list_strings);
|
||||||
QCOMPARE(out_list_numbers, list_numbers);
|
QCOMPARE(out_list_numbers, list_numbers);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void test_SaveAleardyExistingFile()
|
||||||
|
{
|
||||||
|
QString fileName = "test_SaveAleardyExistingFile.ini";
|
||||||
|
QString fileContent = R"(InstanceType=OneSix
|
||||||
|
iconKey=vanillia_icon
|
||||||
|
name=Minecraft Vanillia
|
||||||
|
OverrideCommands=true
|
||||||
|
PreLaunchCommand="$INST_JAVA" -jar packwiz-installer-bootstrap.jar link
|
||||||
|
)";
|
||||||
|
QFile file(fileName);
|
||||||
|
|
||||||
|
if (file.open(QFile::WriteOnly | QFile::Text)) {
|
||||||
|
QTextStream stream(&file);
|
||||||
|
stream << fileContent;
|
||||||
|
file.close();
|
||||||
|
}
|
||||||
|
|
||||||
|
// load
|
||||||
|
INIFile f1;
|
||||||
|
f1.loadFile(fileName);
|
||||||
|
QCOMPARE(f1.get("PreLaunchCommand", "NOT SET").toString(), "\"$INST_JAVA\" -jar packwiz-installer-bootstrap.jar link");
|
||||||
|
f1.saveFile(fileName);
|
||||||
|
INIFile f2;
|
||||||
|
f2.loadFile(fileName);
|
||||||
|
QCOMPARE(f2.get("PreLaunchCommand", "NOT SET").toString(), "\"$INST_JAVA\" -jar packwiz-installer-bootstrap.jar link");
|
||||||
|
QCOMPARE(f2.get("ConfigVersion", "NOT SET").toString(), "1.1");
|
||||||
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
QTEST_GUILESS_MAIN(IniFileTest)
|
QTEST_GUILESS_MAIN(IniFileTest)
|
||||||
|
Loading…
Reference in New Issue
Block a user