Added: Default net-ranges are no longer hard-coded and can be edited

Bugfix: Show user-defined net-range upon application restart in
        drop-down list

3 files changed, 228 insertions, 81 deletions

diff --git a/core/settings/security/security.cpp b/core/settings/security/security.cpp
index 1b90121..d7871b3 100644
--- a/core/settings/security/security.cpp
+++ b/core/settings/security/security.cpp
@@ -36,3 +36,3 @@
     Security::Security( QWidget* parent,  const char* name, WFlags fl )
-: SecurityBase( parent, name, TRUE, fl )
+: SecurityBase( parent, name, TRUE, WStyle_ContextHelp )
 {
@@ -46,3 +46,4 @@
     int auth_peer_bits = cfg.readNumEntry("auth_peer_bits",24);
-    selectNet(auth_peer,auth_peer_bits);
+    selectNet(auth_peer,auth_peer_bits,TRUE);
+  
     connect(syncnet, SIGNAL(textChanged(const QString&)),
@@ -53,3 +54,2 @@
 
-    //There must be a better way to do that...
     for (int i=0; i<syncapp->count(); i++) {
@@ -88,2 +88,6 @@
     syncModeCombo->setCurrentItem( mode - 1 );
+    
+    //since nobody knows what this is and it doesn't do anything, i'll hide it # CoreDump
+    // is this work-in-progress or can it be removed?
+    syncModeCombo->hide();
 
@@ -94,2 +98,4 @@
     connect(syncapp,SIGNAL(activated(int)), this, SLOT(changeSyncApp()));
+    connect(restoredefaults,SIGNAL(clicked()), this, SLOT(restoreDefaults()));    
+    connect(deleteentry,SIGNAL(clicked()), this, SLOT(deleteListEntry()));
     
@@ -106,2 +112,36 @@ Security::~Security()
 
+void Security::deleteListEntry() 
+{
+    syncnet->removeItem(syncnet->currentItem());
+}
+
+void Security::restoreDefaults()
+{		
+    QMessageBox unrecbox(
+    tr("Attention"),
+    tr(	"<p>All user-defined net ranges will be lost."),
+    QMessageBox::Warning,
+    QMessageBox::Cancel, QMessageBox::Yes, QMessageBox::NoButton,
+    0, QString::null, TRUE, WStyle_StaysOnTop);
+    unrecbox.setButtonText(QMessageBox::Cancel, tr("Cancel"));
+    unrecbox.setButtonText(QMessageBox::Yes, tr("Ok"));
+
+    if ( unrecbox.exec() == QMessageBox::Yes)	
+    {
+        syncnet->clear();
+        insertDefaultRanges();
+    }	
+}
+
+void Security::insertDefaultRanges()
+{    
+    syncnet->insertItem( tr( "192.168.129.0/24" ) );
+    syncnet->insertItem( tr( "192.168.1.0/24" ) );
+    syncnet->insertItem( tr( "192.168.0.0/16" ) );
+    syncnet->insertItem( tr( "172.16.0.0/12" ) );
+    syncnet->insertItem( tr( "10.0.0.0/8" ) );
+    syncnet->insertItem( tr( "1.0.0.0/8" ) );
+    syncnet->insertItem( tr( "Any" ) );
+    syncnet->insertItem( tr( "None" ) );
+}
 
@@ -160,3 +200,3 @@ void Security::done(int r)
 
-void Security::selectNet(int auth_peer,int auth_peer_bits)
+void Security::selectNet(int auth_peer,int auth_peer_bits, bool update)
 {
@@ -175,2 +215,40 @@ void Security::selectNet(int auth_peer,int auth_peer_bits)
     }
+
+    //insert user-defined list of netranges upon start
+    if (update) {
+	//User selected/active netrange first
+	syncnet->insertItem( tr( sn ) );
+
+	Config cfg("Security");
+	cfg.setGroup("Sync");
+
+	//set up defaults if needed, if someone manually deletes net0 he'll get a suprise hehe
+	QString test = cfg.readEntry("net0","");
+	if (test.isEmpty()) {
+            insertDefaultRanges();
+	} else {
+            // 10 ought to be enough for everybody... :)
+            // If you need more, don't forget to edit applySecurity() as well
+            bool already_there=FALSE;
+            for (int i=0; i<10; i++) {
+                QString target, netrange;
+                target.sprintf("net%d", i);
+                netrange = cfg.readEntry(target,"");
+                if (! netrange.isEmpty()){
+                    //make sure we have no "twin" entries
+                    for (int i=0; i<syncnet->count(); i++) {
+                        if ( syncnet->text(i) == netrange ) {
+                            already_there=TRUE;      	    
+                        }
+                    }
+                    if (! already_there) {
+                        syncnet->insertItem( tr( netrange ) );
+                    } else {
+                        already_there=FALSE;
+                    }		
+                }	
+            }
+        }	
+    } 
+     
     for (int i=0; i<syncnet->count(); i++) {
@@ -244,3 +322,3 @@ void Security::setSyncNet(const QString& sn)
     parseNet(sn,auth_peer,auth_peer_bits);
-    selectNet(auth_peer,auth_peer_bits);
+    selectNet(auth_peer,auth_peer_bits,FALSE);
 }
@@ -259,4 +337,14 @@ void Security::applySecurity()
         parseNet(sn,auth_peer,auth_peer_bits);
+       
+        //this is the *selected* (active) net range
         cfg.writeEntry("auth_peer",auth_peer);
-        cfg.writeEntry("auth_peer_bits",auth_peer_bits);	
+        cfg.writeEntry("auth_peer_bits",auth_peer_bits);
+	
+	//write back all other net ranges in *cleartext*
+	for (int i=0; i<10; i++) {
+		QString target;
+		target.sprintf("net%d", i);
+        	cfg.writeEntry(target,syncnet->text(i));
+	}
+		
         cfg.writeEntry("syncapp",syncapp->currentText());
diff --git a/core/settings/security/security.h b/core/settings/security/security.h
index b1a3eca..72aa352 100644
--- a/core/settings/security/security.h
+++ b/core/settings/security/security.h
@@ -49,3 +49,5 @@ private slots:
     void changeSyncApp();
-
+    void restoreDefaults();
+    void insertDefaultRanges();
+    void deleteListEntry();
 
@@ -58,3 +60,4 @@ private:
     static void parseNet(const QString& sn,int& auth_peer,int& auth_peer_bits);
-    void selectNet(int auth_peer,int auth_peer_bits);
+    void selectNet(int auth_peer,int auth_peer_bits,bool update);
+
     
diff --git a/core/settings/security/securitybase.ui b/core/settings/security/securitybase.ui
index 934111e..1769ae1 100644
--- a/core/settings/security/securitybase.ui
+++ b/core/settings/security/securitybase.ui
@@ -13,3 +13,3 @@
             <y>0</y>
-            <width>321</width>
+            <width>309</width>
             <height>483</height>
@@ -111,2 +111,8 @@
                                         </property>
+                                        <property>
+                                            <name>whatsThis</name>
+                                            <string>This button will let you change the security passcode.  
+
+Note: This is *not* the sync password.</string>
+                                        </property>
                                     </widget>
@@ -122,2 +128,7 @@
                                         </property>
+                                        <property>
+                                            <name>whatsThis</name>
+                                            <string>Delete the current passcode.  
+You can enter a new one at any time.</string>
+                                        </property>
                                     </widget>
@@ -135,2 +146,6 @@
                                 </property>
+                                <property>
+                                    <name>whatsThis</name>
+                                    <string>If you enable this option, a passcode must be entered after every resume event of your Zaurus.</string>
+                                </property>
                             </widget>
@@ -238,2 +253,8 @@
                                 </property>
+                                <property>
+                                    <name>whatsThis</name>
+                                    <string>If you select this option, opie-login will automagically log you in with the user you have selected in the drop-down list.  
+
+For the Zaurus 5x00 this is always the user "root".</string>
+                                </property>
                             </widget>
@@ -245,2 +266,9 @@
                                 </property>
+                                <property>
+                                    <name>whatsThis</name>
+                                    <string>This drop-down list allows you to select the user for auto-login
+(if enabled above).  
+
+You can only select an actually configured user.</string>
+                                </property>
                             </widget>
@@ -281,3 +309,3 @@
                 </attribute>
-                <vbox>
+                <grid>
                     <property stdset="1">
@@ -290,3 +318,24 @@
                     </property>
-                    <widget>
+                    <spacer row="1"  column="0" >
+                        <property>
+                            <name>name</name>
+                            <cstring>Spacer1</cstring>
+                        </property>
+                        <property stdset="1">
+                            <name>orientation</name>
+                            <enum>Vertical</enum>
+                        </property>
+                        <property stdset="1">
+                            <name>sizeType</name>
+                            <enum>Expanding</enum>
+                        </property>
+                        <property>
+                            <name>sizeHint</name>
+                            <size>
+                                <width>20</width>
+                                <height>20</height>
+                            </size>
+                        </property>
+                    </spacer>
+                    <widget row="0"  column="0" >
                         <class>QGroupBox</class>
@@ -326,50 +375,2 @@
                                 <class>QComboBox</class>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>192.168.129.0/24 (default)</string>
-                                    </property>
-                                </item>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>192.168.1.0/24</string>
-                                    </property>
-                                </item>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>192.168.0.0/16</string>
-                                    </property>
-                                </item>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>172.16.0.0/12</string>
-                                    </property>
-                                </item>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>10.0.0.0/8</string>
-                                    </property>
-                                </item>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>1.0.0.0/8</string>
-                                    </property>
-                                </item>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>Any</string>
-                                    </property>
-                                </item>
-                                <item>
-                                    <property>
-                                        <name>text</name>
-                                        <string>None</string>
-                                    </property>
-                                </item>
                                 <property stdset="1">
@@ -382,2 +383,70 @@
                                 </property>
+                                <property>
+                                    <name>whatsThis</name>
+                                    <string>Select a net-range or enter a new one.  
+
+This drop-down list lets you select a net range which is allowed to connect to your Zaurus.  
+
+For example, a net range of 192.168.1.0/24 will allow any machine with an IP between 192.168.1.1 and 192.168.1.254 to connect to (and sync with) your Zaurus.  
+
+The entry "All" will allow *any* IP to connect. 
+The entry "None" will *deny* any connection.  
+
+If unsure, select "Any".</string>
+                                </property>
+                            </widget>
+                            <widget>
+                                <class>QLayoutWidget</class>
+                                <property stdset="1">
+                                    <name>name</name>
+                                    <cstring>Layout4</cstring>
+                                </property>
+                                <hbox>
+                                    <property stdset="1">
+                                        <name>margin</name>
+                                        <number>0</number>
+                                    </property>
+                                    <property stdset="1">
+                                        <name>spacing</name>
+                                        <number>6</number>
+                                    </property>
+                                    <widget>
+                                        <class>QPushButton</class>
+                                        <property stdset="1">
+                                            <name>name</name>
+                                            <cstring>deleteentry</cstring>
+                                        </property>
+                                        <property stdset="1">
+                                            <name>text</name>
+                                            <string>Delete Entry</string>
+                                        </property>
+                                        <property>
+                                            <name>whatsThis</name>
+                                            <string>Delete the selected net range from the list
+
+If you press this button, the currently selected net range will be deleted from the list of net ranges.
+
+If you hit this button in error, the button "Restore Defaults" will restore
+the list to the state it has been when you initially booted your Zaurus.</string>
+                                        </property>
+                                    </widget>
+                                    <widget>
+                                        <class>QPushButton</class>
+                                        <property stdset="1">
+                                            <name>name</name>
+                                            <cstring>restoredefaults</cstring>
+                                        </property>
+                                        <property stdset="1">
+                                            <name>text</name>
+                                            <string>Restore Defaults</string>
+                                        </property>
+                                        <property>
+                                            <name>whatsThis</name>
+                                            <string>This button will restore the list of net ranges
+to the defaults.
+
+Beware that all manually entered net ranges will be lost!</string>
+                                        </property>
+                                    </widget>
+                                </hbox>
                             </widget>
@@ -412,2 +481,10 @@
                                 </property>
+                                <property>
+                                    <name>whatsThis</name>
+                                    <string>This option changes the login behaviour of the internal FTP server.  
+
+If you have problems syncing, try the "IntelliSync" setting, which will accept any password you throw at the sync (FTP) port.  
+
+The default is Qtopia. It will let you log in with username "root" and password "Qtopia".   The password is CaSeSeNsItIvE.</string>
+                                </property>
                             </widget>
@@ -440,24 +517,3 @@
                     </widget>
-                    <spacer>
-                        <property>
-                            <name>name</name>
-                            <cstring>Spacer1</cstring>
-                        </property>
-                        <property stdset="1">
-                            <name>orientation</name>
-                            <enum>Vertical</enum>
-                        </property>
-                        <property stdset="1">
-                            <name>sizeType</name>
-                            <enum>Expanding</enum>
-                        </property>
-                        <property>
-                            <name>sizeHint</name>
-                            <size>
-                                <width>20</width>
-                                <height>20</height>
-                            </size>
-                        </property>
-                    </spacer>
-                </vbox>
+                </grid>
             </widget>