6 files changed, 30 insertions, 28 deletions

diff --git a/noncore/net/networksetup/TODO b/noncore/net/networksetup/TODO
index ec6d2c6..d6bfee1 100644
--- a/noncore/net/networksetup/TODO
+++ b/noncore/net/networksetup/TODO
@@ -4,4 +4,2 @@ Fix Profiles
 
-Write dns script
-
 WLAN needs to be re-written to not use Config
@@ -9,3 +7,6 @@ WLAN needs to be re-written to not use Config
 Write a class that parses /proc and not ifconfig
-udchcp needs to output the dhcp information
+
+udchcp needs to output the dhcp information so interfaces can read it
+
+interfacesetupimp really doesn't need a interface* pointer
 
diff --git a/noncore/net/networksetup/interfaces/interfacesetupimp.cpp b/noncore/net/networksetup/interfaces/interfacesetupimp.cpp
index a19aaeb..3b1a4de 100644
--- a/noncore/net/networksetup/interfaces/interfacesetupimp.cpp
+++ b/noncore/net/networksetup/interfaces/interfacesetupimp.cpp
@@ -62,3 +62,3 @@ bool InterfaceSetupImp::saveSettings(){
   if(!dhcpCheckBox->isChecked() && (ipAddressEdit->text().isEmpty() || subnetMaskEdit->text().isEmpty())){
-   QMessageBox::information(this, "Not Saved.", "Please fill in the IP address and\n subnet entries.", "Ok");
+   QMessageBox::information(this, "Not Saved.", "Please fill in the IP address and\n subnet entries.", QMessageBox::Ok);
    return false;
diff --git a/noncore/net/networksetup/mainwindowimp.cpp b/noncore/net/networksetup/mainwindowimp.cpp
index 6440bd4..3b0b7e8 100644
--- a/noncore/net/networksetup/mainwindowimp.cpp
+++ b/noncore/net/networksetup/mainwindowimp.cpp
@@ -171,3 +171,3 @@ void MainWindowImp::addClicked(){
   if(list.count() == 0){

-    QMessageBox::information(this, "Sorry", "Nothing to add.", "Ok");

+    QMessageBox::information(this, "Sorry", "Nothing to add.", QMessageBox::Ok);

     return;

@@ -201,3 +201,3 @@ void MainWindowImp::removeClicked(){
   if(!item) {

-    QMessageBox::information(this, "Sorry","Please select an interface First.", "Ok");

+    QMessageBox::information(this, "Sorry","Please select an interface First.", QMessageBox::Ok);

     return; 

@@ -207,3 +207,3 @@ void MainWindowImp::removeClicked(){
   if(i->getModuleOwner() == NULL){

-    QMessageBox::information(this, "Can't remove interface.", "Interface is built in.", "Ok");

+    QMessageBox::information(this, "Can't remove interface.", "Interface is built in.", QMessageBox::Ok);

   }

@@ -211,5 +211,5 @@ void MainWindowImp::removeClicked(){
     if(!i->getModuleOwner()->remove(i))

-      QMessageBox::information(this, "Error", "Unable to remove.", "Ok");

+      QMessageBox::information(this, "Error", "Unable to remove.", QMessageBox::Ok);

     else{

-      QMessageBox::information(this, "Success", "Interface was removed.", "Ok");

+      QMessageBox::information(this, "Success", "Interface was removed.", QMessageBox::Ok);

       // TODO memory managment....

@@ -446,3 +446,3 @@ void MainWindowImp::addProfile(){
   if(profiles.grep(newProfileName).count() > 0){

-    QMessageBox::information(this, "Can't Add","Profile already exists.", "Ok");

+    QMessageBox::information(this, "Can't Add","Profile already exists.", QMessageBox::Ok);

     return;

@@ -459,3 +459,3 @@ void MainWindowImp::removeProfile(){
   if(profilesList->count() <= 1){

-    QMessageBox::information(this, "Can't remove.","At least one profile\nis needed.", "Ok");

+    QMessageBox::information(this, "Can't remove.","At least one profile\nis needed.", QMessageBox::Ok);

     return;

@@ -464,3 +464,3 @@ void MainWindowImp::removeProfile(){
   if(profileToRemove == "All"){

-    QMessageBox::information(this, "Can't remove.","Can't remove default.", "Ok");

+    QMessageBox::information(this, "Can't remove.","Can't remove default.", QMessageBox::Ok);

     return;

@@ -469,3 +469,3 @@ void MainWindowImp::removeProfile(){
   if(profileToRemove == currentProfileLabel->text()){

-    QMessageBox::information(this, "Can't remove.",QString("%1 is the current profile.").arg(profileToRemove), "Ok");

+    QMessageBox::information(this, "Can't remove.",QString("%1 is the current profile.").arg(profileToRemove), QMessageBox::Ok);

     return;

@@ -509,3 +509,3 @@ void MainWindowImp::changeProfile(){
   if(profilesList->currentItem() == -1){

-    QMessageBox::information(this, "Can't Change.","Please select a profile.", "Ok");

+    QMessageBox::information(this, "Can't Change.","Please select a profile.", QMessageBox::Ok);

     return;

diff --git a/noncore/settings/networksettings/TODO b/noncore/settings/networksettings/TODO
index ec6d2c6..d6bfee1 100644
--- a/noncore/settings/networksettings/TODO
+++ b/noncore/settings/networksettings/TODO
@@ -4,4 +4,2 @@ Fix Profiles
 
-Write dns script
-
 WLAN needs to be re-written to not use Config
@@ -9,3 +7,6 @@ WLAN needs to be re-written to not use Config
 Write a class that parses /proc and not ifconfig
-udchcp needs to output the dhcp information
+
+udchcp needs to output the dhcp information so interfaces can read it
+
+interfacesetupimp really doesn't need a interface* pointer
 
diff --git a/noncore/settings/networksettings/interfaces/interfacesetupimp.cpp b/noncore/settings/networksettings/interfaces/interfacesetupimp.cpp
index a19aaeb..3b1a4de 100644
--- a/noncore/settings/networksettings/interfaces/interfacesetupimp.cpp
+++ b/noncore/settings/networksettings/interfaces/interfacesetupimp.cpp
@@ -62,3 +62,3 @@ bool InterfaceSetupImp::saveSettings(){
   if(!dhcpCheckBox->isChecked() && (ipAddressEdit->text().isEmpty() || subnetMaskEdit->text().isEmpty())){
-   QMessageBox::information(this, "Not Saved.", "Please fill in the IP address and\n subnet entries.", "Ok");
+   QMessageBox::information(this, "Not Saved.", "Please fill in the IP address and\n subnet entries.", QMessageBox::Ok);
    return false;
diff --git a/noncore/settings/networksettings/mainwindowimp.cpp b/noncore/settings/networksettings/mainwindowimp.cpp
index 6440bd4..3b0b7e8 100644
--- a/noncore/settings/networksettings/mainwindowimp.cpp
+++ b/noncore/settings/networksettings/mainwindowimp.cpp
@@ -171,3 +171,3 @@ void MainWindowImp::addClicked(){
   if(list.count() == 0){

-    QMessageBox::information(this, "Sorry", "Nothing to add.", "Ok");

+    QMessageBox::information(this, "Sorry", "Nothing to add.", QMessageBox::Ok);

     return;

@@ -201,3 +201,3 @@ void MainWindowImp::removeClicked(){
   if(!item) {

-    QMessageBox::information(this, "Sorry","Please select an interface First.", "Ok");

+    QMessageBox::information(this, "Sorry","Please select an interface First.", QMessageBox::Ok);

     return; 

@@ -207,3 +207,3 @@ void MainWindowImp::removeClicked(){
   if(i->getModuleOwner() == NULL){

-    QMessageBox::information(this, "Can't remove interface.", "Interface is built in.", "Ok");

+    QMessageBox::information(this, "Can't remove interface.", "Interface is built in.", QMessageBox::Ok);

   }

@@ -211,5 +211,5 @@ void MainWindowImp::removeClicked(){
     if(!i->getModuleOwner()->remove(i))

-      QMessageBox::information(this, "Error", "Unable to remove.", "Ok");

+      QMessageBox::information(this, "Error", "Unable to remove.", QMessageBox::Ok);

     else{

-      QMessageBox::information(this, "Success", "Interface was removed.", "Ok");

+      QMessageBox::information(this, "Success", "Interface was removed.", QMessageBox::Ok);

       // TODO memory managment....

@@ -446,3 +446,3 @@ void MainWindowImp::addProfile(){
   if(profiles.grep(newProfileName).count() > 0){

-    QMessageBox::information(this, "Can't Add","Profile already exists.", "Ok");

+    QMessageBox::information(this, "Can't Add","Profile already exists.", QMessageBox::Ok);

     return;

@@ -459,3 +459,3 @@ void MainWindowImp::removeProfile(){
   if(profilesList->count() <= 1){

-    QMessageBox::information(this, "Can't remove.","At least one profile\nis needed.", "Ok");

+    QMessageBox::information(this, "Can't remove.","At least one profile\nis needed.", QMessageBox::Ok);

     return;

@@ -464,3 +464,3 @@ void MainWindowImp::removeProfile(){
   if(profileToRemove == "All"){

-    QMessageBox::information(this, "Can't remove.","Can't remove default.", "Ok");

+    QMessageBox::information(this, "Can't remove.","Can't remove default.", QMessageBox::Ok);

     return;

@@ -469,3 +469,3 @@ void MainWindowImp::removeProfile(){
   if(profileToRemove == currentProfileLabel->text()){

-    QMessageBox::information(this, "Can't remove.",QString("%1 is the current profile.").arg(profileToRemove), "Ok");

+    QMessageBox::information(this, "Can't remove.",QString("%1 is the current profile.").arg(profileToRemove), QMessageBox::Ok);

     return;

@@ -509,3 +509,3 @@ void MainWindowImp::changeProfile(){
   if(profilesList->currentItem() == -1){

-    QMessageBox::information(this, "Can't Change.","Please select a profile.", "Ok");

+    QMessageBox::information(this, "Can't Change.","Please select a profile.", QMessageBox::Ok);

     return;