ui: fixes

* Fixed send ping after changing plugin ports
* Fixed can't close EULA window

src/lib/component/controller/IDECommunicationController.cpp

@@ -68,9 +68,16 @@ void IDECommunicationController::setEnabled(const bool enabled)
 	m_enabled = enabled;
 }
 
-void IDECommunicationController::sendInitialPing()
+void IDECommunicationController::sendUpdatePing()
 {
-	sendUpdatePing();
+	// first reset connection status
+	MessagePingReceived msg;
+	msg.ideId = "";
+	msg.ideName = "";
+	msg.dispatch();
+
+	// send ping to update connection status
+	sendMessage(NetworkProtocolHelper::buildPingMessage());
 }
 
 void IDECommunicationController::handleSetActiveTokenMessage(
@@ -220,15 +227,3 @@ void IDECommunicationController::handleMessage(MessagePluginPortChange* message)
 	stopListening();
 	startListening();
 }
-
-void IDECommunicationController::sendUpdatePing()
-{
-	// first reset connection status
-	MessagePingReceived msg;
-	msg.ideId = "";
-	msg.ideName = "";
-	msg.dispatch();
-
-	// send ping to update connection status
-	sendMessage(NetworkProtocolHelper::buildPingMessage());
-}

src/lib/component/controller/IDECommunicationController.h

@@ -37,7 +37,7 @@ public:
 	void setEnabled(const bool enabled);
 
 protected:
-	void sendInitialPing();
+	void sendUpdatePing();
 
 private:
 	void handleSetActiveTokenMessage(const NetworkProtocolHelper::SetActiveTokenMessage& message);
@@ -51,8 +51,6 @@ private:
 	virtual void handleMessage(MessagePluginPortChange* message);
 	virtual void sendMessage(const std::string& message) const = 0;
 
-	void sendUpdatePing();
-
 	StorageAccess* m_storageAccess;
 
 	bool m_enabled;

src/lib_gui/qt/network/QtIDECommunicationController.cpp

@@ -23,10 +23,11 @@ void QtIDECommunicationController::startListening()
 			m_tcpWrapper.setServerPort(appSettings->getSourcetrailPort());
 			m_tcpWrapper.setClientPort(appSettings->getPluginPort());
 			m_tcpWrapper.startListening();
+
+			sendUpdatePing();
 		}
 	);
 
-	sendInitialPing();
 }
 
 void QtIDECommunicationController::stopListening()

src/lib_gui/qt/window/QtMainWindow.cpp

@@ -336,17 +336,21 @@ void QtMainWindow::showBugtracker()
 
 void QtMainWindow::showEula(bool forceAccept)
 {
-	QtEulaWindow* eulaWindow = new QtEulaWindow(this, forceAccept);
-	m_windowStack.pushWindow(eulaWindow);
-	eulaWindow->setup();
+	QtEulaWindow* window = new QtEulaWindow(this, forceAccept);
+	m_windowStack.pushWindow(window);
+	window->setup();
 
 	if (forceAccept)
 	{
 		setEnabled(false);
-		eulaWindow->setEnabled(true);
+		window->setEnabled(true);
 
-		connect(eulaWindow, SIGNAL(finished()), this, SLOT(acceptedEula()));
-		connect(eulaWindow, SIGNAL(canceled()), dynamic_cast<QApplication*>(QCoreApplication::instance()), SLOT(quit()));
+		connect(window, SIGNAL(finished()), this, SLOT(acceptedEula()));
+		connect(window, SIGNAL(canceled()), dynamic_cast<QApplication*>(QCoreApplication::instance()), SLOT(quit()));
+	}
+	else
+	{
+		connect(window, SIGNAL(canceled()), &m_windowStack, SLOT(popWindow()));
 	}
 }