fixed an error in storing macs with zeroes.

[woller] / woller.cpp

diff --git a/woller.cpp b/woller.cpp
index 529835d..d78162c 100644 (file)
--- a/woller.cpp
+++ b/woller.cpp
@@ -23,10 +23,10 @@
 
 MainWindow::MainWindow()
 {
-//#ifdef Q_WS_MAEMO_5
-//    this->setAttribute(Qt::WA_Maemo5StackedWindow);
-//#endif
-    woller = new Woller;
+#ifdef Q_WS_MAEMO_5
+    this->setAttribute(Qt::WA_Maemo5StackedWindow);
+#endif
+    woller = new Woller(this);
     this->setCentralWidget(woller);
 }
 
@@ -132,7 +132,7 @@ void Woller::targets_act(int i)
     {
         // build the config widget
         qDebug() << "building the config" << endl;
-        config_win = new ConfigWidget(0, hosts_list);
+        config_win = new ConfigWidget(this, hosts_list);
         connect(config_win, SIGNAL(hosts_changed()), this, SLOT(hosts_changed()));
         config_win->show();
     }
@@ -210,10 +210,10 @@ ConfigWidget::ConfigWidget(QWidget *parent, QList<host_s> *hosts)
     this->setWindowTitle(tr("Woller Config"));
 #ifdef Q_WS_MAEMO_5
     this->setAttribute(Qt::WA_Maemo5StackedWindow);
-    this->setWindowFlags(this->windowFlags() | Qt::Window);
 #else
     this->setWindowModality(Qt::ApplicationModal);
 #endif
+    this->setWindowFlags(this->windowFlags() | Qt::Window);
     add_host = new QPushButton(tr("&Add Host"), this);
     edit_host = new QPushButton(tr("&Edit Host"), this);
     del_host = new QPushButton(tr("&Delete Host"), this);
@@ -269,7 +269,7 @@ void ConfigWidget::add_sig()
     host_tmp.hostname.clear();
     host_tmp.mac.clear();
     host_tmp.ip.clear();
-    host_widget = new HostWidget(0, &host_tmp);
+    host_widget = new HostWidget(this, &host_tmp);
 
     connect(host_widget, SIGNAL(change_host()), this, SLOT(host_added()));
     host_widget->show();
@@ -283,7 +283,7 @@ void ConfigWidget::edit_sig()
     host_tmp.hostname = hosts_list->at(list->currentIndex().row()).hostname;
     host_tmp.mac = hosts_list->at(list->currentIndex().row()).mac;
     host_tmp.ip = hosts_list->at(list->currentIndex().row()).ip;
-    host_widget = new HostWidget(0, &host_tmp);
+    host_widget = new HostWidget(this, &host_tmp);
     connect(host_widget, SIGNAL(change_host()), this, SLOT(host_edited()));
 
     //hostWidget->hostname->setPlaceholderText("my_host_name");
@@ -365,13 +365,13 @@ HostWidget::HostWidget(QWidget *parent, host_s *host)
 
     qDebug() << "host widget" << endl;
     new_host = host;
-    this->setWindowTitle("Add new WOL target host");
+    this->setWindowTitle(tr("WOL target details"));
 #ifdef Q_WS_MAEMO_5
     this->setAttribute(Qt::WA_Maemo5StackedWindow);
-    this->setWindowFlags(this->windowFlags() | Qt::Window);
 #else
     this->setWindowModality(Qt::ApplicationModal);
 #endif
+    this->setWindowFlags(this->windowFlags() | Qt::Window);
     vlayout = new QVBoxLayout(this);
 
     host_row = new QHBoxLayout;
@@ -389,7 +389,7 @@ HostWidget::HostWidget(QWidget *parent, host_s *host)
             tr("HW address of target, e.g. 00:01:02:AA:BB:CC. "
                "Hint, in Linux/Unix: \"ifconfig\","
                "in Windows \"ipconfig /all\""));
-    mac->setInputMask("HH:HH:HH:HH:HH:HH;0");
+    mac->setInputMask("HH:HH:HH:HH:HH:HH;_");
     mac_row->addWidget(mac_lbl);
     mac_row->addWidget(mac);
     vlayout->addLayout(mac_row);
@@ -442,11 +442,13 @@ void HostWidget::ok_sig()
         msgBox.exec();
         return;
     }
-    new_host->mac= this->mac->text();
-    if (new_host->mac.isEmpty())
+
+    new_host->mac = this->mac->text();
+    if (new_host->mac.length() != MAX_MAC_STR_LEN)
     {
         QMessageBox msgBox;
-        msgBox.setText("You must set mac!");
+        msgBox.setText("You must set mac to full 12 digits!\n" \
+                       "E.g. 00:11:22:33:44:55:66");
         msgBox.exec();
         return;
     }
@@ -462,7 +464,6 @@ void HostWidget::ok_sig()
 HostWidget::~HostWidget()
 {
     qDebug() << "host ~widget" << endl;
-    delete vlayout;
     delete host_row;
     delete host_lbl;
     delete hostname;
@@ -475,7 +476,7 @@ HostWidget::~HostWidget()
     delete button_row;
     delete cancel;
     delete ok;
-    //delete new_host;
+    delete vlayout;
 }