fixup! Implement basic 'Terminal=' support for .desktop files

entryprovider.cpp

@@ -13,11 +13,6 @@ EntryProvider::EntryProvider(QStringList userEntriesDirsPaths, QStringList syste
 					   << "%u";
 }
 
-bool EntryProvider::isSavable(const EntryConfig &config) const
-{
-	return ! config.entryPath.isEmpty() && (config.type == EntryType::USER || config.type == EntryType::INHERIT);
-}
-
 EntryConfig EntryProvider::readFromDesktopFile(const QString &path)
 {
 	EntryConfig result;
@@ -290,7 +285,7 @@ QVector<EntryConfig> EntryProvider::getSystemEntries()
 
 void EntryProvider::saveUserEntry(const EntryConfig &config)
 {
-	if(!isSavable(config))
+	if(config.type == EntryType::SYSTEM || config.entryPath.isEmpty())
 	{
 		throw std::runtime_error("Only user/inherited entries can be saved");
 	}
@@ -344,7 +339,7 @@ void EntryProvider::saveUserEntry(const EntryConfig &config)
 
 bool EntryProvider::deleteUserEntry(const EntryConfig &config)
 {
-	if(!isSavable(config))
+	if(config.type == EntryType::SYSTEM || config.entryPath.isEmpty())
 	{
 		throw std::runtime_error("Only user/inherited entries can be deleted");
 	}

entryprovider.h

@@ -55,7 +55,6 @@ class EntryProvider
 
   public:
 	EntryProvider(QStringList userEntriesDirsPaths, QStringList systemEntriesDirsPaths);
-	bool isSavable(const EntryConfig &config) const;
 	QVector<EntryConfig> getUserEntries();
 	QVector<EntryConfig> getSystemEntries();
 	void saveUserEntry(const EntryConfig &config);

main.cpp

@@ -14,7 +14,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 #include <QApplication>
-#include <QCommandLineParser>
 #include <QFuture>
 #include <QFutureWatcher>
 #include <QtConcurrent/QtConcurrentRun>
@@ -33,15 +32,7 @@ int main(int argc, char *argv[])
 	QDir dir;
 	if(argc >= 2)
 	{
-		QCommandLineParser parser;
+		configDirectoryPath = QCoreApplication::arguments().at(1);
-		parser.addOptions({
-			{"new-instance", "Launch a new instance, ignoring any running ones"},
-			{"kill-instances", "Kill all running instances"},
-			{"config", "Use supplied config dir instead of default"},
-		});
-		parser.addHelpOption();
-		parser.process(app.arguments());
-		configDirectoryPath = parser.value("config");
 		if(!dir.exists(configDirectoryPath))
 		{
 			QMessageBox::warning(nullptr, "Directory not found", configDirectoryPath + " was not found");
@@ -52,30 +43,30 @@ int main(int argc, char *argv[])
 	{
 		configDirectoryPath = QDir::homePath() + "/.config/qsrun/";
 	}
-	qRegisterMetaType<QVector<QString>>("QVector<QString>");
+	qRegisterMetaType<QVector<QString> >("QVector<QString>");
+
 
 	if(!dir.exists(configDirectoryPath))
 	{
 		if(!dir.mkdir(configDirectoryPath))
 		{
-			QMessageBox::warning(nullptr, "Failed to create dir",
+			QMessageBox::warning(nullptr, "Failed to create dir", configDirectoryPath + " was not found and could not be created!");
-								 configDirectoryPath + " was not found and could not be created!");
 			return 1;
 		}
 	}
 
 	QSettings settings(configDirectoryPath + "qsrun.config", QSettings::NativeFormat);
 
-	SettingsProvider settingsProvider{settings};
+	SettingsProvider settingsProvider { settings };
 	EntryProvider entryProvider(settingsProvider.userEntriesPaths(), settingsProvider.systemApplicationsEntriesPaths());
-	// TODO if setting single instance mode
+	//TODO if setting single instance mode
 	QLocalSocket localSocket;
 	localSocket.connectToServer("/tmp/qsrun.socket");
 	SingleInstanceServer server;
 	if(localSocket.isOpen() && localSocket.isWritable())
 	{
 		QDataStream stream(&localSocket);
-		stream << (int)0x01; // maximize
+		stream << (int)0x01; //maximize
 		localSocket.flush();
 		localSocket.waitForBytesWritten();
 		localSocket.disconnectFromServer();
@@ -87,8 +78,8 @@ int main(int argc, char *argv[])
 		{
 			qDebug() << "Failed to listen on socket!";
 		}
-		Window *w = new Window{entryProvider, settingsProvider};
+		Window *w = new Window { entryProvider, settingsProvider };
-		QObject::connect(&server, &SingleInstanceServer::receivedMaximizationRequest, [&w] {
+		QObject::connect(&server, &SingleInstanceServer::receivedMaximizationRequest, [&w]{
 			if(w != nullptr)
 			{
 				qInfo() << "maximizing as requested by other instance";
@@ -100,7 +91,10 @@ int main(int argc, char *argv[])
 		});
 		w->showMaximized();
 		w->focusInput();
+
 	}
 
+
+
 	return app.exec();
 }

window.cpp

@@ -123,7 +123,7 @@ void Window::executeConfig(const EntryConfig &config)
 	if(config.isTerminalCommand)
 	{
 		QString cmd = settingsProvider->getTerminalCommand();
-		cmd.replace("%c", config.command);
+		cmd.replace("%c", config.command + " " + config.arguments.join(' '));
 		QProcess::startDetached(cmd);
 	}
 	else