What does happen if you store the whole config

inside the conf map?
yes on save custom data overwrites fixed data...
As a workaround we now do save custom data first
and then the fixed stuff

1 files changed, 20 insertions, 13 deletions

diff --git a/noncore/apps/opie-console/profilemanager.cpp b/noncore/apps/opie-console/profilemanager.cpp
index c965700..4b88dec 100644
--- a/noncore/apps/opie-console/profilemanager.cpp
+++ b/noncore/apps/opie-console/profilemanager.cpp
@@ -39,12 +39,14 @@ void ProfileManager::load() {
         prof.setIOLayer( conf.readEntry("iolayer").utf8() );
         prof.setTerminalName( conf.readEntry("term").utf8() );
         qWarning(" %s %s", conf.readEntry("iolayer").latin1(), prof.ioLayerName().data() );
         prof.setBackground( conf.readNumEntry("back") );
         prof.setForeground( conf.readNumEntry("fore") );
         prof.setTerminal( conf.readNumEntry("terminal") );
+
+        // THIS is evil because all data get's reset
         prof.setConf( conf.items( (*it) ) );
 
         /* now add it */
         m_list.append( prof );
     }
 
@@ -91,27 +93,32 @@ Session* ProfileManager::fromProfile( const Profile& prof,  QWidget* parent) {
 
     return session;
 }
 void ProfileManager::save(  ) {
     QFile::remove( (QString(getenv("HOME") )+ "/Settings/opie-console-profiles.conf" ) );
     ProfileConfig conf("opie-console-profiles");
-    Profile::ValueList::Iterator it;
-    for (it = m_list.begin(); it != m_list.end(); ++it ) {
-        conf.setGroup( (*it).name() );
-        conf.writeEntry( "name", (*it).name() );
-        conf.writeEntry( "iolayer", QString::fromUtf8( (*it).ioLayerName() ) );
-        conf.writeEntry( "term", QString::fromUtf8( (*it).terminalName()  ) );
-        conf.writeEntry( "back", (*it).background() );
-        conf.writeEntry( "fore", (*it).foreground() );
-        conf.writeEntry( "terminal", (*it).terminal() );
+    Profile::ValueList::Iterator it2;
+    for (it2 = m_list.begin(); it2 != m_list.end(); ++it2 ) {
+        conf.setGroup( (*it2).name() );
+
         /* now the config stuff */
-        QMap<QString, QString> map =  (*it).conf();
-        QMap<QString, QString>::Iterator it;
-        for ( it = map.begin(); it != map.end(); ++it ) {
-            conf.writeEntry( it.key(), it.data() );
+        QMap<QString, QString> map =  (*it2).conf();
+        QMap<QString, QString>::Iterator confIt;
+        for ( confIt = map.begin(); confIt != map.end(); ++confIt ) {
+            conf.writeEntry( confIt.key(), confIt.data() );
         }
+
+        conf.writeEntry( "name", (*it2).name() );
+        QString str = QString::fromUtf8( (*it2).ioLayerName() );
+	qWarning("IOLayerName " + str );
+
+        conf.writeEntry( "iolayer", str );
+        conf.writeEntry( "term", QString::fromUtf8( (*it2).terminalName()  ) );
+        conf.writeEntry( "back", (*it2).background() );
+        conf.writeEntry( "fore", (*it2).foreground() );
+        conf.writeEntry( "terminal", (*it2).terminal() );
     }
 }
 void ProfileManager::setProfiles( const Profile::ValueList& list ) {
     m_list = list;
 };
 Profile ProfileManager::profile( const QString& name )const {