summaryrefslogtreecommitdiff
authorclem <clem>2004-09-30 15:11:21 (UTC)
committer clem <clem>2004-09-30 15:11:21 (UTC)
commita823eb5b367c32936a9958f69a9981a55e2022bf (patch) (side-by-side diff)
tree96c61ec870bd72f53047439eb9c610207e510f6a
parent5cb30454d4a974bacfae126a5b8b321bae8ca600 (diff)
downloadopie-a823eb5b367c32936a9958f69a9981a55e2022bf.zip
opie-a823eb5b367c32936a9958f69a9981a55e2022bf.tar.gz
opie-a823eb5b367c32936a9958f69a9981a55e2022bf.tar.bz2
- As Zecke suggested, we now hide the plugin list and authentication
configuration tabs if no authentication plugin is installed (but we tell the user a few things about authentication in a third tab). - Small warning about syncnet removed.
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--core/settings/security/multiauthconfig.cpp264
-rw-r--r--core/settings/security/multiauthconfig.h1
2 files changed, 158 insertions, 107 deletions
diff --git a/core/settings/security/multiauthconfig.cpp b/core/settings/security/multiauthconfig.cpp
index ff11e49..3c73d7b 100644
--- a/core/settings/security/multiauthconfig.cpp
+++ b/core/settings/security/multiauthconfig.cpp
@@ -5,2 +5,3 @@
#include <qgroupbox.h>
+#include <qvgroupbox.h>
#include <qpe/resource.h>
@@ -57,4 +58,4 @@ class ToolButton : public QToolButton {
-MultiauthGeneralConfig::MultiauthGeneralConfig(QWidget * parent, const char * name = "general Opie-multiauthentication config widget")
- : QWidget(parent, name), onStart(0), onResume(0), nbSuccessMin(0)
+ MultiauthGeneralConfig::MultiauthGeneralConfig(QWidget * parent, const char * name = "general Opie-multiauthentication config widget")
+: QWidget(parent, name), onStart(0), onResume(0), nbSuccessMin(0)
{
@@ -133,3 +134,3 @@ static void test_and_start() {
if (protectConfigDialog && Opie::Security::Internal::runPlugins() != 0) {
- owarn << "authentication failed, not showing opie-security" << oendl;
+ owarn << "authentication failed, not showing opie-security" << oendl;
exit( -1 );
@@ -140,12 +141,23 @@ static void test_and_start() {
-MultiauthConfig::MultiauthConfig(QWidget* par, const char* w, WFlags f)
- : QDialog(par, w, TRUE, f),
- m_mainTW(0), m_pluginListView(0), m_pluginListWidget(0),
- m_generalConfig(0), m_loginWidget(0), m_syncWidget(0),
- m_nbSuccessReq(0), m_plugins_changed(false)
+ MultiauthConfig::MultiauthConfig(QWidget* par, const char* w, WFlags f)
+: QDialog(par, w, TRUE, f),
+ m_mainTW(0), m_pluginListView(0), m_pluginListWidget(0),
+ m_generalConfig(0), m_loginWidget(0), m_syncWidget(0),
+ m_nbSuccessReq(0), m_plugins_changed(false)
{
/* Initializes the global configuration window
- */
+ */
test_and_start();
+ /* Checks (and memorizes) if any authentication plugins are
+ * installed on the system
+ */
+ QString path = QPEApplication::qpeDir() + "/plugins/security";
+ QDir dir( path, "lib*.so" );
+ QStringList list = dir.entryList();
+
+ m_pluginsInstalled = ! list.isEmpty();
+ if (m_pluginsInstalled == false)
+ owarn << "no authentication plugins installed! Talking about it in the last tab..." << oendl;
+
setCaption( tr( "Security configuration" ) );
@@ -154,28 +166,32 @@ MultiauthConfig::MultiauthConfig(QWidget* par, const char* w, WFlags f)
layout->addWidget(m_mainTW);
- m_pluginListWidget = new QWidget(m_mainTW, "plugin list widget");
- QVBoxLayout * pluginListLayout = new QVBoxLayout(m_pluginListWidget);
- pluginListLayout->setSpacing(6);
- pluginListLayout->setMargin(11);
- QLabel * pluginListTitle = new QLabel( tr( "Load which plugins in what order:" ), m_pluginListWidget );
- pluginListLayout->addWidget(pluginListTitle);
- QHBox * pluginListHB = new QHBox(m_pluginListWidget);
- pluginListLayout->addWidget(pluginListHB);
-
- m_pluginListView = new QListView(pluginListHB);
- m_pluginListView->addColumn("PluginList");
- m_pluginListView->header()->hide();
- m_pluginListView->setSorting(-1);
- QWhatsThis::add(m_pluginListView, tr( "Check a checkbox to activate/deactivate a plugin or use the arrow buttons on the right to change the order they will appear in" ));
-
- QVBox * pluginListVB = new QVBox(pluginListHB);
- new ToolButton( pluginListVB, tr( "Move Up" ), "up", this , SLOT( moveSelectedUp() ) );
- new ToolButton( pluginListVB, tr( "Move Down" ), "down", this , SLOT( moveSelectedDown() ) );
- m_mainTW->addTab( m_pluginListWidget, "pass", tr( "plugins" ) );
-
- connect ( m_pluginListView , SIGNAL( clicked ( QListViewItem * ) ), this, SLOT( pluginsChanged ( ) ) );
-
- // general Opie multi-authentication configuration tab
- m_generalConfig = new MultiauthGeneralConfig(m_mainTW);
- m_mainTW->addTab(m_generalConfig, "SettingsIcon", tr( "Authentication") );
+ if (m_pluginsInstalled)
+ {
+ m_pluginListWidget = new QWidget(m_mainTW, "plugin list widget");
+ QVBoxLayout * pluginListLayout = new QVBoxLayout(m_pluginListWidget);
+ pluginListLayout->setSpacing(6);
+ pluginListLayout->setMargin(11);
+ QLabel * pluginListTitle = new QLabel( tr( "Load which plugins in what order:" ), m_pluginListWidget );
+ pluginListLayout->addWidget(pluginListTitle);
+ QHBox * pluginListHB = new QHBox(m_pluginListWidget);
+ pluginListLayout->addWidget(pluginListHB);
+
+ m_pluginListView = new QListView(pluginListHB);
+ m_pluginListView->addColumn("PluginList");
+ m_pluginListView->header()->hide();
+ m_pluginListView->setSorting(-1);
+ QWhatsThis::add(m_pluginListView, tr( "Check a checkbox to activate/deactivate a plugin or use the arrow buttons on the right to change the order they will appear in" ));
+
+ QVBox * pluginListVB = new QVBox(pluginListHB);
+ new ToolButton( pluginListVB, tr( "Move Up" ), "up", this , SLOT( moveSelectedUp() ) );
+ new ToolButton( pluginListVB, tr( "Move Down" ), "down", this , SLOT( moveSelectedDown() ) );
+ m_mainTW->addTab( m_pluginListWidget, "pass", tr( "plugins" ) );
+
+ connect ( m_pluginListView , SIGNAL( clicked ( QListViewItem * ) ), this, SLOT( pluginsChanged ( ) ) );
+
+ // general Opie multi-authentication configuration tab
+ m_generalConfig = new MultiauthGeneralConfig(m_mainTW);
+ m_mainTW->addTab(m_generalConfig, "SettingsIcon", tr( "Authentication") );
+
+ }
// login settings page
@@ -191,36 +207,59 @@ MultiauthConfig::MultiauthConfig(QWidget* par, const char* w, WFlags f)
- /* loads plugins configuration widgets in mainTW tabs and in pluginListView
- */
-
- loadPlugins();
-
- for ( int i = pluginList.count() - 1; i >= 0; i-- ) {
- MultiauthPlugin plugin = pluginList[i];
- // load the config widgets in the tabs
- // (configWidget will return 0l if there is no configuration GUI)
- MultiauthConfigWidget* widget = plugin.pluginObject->configWidget(m_mainTW);
- if ( widget != 0l ) {
- odebug << "plugin " << plugin.name << " has a configuration widget" << oendl;
- configWidgetList.append(widget);
- m_mainTW->addTab( widget, plugin.pluginObject->pixmapNameConfig(),
- plugin.pluginObject->pluginName() );
- }
- // set the order/activate tab
- QPixmap icon = Resource::loadPixmap( plugin.pluginObject->pixmapNameWidget() );
- QCheckListItem * item = new QCheckListItem(m_pluginListView, plugin.pluginObject->pluginName(), QCheckListItem::CheckBox );
- if ( !icon.isNull() ) {
- item->setPixmap( 0, icon );
- }
- if ( m_excludePlugins.find( plugin.name ) == m_excludePlugins.end() ) {
- item->setOn( TRUE );
+ if (m_pluginsInstalled)
+ {
+ /* loads plugins configuration widgets in mainTW tabs and in pluginListView
+ */
+
+ loadPlugins();
+
+ for ( int i = pluginList.count() - 1; i >= 0; i-- ) {
+ MultiauthPlugin plugin = pluginList[i];
+
+ // load the config widgets in the tabs
+ // (configWidget will return 0l if there is no configuration GUI)
+ MultiauthConfigWidget* widget = plugin.pluginObject->configWidget(m_mainTW);
+ if ( widget != 0l ) {
+ odebug << "plugin " << plugin.name << " has a configuration widget" << oendl;
+ configWidgetList.append(widget);
+ m_mainTW->addTab( widget, plugin.pluginObject->pixmapNameConfig(),
+ plugin.pluginObject->pluginName() );
+ }
+ // set the order/activate tab
+ QPixmap icon = Resource::loadPixmap( plugin.pluginObject->pixmapNameWidget() );
+ QCheckListItem * item = new QCheckListItem(m_pluginListView, plugin.pluginObject->pluginName(), QCheckListItem::CheckBox );
+ if ( !icon.isNull() ) {
+ item->setPixmap( 0, icon );
+ }
+ if ( m_excludePlugins.find( plugin.name ) == m_excludePlugins.end() ) {
+ item->setOn( TRUE );
+ }
+ m_plugins[plugin.name] = item;
}
- m_plugins[plugin.name] = item;
- }
- // set the first tab as default.
- m_mainTW->setCurrentTab(m_pluginListWidget);
+ // set the first tab as default.
+ m_mainTW->setCurrentTab(m_pluginListWidget);
- // put the number of plugins as the max number of req. auth.
- m_generalConfig->nbSuccessMin->setMaxValue( pluginList.count() );
+ // put the number of plugins as the max number of req. auth.
+ m_generalConfig->nbSuccessMin->setMaxValue( pluginList.count() );
+ }
+ else
+ {
+ /* we don't have any installed plugin there. Let's tell
+ * that to the user in a third tab, using the m_pluginListWidget widget
+ */
+ m_pluginListWidget = new QWidget(m_mainTW, "plugin list widget (no plugins warning)");
+ QVBoxLayout * pluginListLayout = new QVBoxLayout(m_pluginListWidget);
+ pluginListLayout->setSpacing(11);
+ pluginListLayout->setMargin(11);
+ pluginListLayout->setAlignment( Qt::AlignTop );
+ QVGroupBox *warningBox = new QVGroupBox(tr("Important notice"), m_pluginListWidget, "noPlugins warning box");
+ pluginListLayout->addWidget(warningBox);
+ QLabel * warningText = new QLabel( "<p>" + tr("To be able to protect your PDA with one or more authentication plugins (for example, a simple PIN authentication), you must install at least one <em>opie-multiauth-*</em> package! Once you have done that, you will be able to configure your PDA protection here.") + "</p>", warningBox );
+
+ m_mainTW->addTab(m_pluginListWidget, "security/Security", tr( "Locking") );
+
+ // set the first tab as default.
+ m_mainTW->setCurrentTab(m_loginWidget);
+ }
@@ -273,13 +312,17 @@ void MultiauthConfig::readConfig()
Config* pcfg = new Config("Security");
- pcfg->setGroup( "Misc" );
- m_generalConfig->onStart->setChecked( pcfg->readBoolEntry( "onStart", false ) );
- m_generalConfig->onResume->setChecked( pcfg->readBoolEntry( "onResume", false ) );
- m_generalConfig->nbSuccessMin->setValue( pcfg->readNumEntry( "nbSuccessMin", 1 ) );
- m_generalConfig->noProtectConfig->setChecked( pcfg->readBoolEntry( "noProtectConfig", true) );
- m_generalConfig->explanScreens->setChecked( pcfg->readBoolEntry( "explanScreens", true ) );
- m_generalConfig->allowBypass->setChecked( pcfg->readBoolEntry( "allowBypass", false ) );
-
- pcfg->setGroup( "Plugins" );
- m_excludePlugins = pcfg->readListEntry( "ExcludePlugins", ',' );
- m_allPlugins = pcfg->readListEntry( "AllPlugins", ',' );
+
+ if (m_pluginsInstalled)
+ {
+ pcfg->setGroup( "Misc" );
+ m_generalConfig->onStart->setChecked( pcfg->readBoolEntry( "onStart", false ) );
+ m_generalConfig->onResume->setChecked( pcfg->readBoolEntry( "onResume", false ) );
+ m_generalConfig->nbSuccessMin->setValue( pcfg->readNumEntry( "nbSuccessMin", 1 ) );
+ m_generalConfig->noProtectConfig->setChecked( pcfg->readBoolEntry( "noProtectConfig", true) );
+ m_generalConfig->explanScreens->setChecked( pcfg->readBoolEntry( "explanScreens", true ) );
+ m_generalConfig->allowBypass->setChecked( pcfg->readBoolEntry( "allowBypass", false ) );
+
+ pcfg->setGroup( "Plugins" );
+ m_excludePlugins = pcfg->readListEntry( "ExcludePlugins", ',' );
+ m_allPlugins = pcfg->readListEntry( "AllPlugins", ',' );
+ }
@@ -315,3 +358,3 @@ void MultiauthConfig::readConfig()
ssh->hide();
- */
+ */
@@ -353,34 +396,38 @@ void MultiauthConfig::writeConfig()
Config* pcfg = new Config("Security");
- pcfg->setGroup( "Plugins" );
- QStringList exclude;
- QStringList include;
- QStringList allPlugins;
-
- QListViewItemIterator list_it( m_pluginListView );
-
- // this makes sure the names get saved in the order selected
- for ( ; list_it.current(); ++list_it ) {
- QMap <QString, QCheckListItem *>::Iterator it;
- for ( it = m_plugins.begin(); it != m_plugins. end (); ++it ) {
- if ( list_it.current() == (*it) && !(*it)-> isOn () ) {
- exclude << it.key();
- } else if ( list_it.current() == (*it) && (*it)-> isOn () ){
- include << it.key();
- }
- if ( list_it.current() == (*it) ) {
- allPlugins << it.key();
+
+ if (m_pluginsInstalled)
+ {
+ pcfg->setGroup( "Plugins" );
+ QStringList exclude;
+ QStringList include;
+ QStringList allPlugins;
+
+ QListViewItemIterator list_it( m_pluginListView );
+
+ // this makes sure the names get saved in the order selected
+ for ( ; list_it.current(); ++list_it ) {
+ QMap <QString, QCheckListItem *>::Iterator it;
+ for ( it = m_plugins.begin(); it != m_plugins. end (); ++it ) {
+ if ( list_it.current() == (*it) && !(*it)-> isOn () ) {
+ exclude << it.key();
+ } else if ( list_it.current() == (*it) && (*it)-> isOn () ){
+ include << it.key();
+ }
+ if ( list_it.current() == (*it) ) {
+ allPlugins << it.key();
+ }
}
}
+ pcfg->writeEntry( "ExcludePlugins", exclude, ',' );
+ pcfg->writeEntry( "IncludePlugins", include, ',' );
+ pcfg->writeEntry( "AllPlugins", allPlugins, ',' );
+
+ pcfg->setGroup( "Misc" );
+ pcfg->writeEntry( "onStart", m_generalConfig->onStart->isChecked() );
+ pcfg->writeEntry( "onResume", m_generalConfig->onResume->isChecked() );
+ pcfg->writeEntry( "nbSuccessMin", m_generalConfig->nbSuccessMin->text() );
+ pcfg->writeEntry( "noProtectConfig", m_generalConfig->noProtectConfig->isChecked() );
+ pcfg->writeEntry( "explanScreens", m_generalConfig->explanScreens->isChecked() );
+ pcfg->writeEntry( "allowBypass", m_generalConfig->allowBypass->isChecked() );
}
- pcfg->writeEntry( "ExcludePlugins", exclude, ',' );
- pcfg->writeEntry( "IncludePlugins", include, ',' );
- pcfg->writeEntry( "AllPlugins", allPlugins, ',' );
-
- pcfg->setGroup( "Misc" );
- pcfg->writeEntry( "onStart", m_generalConfig->onStart->isChecked() );
- pcfg->writeEntry( "onResume", m_generalConfig->onResume->isChecked() );
- pcfg->writeEntry( "nbSuccessMin", m_generalConfig->nbSuccessMin->text() );
- pcfg->writeEntry( "noProtectConfig", m_generalConfig->noProtectConfig->isChecked() );
- pcfg->writeEntry( "explanScreens", m_generalConfig->explanScreens->isChecked() );
- pcfg->writeEntry( "allowBypass", m_generalConfig->allowBypass->isChecked() );
@@ -402,3 +449,6 @@ void MultiauthConfig::writeConfig()
target.sprintf("net%d", i);
- pcfg->writeEntry(target,m_syncWidget->syncnet->text(i));
+ if ( i < m_syncWidget->syncnet->count() )
+ pcfg->writeEntry(target, m_syncWidget->syncnet->text(i));
+ else // no more entry in the syncnet list -> we clear the line
+ pcfg->writeEntry(target, "");
}
@@ -431,3 +481,3 @@ void MultiauthConfig::writeConfig()
// ### write ssh/telnet sys config files
- */
+ */
diff --git a/core/settings/security/multiauthconfig.h b/core/settings/security/multiauthconfig.h
index 649815d..5287083 100644
--- a/core/settings/security/multiauthconfig.h
+++ b/core/settings/security/multiauthconfig.h
@@ -123,2 +123,3 @@ private:
bool m_plugins_changed;
+ bool m_pluginsInstalled;