first revision of the Blueping configuration widget, which asks simply for a MAC address. It follows the style of e.g. NoticeConfigWidget

5 files changed, 150 insertions, 12 deletions

diff --git a/noncore/securityplugins/blueping/bluepingConfigWidget.cpp b/noncore/securityplugins/blueping/bluepingConfigWidget.cpp
new file mode 100644
index 0000000..876ccda
--- a/dev/null
+++ b/noncore/securityplugins/blueping/bluepingConfigWidget.cpp
@@ -0,0 +1,74 @@
+#include "bluepingConfigWidget.h"
+
+#include <qwidget.h>
+#include <qlayout.h>
+#include <qlabel.h>
+#include <qgroupbox.h>
+#include <qmessagebox.h>
+#include <qregexp.h>
+
+using Opie::Security::MultiauthConfigWidget;
+
+BluepingConfigWidget::BluepingConfigWidget(QWidget* parent = 0, const char* name = "Blueping configuration widget")
+: MultiauthConfigWidget(parent, name)
+{
+    m_config = new Config("Security");
+    m_config->setGroup("BluepingPlugin");
+    QVBoxLayout * baseLayout = new QVBoxLayout( this);
+    baseLayout->setSpacing(11);
+    baseLayout->setMargin(11);
+    baseLayout->setAlignment( Qt::AlignTop );
+
+    QGroupBox * configBox = new QGroupBox(0, Qt::Vertical, tr("Set the MAC address to ping here"), this);
+    baseLayout->addWidget(configBox);
+    QVBoxLayout *boxLayout = new QVBoxLayout( configBox->layout() );
+
+    QHBoxLayout * configLayout = new QHBoxLayout();
+    configLayout->setSpacing(6);
+    boxLayout->addLayout(configLayout);
+
+    QString mac = m_config->readEntry("mac");
+    if ( mac.isEmpty() )
+        mac = "00:00:00:00:00:00";
+
+    editMAC = new QLineEdit( mac, configBox, "editMAC" );
+    setMAC = new QPushButton( tr("Set"), configBox, "setMAC" );
+    configLayout->addWidget(editMAC);
+    configLayout->addWidget(setMAC);
+
+    QLabel * description = new QLabel( "<p>" + tr("Detecting another Bluetooth device by its MAC address provides a minimal and automatic level of protection.") + "</p><p><em>"
+                                       + tr("Note: if you don't put a valid MAC here but you activate the plugin, it will always succeed!") + "</em></p>", configBox );
+    boxLayout->addWidget(description);
+
+
+    connect(setMAC, SIGNAL( clicked() ), this, SLOT( changeMAC() ));
+
+}
+
+/// checks and writes the MAC in the config file, if its format is OK
+void BluepingConfigWidget::changeMAC() {
+    QString mac = editMAC->text();
+    QRegExp macPattern("[0-9A-Fa-f][0-9A-Fa-f]:[0-9A-Fa-f][0-9A-Fa-f]:[0-9A-Fa-f][0-9A-Fa-f]:[0-9A-Fa-f][0-9A-Fa-f]:[0-9A-Fa-f][0-9A-Fa-f]:[0-9A-Fa-f][0-9A-Fa-f]");
+    if ( (mac.length() == 17) && (macPattern.match(mac) == 0) )
+    {
+        m_config->writeEntry("mac", mac);
+        QMessageBox success( tr("MAC address saved!"), "<p>" + tr("Make sure that Bluetooth is turned on on the corresponding device when the Blueping plugin needs it.") + "</p>",
+                             QMessageBox::Information, QMessageBox::Ok, QMessageBox::NoButton, QMessageBox::NoButton );
+        success.exec();
+    } else {
+        QMessageBox failure( tr("Please enter a valid MAC"), "<p>" + tr("Please separate the six pairs of digits of your MAC like this: 01:02:03:04:05:06") + tr("</p>"),
+                             QMessageBox::Warning, QMessageBox::Ok, QMessageBox::NoButton, QMessageBox::NoButton );
+        failure.exec();
+    }
+}
+
+/// deletes the m_config pointer
+BluepingConfigWidget::~BluepingConfigWidget()
+{
+    delete m_config;
+}
+
+// does nothing (there's a button to save the config)
+void BluepingConfigWidget::writeConfig()
+{
+}
diff --git a/noncore/securityplugins/blueping/bluepingConfigWidget.h b/noncore/securityplugins/blueping/bluepingConfigWidget.h
new file mode 100644
index 0000000..bd1bec9
--- a/dev/null
+++ b/noncore/securityplugins/blueping/bluepingConfigWidget.h
@@ -0,0 +1,59 @@
+/**
+ * \file bluepingConfigWidget.h
+ * \brief "BluePing" plugin configuration widget
+ * \author Clément Séveillac (clement . seveillac (at) via . ecp . fr)
+ */
+/*
+               =.            This file is part of the Opie Project
+             .=l.            Copyright (C) 2004 Opie Developer Team <opie-devel@handhelds.org>
+           .>+-=
+ _;:,     .>    :=|.         This library is free software; you can
+.> <`_,   >  .   <=          redistribute it and/or  modify it under
+:`=1 )Y*s>-.--   :           the terms of the GNU Library General Public
+.="- .-=="i,     .._         License as published by the Free Software
+ - .   .-<_>     .<>         Foundation; either version 2 of the License,
+     ._= =}       :          or (at your option) any later version.
+    .%`+i>       _;_.
+    .i_,=:_.      -<s.       This library is distributed in the hope that
+     +  .  -:.       =       it will be useful,  but WITHOUT ANY WARRANTY;
+    : ..    .:,     . . .    without even the implied warranty of
+    =_        +     =;=|`    MERCHANTABILITY or FITNESS FOR A
+  _.=:.       :    :=>`:     PARTICULAR PURPOSE. See the GNU
+..}^=.=       =       ;      Library General Public License for more
+++=   -.     .`     .:       details.
+ :     =  ...= . :.=-
+ -.   .:....=;==+<;          You should have received a copy of the GNU
+  -_. . .   )=.  =           Library General Public License along with
+    --        :-=`           this library; see the file COPYING.LIB.
+                             If not, write to the Free Software Foundation,
+                             Inc., 59 Temple Place - Suite 330,
+                             Boston, MA 02111-1307, USA.
+
+*/
+
+#ifndef BLUEPINGCONFIGWIDGET_H
+#define BLUEPINGCONFIGWIDGET_H
+#include <qlineedit.h>
+#include <qpushbutton.h>
+#include <qpe/config.h>
+
+#include <opie2/multiauthconfigwidget.h>
+
+class BluepingConfigWidget : public Opie::Security::MultiauthConfigWidget {
+
+    Q_OBJECT
+
+public:
+    BluepingConfigWidget(QWidget* parent, const char* name);
+    virtual ~BluepingConfigWidget();
+    virtual void writeConfig();
+private:
+    QLineEdit *editMAC;
+    QPushButton *setMAC;
+    Config *m_config;
+private slots:
+    void changeMAC();
+
+};
+
+#endif // BLUEPINGCONFIGWIDGET_H
diff --git a/noncore/securityplugins/blueping/bluepingplugin.cpp b/noncore/securityplugins/blueping/bluepingplugin.cpp
index 05fd3c2..f5d9ad6 100644
--- a/noncore/securityplugins/blueping/bluepingplugin.cpp
+++ b/noncore/securityplugins/blueping/bluepingplugin.cpp
@@ -15,35 +15,36 @@ using namespace Opie::Core;
 using Opie::Security::MultiauthPluginObject;
 using Opie::Security::MultiauthConfigWidget;
 
 
