Added braces round key checking if statement to fix bug that WEP settings

were never saved if WEP was enabled

2 files changed, 10 insertions, 4 deletions

diff --git a/noncore/net/networksetup/wlan/wlanimp.cpp b/noncore/net/networksetup/wlan/wlanimp.cpp
index d4b4af9..e64a633 100644
--- a/noncore/net/networksetup/wlan/wlanimp.cpp
+++ b/noncore/net/networksetup/wlan/wlanimp.cpp
@@ -200,10 +200,13 @@ void WLANImp::changeAndSaveSettingFile(){
  * Save wireless.opts, save interfaces
  */
 void WLANImp::accept(){
+                printf( "Accept pressed\n" );
   if(wepEnabled->isChecked()){
     if(keyLineEdit0->text().isEmpty() && keyLineEdit1->text().isEmpty() && keyLineEdit2->text().isEmpty() && keyLineEdit3->text().isEmpty() )
-    QMessageBox::information(this, "", "Please enter a key for WEP.", QMessageBox::Ok);
-    return;
+        {
+            QMessageBox::information(this, "", "Please enter a key for WEP.", QMessageBox::Ok);
+            return;
+        }
           }
   
   // Ok settings are good here, save
diff --git a/noncore/settings/networksettings/wlan/wlanimp.cpp b/noncore/settings/networksettings/wlan/wlanimp.cpp
index d4b4af9..e64a633 100644
--- a/noncore/settings/networksettings/wlan/wlanimp.cpp
+++ b/noncore/settings/networksettings/wlan/wlanimp.cpp
@@ -200,10 +200,13 @@ void WLANImp::changeAndSaveSettingFile(){
  * Save wireless.opts, save interfaces
  */
 void WLANImp::accept(){
+                printf( "Accept pressed\n" );
   if(wepEnabled->isChecked()){
     if(keyLineEdit0->text().isEmpty() && keyLineEdit1->text().isEmpty() && keyLineEdit2->text().isEmpty() && keyLineEdit3->text().isEmpty() )
-    QMessageBox::information(this, "", "Please enter a key for WEP.", QMessageBox::Ok);
-    return;
+        {
+            QMessageBox::information(this, "", "Please enter a key for WEP.", QMessageBox::Ok);
+            return;
+        }
           }
   
   // Ok settings are good here, save