main: don't ignore config value for single instance mode. allow override using cli param

main.cpp

@@ -31,6 +31,7 @@ int main(int argc, char *argv[])
 	QApplication app(argc, argv);
 	QString configDirectoryPath;
 	QDir dir;
+	bool newInstanceRequested = false;
 	if(argc >= 2)
 	{
 		QCommandLineParser parser;
@@ -42,16 +43,19 @@ int main(int argc, char *argv[])
 		parser.addHelpOption();
 		parser.process(app.arguments());
 		configDirectoryPath = parser.value("config");
-		if(!dir.exists(configDirectoryPath))
+		newInstanceRequested = parser.isSet("new-instance");
+
+		if(!configDirectoryPath.isEmpty() && !dir.exists(configDirectoryPath))
 		{
 			QMessageBox::warning(nullptr, "Directory not found", configDirectoryPath + " was not found");
 			return 1;
 		}
 	}
-	else
+	if(configDirectoryPath.isEmpty())
 	{
 		configDirectoryPath = QDir::homePath() + "/.config/qsrun/";
 	}
+
 	qRegisterMetaType<QVector<QString>>("QVector<QString>");
 
 	if(!dir.exists(configDirectoryPath))
@@ -68,10 +72,14 @@ int main(int argc, char *argv[])
 
 	SettingsProvider settingsProvider{settings};
 	EntryProvider entryProvider(settingsProvider.userEntriesPaths(), settingsProvider.systemApplicationsEntriesPaths());
-	// TODO if setting single instance mode
+
+	SingleInstanceServer *server = nullptr;
+
+	bool singleInstanceMode = !newInstanceRequested && settingsProvider.singleInstanceMode();
+	if(singleInstanceMode)
+	{
 		QLocalSocket localSocket;
-	localSocket.connectToServer("/tmp/qsrun.socket");
+		localSocket.connectToServer(settingsProvider.socketPath());
-	SingleInstanceServer server;
 		if(localSocket.isOpen() && localSocket.isWritable())
 		{
 			QDataStream stream(&localSocket);
@@ -81,14 +89,19 @@ int main(int argc, char *argv[])
 			localSocket.disconnectFromServer();
 			return 0;
 		}
-	else
+
-	{
+		server = new SingleInstanceServer();
-		if(!server.listen("/tmp/qsrun.socket"))
+		if(!server->listen(settingsProvider.socketPath()))
 		{
 			qDebug() << "Failed to listen on socket!";
+			return 1;
 		}
+	}
+
 	Window *w = new Window{entryProvider, settingsProvider};
-		QObject::connect(&server, &SingleInstanceServer::receivedMaximizationRequest, [&w] {
+	if(singleInstanceMode && server != nullptr)
+	{
+		QObject::connect(server, &SingleInstanceServer::receivedMaximizationRequest, [&w] {
 			if(w != nullptr)
 			{
 				qInfo() << "maximizing as requested by other instance";
@@ -98,9 +111,10 @@ int main(int argc, char *argv[])
 				w->focusInput();
 			}
 		});
+	}
+
 	w->showMaximized();
 	w->focusInput();
-	}
 
 	return app.exec();
 }

settingsprovider.cpp

@@ -33,3 +33,8 @@ QString SettingsProvider::getTerminalCommand() const
 {
 	return settings->value("terminal", "/usr/bin/x-terminal-emulator -e %c").toString();
 }
+
+QString SettingsProvider::socketPath() const
+{
+	return settings->value("singleInstanceSocket", "/tmp/qsrun").toString();
+}

settingsprovider.h

@@ -16,6 +16,7 @@ class SettingsProvider
 	virtual int getMaxCols() const;
 	virtual bool singleInstanceMode() const;
 	QString getTerminalCommand() const;
+	QString socketPath() const;
 };
 
 #endif // SETTINGSPROVIDER_H