-/// creates and initializes the m_config Config object
-BluepingPlugin::BluepingPlugin() : MultiauthPluginObject(), m_ping(0) {
-    m_config = new Config("Security");
-    m_config->setGroup("BluepingPlugin");
+/// standard c'tor
+BluepingPlugin::BluepingPlugin() : MultiauthPluginObject(), m_ping(0), m_bluepingW(0) {
     bluetoothAlreadyRestarted = false;
 }
 
-/// deletes the m_config Config object and noticeW if necessary
+/// cleans m_ping if we need to
 BluepingPlugin::~BluepingPlugin() {
-    delete m_config;
     delete m_ping;
+    if (m_bluepingW != 0)
+        delete m_bluepingW;
 }
 
 /// Simply return its name (Blueping plugin)
 QString BluepingPlugin::pluginName() const {
     return "Blueping plugin";
 }
 
-/// no configuration widget for the moment
+/// returns a BluepingConfigWidget
 MultiauthConfigWidget * BluepingPlugin::configWidget(QWidget * parent) {
-    return 0l;
+    if (m_bluepingW == 0)
+        m_bluepingW = new BluepingConfigWidget(parent, "Blueping configuration widget");
+    return m_bluepingW;
 }
 QString BluepingPlugin::pixmapNameWidget() const {
     return "security/bluepingplugin";
 }
 QString BluepingPlugin::pixmapNameConfig() const {
-    return 0l;
+    return "security/bluepingplugin";
 }
 
 /// Emit the MultiauthPluginObject::Success emitCode
 void BluepingPlugin::success() {
@@ -134,8 +135,10 @@ int BluepingPlugin::authenticate() {
         QObject::connect(&pbSkip, SIGNAL(clicked()), this, SLOT(skip()));
         // connect the signal emitting functions to the bluepingDialog done(int) finishing function
         QObject::connect(this, SIGNAL(emitCode(int)), &bluepingDialog, SLOT(done(int)));
 
+        // we can uncomment the following when testing
+        //bluetoothAlreadyRestarted = true;
         if (!bluetoothAlreadyRestarted)
         {
             // we have just started or resumed the device, so Bluetooth has to be (re)started
             OProcess killB;
diff --git a/noncore/securityplugins/blueping/bluepingplugin.h b/noncore/securityplugins/blueping/bluepingplugin.h
index 0df3f78..b8d6734 100644
--- a/noncore/securityplugins/blueping/bluepingplugin.h
+++ b/noncore/securityplugins/blueping/bluepingplugin.h
@@ -33,8 +33,10 @@
 
 #ifndef BLUEPING_PLUGIN_H
 #define BLUEPING_PLUGIN_H
 
+#include "bluepingConfigWidget.h"
+
 #include <opie2/multiauthplugininterface.h>
 
 #include <opie2/oprocess.h>
 
@@ -71,9 +73,9 @@ class BluepingPlugin : public QObject, public Opie::Security::MultiauthPluginObj
         void pingFinished(Opie::Core::OProcess * ping);
         
      private:
         Opie::Core::OProcess *m_ping;
-        Config * m_config;
+        BluepingConfigWidget * m_bluepingW;
         bool bluetoothAlreadyRestarted;
         QString macToPing;
 
 };
diff --git a/noncore/securityplugins/blueping/bluepingplugin.pro b/noncore/securityplugins/blueping/bluepingplugin.pro
index 2cc0cd3..0946749 100644
--- a/noncore/securityplugins/blueping/bluepingplugin.pro
+++ b/noncore/securityplugins/blueping/bluepingplugin.pro
@@ -1,10 +1,10 @@
 TEMPLATE = lib
 CONFIG += qt warn_on
 
-HEADERS		= bluepingplugin.h bluepingpluginimpl.h
+HEADERS		= bluepingplugin.h bluepingpluginimpl.h bluepingConfigWidget.h
 
-SOURCES		= bluepingplugin.cpp bluepingpluginimpl.cpp
+SOURCES		= bluepingplugin.cpp bluepingpluginimpl.cpp bluepingConfigWidget.cpp
 
 
 INCLUDEPATH	+= $(OPIEDIR)/include
 DEPENDPATH	+=