Report backend connection modification failures to user

Refresh network list on connection modification

tdenetworkmanager/src/configwidgets/tdenetman-connection_settings_dialog.cpp

@@ -286,8 +286,7 @@ ConnectionSettingsDialogImpl::slotConnect()
 	TDEGlobalNetworkManager* nm = KGlobal::networkManager();
 
 	// add/save the connection
-	nm->saveConnection(_conn);
-
+	if (nm->saveConnection(_conn)) {
 		// activate device
 		if (_conn && nm) {
 	// 		// VPN connection needs a special specific object
@@ -301,6 +300,10 @@ ConnectionSettingsDialogImpl::slotConnect()
 				nm->initiateConnection(_conn->UUID);
 	// 		}
 		}
+	}
+	else {
+		KMessageBox::error(this, i18n("<qt><b>Unable to save network connection!</b><p>Potential causes:<br> * Insufficient permissions<br> * NetworkManager not running<br> * DBUS failure</qt>"), i18n("Unable to perform requested operation"));
+	}
 
 	emit connectionSaved();
 	this->close(true);
@@ -325,7 +328,9 @@ ConnectionSettingsDialogImpl::slotSave()
 	TDEGlobalNetworkManager* nm = KGlobal::networkManager();
 
 	// save the connection
-	nm->saveConnection(_conn);
+	if (!nm->saveConnection(_conn)) {
+		KMessageBox::error(this, i18n("<qt><b>Unable to save network connection!</b><p>Potential causes:<br> * Insufficient permissions<br> * NetworkManager not running<br> * DBUS failure</qt>"), i18n("Unable to perform requested operation"));
+	}
 
 	emit connectionSaved();
 	this->close(true);

tdenetworkmanager/src/tdenetman-connection_editor.cpp

@@ -33,13 +33,14 @@
 #include <tqlabel.h>
 #include <tqpopupmenu.h>
 
-// kde headers
+// tde headers
 #include <kiconloader.h>
 #include <kdebug.h>
 #include <kpushbutton.h>
 #include <klistview.h>
 #include <klocale.h>
 #include <kcombobox.h>
+#include <kmessagebox.h>
 
 // tdenm headers
 #include "tdenetman-connection_editor.h"
@@ -110,7 +111,7 @@ ConnectionEditorImpl::ConnectionEditorImpl(TQWidget* parent, const char* name, b
 	popup->insertItem(KGlobal::iconLoader()->loadIcon("wired", KIcon::Small), i18n("Wired"), this, TQT_SLOT(slotNewWiredConnection()));
 
 // 	if (!VPNManager::getVPNServices().isEmpty()) {
-// 		popup->insertItem(KGlobal::iconLoader()->loadIcon("encrypted", KIcon::Small), i18n("VPN"), this, TQT_SLOT(slotNewVPNConnection()));
+		popup->insertItem(KGlobal::iconLoader()->loadIcon("encrypted", KIcon::Small), i18n("VPN"), this, TQT_SLOT(slotNewVPNConnection()));
 // 	}
 
 	pbNew->setPopup(popup);
@@ -158,7 +159,7 @@ void ConnectionEditorImpl::slotNewWiredConnection()
  */
 void ConnectionEditorImpl::slotNewVPNConnection()
 {
-// 	slotEditNewConnection(new TDEVPNConnection());
+	slotEditNewConnection(new TDEVPNConnection());
 }
 
 /*
@@ -174,6 +175,11 @@ void ConnectionEditorImpl::slotEditNewConnection(TDENetworkConnection* conn)
 
 void ConnectionEditorImpl::slotRefreshConnectionList()
 {
+	TDEGlobalNetworkManager* nm = KGlobal::networkManager();
+	if (!nm) {
+		return;
+	}
+	nm->loadConnectionInformation();
 	fillConnectionList();
 }
 
@@ -224,7 +230,9 @@ void ConnectionEditorImpl::slotRemoveCurrentConnection()
 	lvConnections->takeItem(item);
 	delete item;
 
-	nm->deleteConnection(conn->UUID);
+	if (!nm->deleteConnection(conn->UUID)) {
+		KMessageBox::error(this, i18n("<qt><b>Unable to delete network connection!</b><p>Potential causes:<br> * Insufficient permissions<br> * NetworkManager not running<br> * DBUS failure</qt>"), i18n("Unable to perform requested operation"));
+	}
 }
 
 /*

tdenetworkmanager/src/tdenetman-tray.cpp

@@ -165,7 +165,7 @@ void Tray::slotEditConnections()
 	if (!nm) return;
 	if (tdenetworkmanager_editor_dialog_count == 0) nm->loadConnectionInformation();
 
-	ConnectionEditorImpl* dlg = new ConnectionEditorImpl(this);
+	ConnectionEditorImpl* dlg = new ConnectionEditorImpl(this, NULL, TRUE, (WFlags)TQt::WDestructiveClose);
 	dlg->show();
 }