usbaility updates

3 files changed, 208 insertions, 179 deletions

diff --git a/core/pim/addressbook/addressbook.cpp b/core/pim/addressbook/addressbook.cpp
index beb953a..8229212 100644
--- a/core/pim/addressbook/addressbook.cpp
+++ b/core/pim/addressbook/addressbook.cpp
@@ -181,27 +181,27 @@ AddressbookWindow::AddressbookWindow( QWidget *parent, const char *name,
     // not to mention it also does the necessary stuff for the
     // journaling...
     QString str = addressbookXMLFilename();
     if ( str.isNull() ) {
         QMessageBox::warning( this, tr("Out of Space"),
                               tr("There is not enough space to create\n"
                                  "neccessary startup files.\n"
                                  "\nFree up some space before\nentering data!")
                               );
     }
 
     listContainer = new QWidget( this );
-    
+
     QVBoxLayout *vb = new QVBoxLayout( listContainer );
-    
+
     abList = new AbTable( &orderedFields, listContainer, "table" );
     vb->addWidget(abList);
     abList->setHScrollBarMode( QScrollView::AlwaysOff );
     connect( abList, SIGNAL( empty( bool ) ),
              this, SLOT( listIsEmpty( bool ) ) );
     connect( abList, SIGNAL( details() ),
              this, SLOT( slotListView() ) );
     connect( abList, SIGNAL(currentChanged(int,int)),
              this, SLOT(slotUpdateToolbar()) );
 
     mView = 0;
 
@@ -224,39 +224,39 @@ AddressbookWindow::AddressbookWindow( QWidget *parent, const char *name,
 
     fontMenu = new QPopupMenu(this);
     fontMenu->setCheckable( true );
     connect( fontMenu, SIGNAL(activated(int)), this, SLOT(slotSetFont(int)));
 
     fontMenu->insertItem(tr( "Small" ), 0);
     fontMenu->insertItem(tr( "Normal" ), 1);
     fontMenu->insertItem(tr( "Large" ), 2);
 
     defaultFont = new QFont( abList->font() );
 
     slotSetFont(startFontSize);
-                    
+
     mbList->insertItem( tr("Font"), fontMenu);
     setCentralWidget(listContainer);
 
     //    qDebug("adressbook contrsuction: t=%d", t.elapsed() );
 }
 void AddressbookWindow::slotSetFont( int size ) {
 
         if (size > 2 || size < 0)
                 size = 1;
 
         startFontSize = size;
 
         QFont *currentFont;
-        
+
         switch (size) {
                 case 0:
                         fontMenu->setItemChecked(0, true);
                         fontMenu->setItemChecked(1, false);
                         fontMenu->setItemChecked(2, false);
                         abList->setFont( QFont( defaultFont->family(), defaultFont->pointSize() - 2 ) );
                         currentFont = new QFont (abList->font());
                         abList->resizeRows(currentFont->pixelSize() + 7);
                         break;
                 case 1:
                         fontMenu->setItemChecked(0, false);
                         fontMenu->setItemChecked(1, true);
@@ -462,25 +462,25 @@ static void parseName( const QString& name, QString *first, QString *middle,
     } else {
         int space = name.findRev( ' ' );
         *last = name.mid( space+1 );
         rest = name.left( space );
     }
     int space = rest.find( ' ' );
     if ( space <= 0 ) {
         *first = rest;
     } else {
         *first = rest.left( space );
         *middle = rest.mid( space+1 );
     }
-        
+
 }
 
 
 void AddressbookWindow::appMessage(const QCString &msg, const QByteArray &data)
 {
     if (msg == "editPersonal()") {
         editPersonal();
     } else if (msg == "editPersonalAndClose()") {
         editPersonal();
         close();
     } else if ( msg == "addContact(QString,QString)" ) {
         QDataStream stream(data,IO_ReadOnly);
@@ -494,25 +494,25 @@ void AddressbookWindow::appMessage(const QCString &msg, const QByteArray &data)
         cnt.setFirstName( fn );
         cnt.setMiddleName( mn );
         cnt.setLastName( ln );
         cnt.setEmails( email );
         cnt.setDefaultEmail( email );
         cnt.setFileAs();
 
         if ( bAbEditFirstTime ) {
             abEditor = new ContactEditor( cnt, &orderedFields, &slOrderedFields,
                                      this, "editor" );
             bAbEditFirstTime = FALSE;
         } else {
-                    abEditor->setEntry( cnt );
+            abEditor->setEntry( cnt );
         }
         abView()->init( cnt );
         editEntry( NewEntry );
 
 
 
     }
 #if 0
     else if (msg == "pickAddresses(QCString,QCString,QStringList,...)" ) {
         QDataStream stream(data,IO_ReadOnly);
         QCString ch,m;
         QStringList types;
@@ -785,25 +785,25 @@ void AddressbookWindow::initFields()
 
         zn = cfg.readEntry( "Category" + QString::number(i), QString::null );
         while ( !zn.isNull() ) {
             if ( zn.contains( tr("Work") ) || zn.contains( tr("Mb") ) ) {
                 slOrderedFields.clear();
                 break;
             }
             slOrderedFields.append( zn );
             zn = cfg.readEntry( "Category" + QString::number(++i), QString::null );
         }
         cfg.setGroup( "Font" );
         startFontSize = cfg.readNumEntry( "fontSize", 1 );
-        
+
 
     } else {
         QString str;
         str = getenv("HOME");
         str += "/Settings/AddressBook.conf";
         QFile::remove( str );
     }
     if ( slOrderedFields.count() > 0 ) {
         for( QStringList::ConstIterator it = slOrderedFields.begin();
              it != slOrderedFields.end(); ++it ) {
             QValueList<int>::ConstIterator itVl;
             QStringList::ConstIterator itVis;
@@ -850,32 +850,32 @@ AbLabel *AddressbookWindow::abView()
       mView = new AbLabel( this, "viewer" );
       mView->init( Contact()  );
       connect( mView, SIGNAL( okPressed() ), this, SLOT( slotListView() ) );
     }
     return mView;
 }
 
 void AddressbookWindow::slotFind()
 {
 #ifndef MAKE_FOR_SHARP_ROM
     if ( centralWidget() == abView() )
         showList();
-    
+
     FindDialog frmFind( "Contacts", this );
     QObject::connect( &frmFind, SIGNAL(signalFindClicked(const QString &, bool, bool, int)), abList, SLOT(slotDoFind( const QString&,bool,bool,int)));
     QObject::connect( abList, SIGNAL(signalNotFound()), &frmFind, SLOT(slotNotFound()) );
     QObject::connect( abList, SIGNAL(signalWrapAround()), &frmFind, SLOT(slotWrapAround()) );
-    
+
     frmFind.exec();
-    
+
     if ( abList->numSelections() )
         abList->clearSelection();
 
     abList->clearFindRow();
 #endif
 }
 
 void AddressbookWindow::slotSetCategory( int c )
 {
     if ( c <= 0 )
         return;
     for ( unsigned int i = 1; i < catMenu->count(); i++ )
@@ -897,24 +897,25 @@ void AddressbookWindow::slotSetLetter( char c ) {
 
         abList->setShowByLetter( c );
 
 }
 
 void AddressbookWindow::populateCategories()
 {
     catMenu->clear();
 
     int id,
         rememberId;
     id = 1;
+    rememberId = 0;
     catMenu->insertItem( tr( "All" ), id++ );
     QStringList categories = abList->categories();
     categories.append( tr( "Unfiled" ) );
     for ( QStringList::Iterator it = categories.begin();
           it != categories.end(); ++it ) {
         catMenu->insertItem( *it, id );
          if ( *it == abList->showCategory() )
              rememberId = id;
         ++id;
     }
     if ( abList->showCategory().isEmpty() )
         slotSetCategory( 1 );
diff --git a/core/pim/addressbook/contacteditor.cpp b/core/pim/addressbook/contacteditor.cpp
index e74da87..fa60f98 100644
--- a/core/pim/addressbook/contacteditor.cpp
+++ b/core/pim/addressbook/contacteditor.cpp
@@ -48,238 +48,238 @@ void parseEmailFrom( const QString &txt, QString &strDefaultEmail,
 // helper convert from file format to comma delimited...
 void parseEmailTo( const QString &strDefaultEmail,
                    const QString &strOtherEmail, QString &strBack );
 
         ContactEditor::ContactEditor(const Contact &entry,
                                 const QValueList<int> *newOrderedValues,
                                 QStringList *slNewOrdered,
                                 QWidget *parent,
                                 const char *name,
                                 WFlags fl )
         : QDialog( parent, name, TRUE, fl ),
           orderedValues( newOrderedValues ),
-          slOrdered( slNewOrdered )
+          slOrdered( *slNewOrdered )
 {
-        
+
         init();
         initMap();
         setEntry( entry );
         qDebug("finish");
 }
 
 ContactEditor::~ContactEditor() {
 }
 
 void ContactEditor::init() {
 
         useFullName = TRUE;
 
         int i = 0;
-
+/** SHut up and stop leaking
         slHomeAddress = new QStringList;
         slBusinessAddress = new QStringList;
         slChooserNames = new QStringList;
         slChooserValues = new QStringList;
 
         slDynamicEntries = new QStringList;
-
+*/
         //*slDynamicEntries = *slOrdered;
 
         for (i = 0; i <= 6; i++) {
-                slHomeAddress->append( "" );
-                slBusinessAddress->append( "" );
+                slHomeAddress.append( "" );
+                slBusinessAddress.append( "" );
         }
 
         {
                 hasGender = FALSE;
                 hasTitle = FALSE;
                 hasCompany = FALSE;
                 hasNotes = FALSE;
                 hasStreet = FALSE;
                 hasStreet2 = FALSE;
                 hasPOBox = FALSE;
                 hasCity = FALSE;
                 hasState = FALSE;
                 hasZip = FALSE;
                 hasCountry = FALSE;
-                
-                QStringList::ConstIterator it = slOrdered->begin();
-                for ( i = 0; it != slOrdered->end(); i++, ++it ) {
+
+                QStringList::ConstIterator it = slOrdered.begin();
+                for ( i = 0; it != slOrdered.end(); i++, ++it ) {
 
                         if ( (*it).right( 3 ) == tr( "Fax" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( tr( "" ) );
+                                slChooserNames.append( *it );
+                                slChooserValues.append("" );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( (*it).right( 5 ) == tr( "Phone" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( "" );
+                                slChooserNames.append( *it );
+                                slChooserValues.append( "" );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( (*it).right( 2 ) == tr( "IM" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( ""  );
+                                slChooserNames.append( *it );
+                                slChooserValues.append( ""  );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( (*it).right( 6 ) == tr( "Mobile" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( "" );
+                                slChooserNames.append( *it );
+                                slChooserValues.append( "" );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( (*it).right( 4 ) == tr( "Page" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( "" );
+                                slChooserNames.append( *it );
+                                slChooserValues.append( "" );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( (*it).right( 5 ) == tr( "Pager" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( "" );
+                                slChooserNames.append( *it );
+                                slChooserValues.append( "" );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( *it == tr( "Default Email" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( "" );
+                                slChooserNames.append( *it );
+                                slChooserValues.append( "" );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( *it == tr( "Emails" ) ) {
-                                slChooserNames->append( *it );
-                                slChooserValues->append( "" );
+                                slChooserNames.append( *it );
+                                slChooserValues.append( "" );
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( *it == tr( "Name Title" ) ) {
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( *it == tr( "First Name" ) ) {
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-                        
+
                         if ( *it == tr( "Middle Name" ) ) {
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-                        
+
                         if ( *it == tr( "Last Name" ) ) {
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( *it == tr( "Suffix" ) ) {
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
                         if ( *it == tr( "File As" ) ) {
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( *it == tr( "Gender" ) ) {
                                 hasGender = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( *it == tr( "Job Title" ) ) {
                                 hasTitle = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( *it == tr( "Company" ) ) {
                                 hasCompany = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( *it == tr( "Notes" ) ) {
                                 hasNotes = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( *it == tr( "Groups" ) ) {
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( (*it).right( 6 ) == tr( "Street" ) ) {
                                 hasStreet = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( (*it).right( 8 ) == tr( "Street 2" ) ) {
                                 hasStreet2 = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( (*it).right( 8 ) == tr( "P.O. Box" ) ) {
                                 hasPOBox = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( (*it).right( 4 ) == tr( "City" ) ) {
                                 hasCity = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( (*it).right( 5 ) == tr( "State" ) ) {
                                 hasState = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( (*it).right( 3 ) == tr( "Zip" ) ) {
                                 hasZip = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
-        
+
                         if ( (*it).right( 7 ) == tr( "Country" ) ) {
                                 hasCountry = TRUE;
                                 //slDynamicEntries->remove( it );
                                 continue;
                         }
 
-                        slDynamicEntries->append( *it );
+                        slDynamicEntries.append( *it );
                 }
         }
 
         QVBoxLayout *vb = new QVBoxLayout( this );
-        
+
         tabMain = new QTabWidget( this );
         vb->addWidget( tabMain );
-        
+
         QWidget *tabViewport = new QWidget ( tabMain );
 
         vb = new QVBoxLayout( tabViewport );
 
         svGeneral = new QScrollView( tabViewport );
         vb->addWidget( svGeneral, 0, 0 );
         svGeneral->setResizePolicy( QScrollView::AutoOneFit );
         svGeneral->setFrameStyle( QFrame::NoFrame );
 
         QWidget *container = new QWidget( svGeneral->viewport() );
         svGeneral->addChild( container );
 
@@ -337,39 +337,39 @@ void ContactEditor::init() {
         tabViewport = new QWidget ( tabMain );
 
         vb = new QVBoxLayout( tabViewport );
 
         svAddress = new QScrollView( tabViewport );
         vb->addWidget( svAddress, 0, 0 );
         svAddress->setResizePolicy( QScrollView::AutoOneFit );
         svAddress->setFrameStyle( QFrame::NoFrame );
 
         container = new QWidget( svAddress->viewport() );
         svAddress->addChild( container );
 
-        gl = new QGridLayout( container, 6, 3, 2, 4 );
+        gl = new QGridLayout( container, 8, 3, 2, 4 ); // row 7 QSpacerItem
 
         cmbAddress = new QComboBox( FALSE, container );
         cmbAddress->insertItem( tr( "Business" ) );
         cmbAddress->insertItem( tr( "Home" ) );
         gl->addMultiCellWidget( cmbAddress, 0, 0, 0, 1 );
 
         l = new QLabel( tr( "Address" ), container );
         gl->addWidget( l, 1, 0 );
         txtAddress = new QLineEdit( container );
         gl->addMultiCellWidget( txtAddress, 1, 1, 1, 2 );
 /*
         l = new QLabel( tr( "Address 2" ), container );
         gl->addWidget( l, 2, 0 );
-        txtAddress2 = new QLineEdit( container ); 
+        txtAddress2 = new QLineEdit( container );
         gl->addMultiCellWidget( txtAddress2, 2, 2, 1, 2 );
 
         l = new QLabel( tr( "P.O. Box" ), container );
         gl->addWidget( l, 3, 0 );
         txtPOBox = new QLineEdit( container );
         gl->addMultiCellWidget( txtPOBox, 3, 3, 1, 2 );
 */
         l = new QLabel( tr( "City" ), container );
         gl->addWidget( l, 2, 0 );
         txtCity = new QLineEdit( container );
         gl->addMultiCellWidget( txtCity, 2, 2, 1, 2 );
 
@@ -598,121 +598,147 @@ void ContactEditor::init() {
         cmbCountry->insertItem( tr ( "Zambia" ) );
         cmbCountry->insertItem( tr ( "Zimbabwe" ) );
 
         cmbCountry->setMaximumWidth( 135 );
 
         gl->addMultiCellWidget( cmbCountry, 5, 5, 1, 2 );
 
         cmbChooserField4 = new QComboBox( FALSE, container );
         cmbChooserField4->setMaximumWidth( 90 );
         gl->addWidget( cmbChooserField4, 6, 0 );
         txtChooserField4 = new QLineEdit( container );
         gl->addMultiCellWidget( txtChooserField4, 6, 6, 1, 2 );
+
+        QSpacerItem *space = new QSpacerItem(1,1,
+                                             QSizePolicy::Maximum,
+                                             QSizePolicy::MinimumExpanding );
+        gl->addItem( space, 7, 0 );
+
         tabMain->insertTab( tabViewport, tr( "Address" ) );
 
         tabViewport = new QWidget ( tabMain );
 
         vb = new QVBoxLayout( tabViewport );
 
         svDetails = new QScrollView( tabViewport );
         vb->addWidget( svDetails, 0, 0 );
         svDetails->setResizePolicy( QScrollView::AutoOneFit );
         svDetails->setFrameStyle( QFrame::NoFrame );
 
         container = new QWidget( svDetails->viewport() );
         svDetails->addChild( container );
 
         gl = new QGridLayout( container, 1, 2, 2, 4 );
 
-        QStringList::ConstIterator it = slDynamicEntries->begin();
-        for (i = 0; it != slDynamicEntries->end(); i++, ++it) {
+        QStringList::ConstIterator it = slDynamicEntries.begin();
+        for (i = 0; it != slDynamicEntries.end(); i++, ++it) {
                 l = new QLabel( *it, container );
                 listName.append( l );
                 gl->addWidget( l, i, 0 );
                 QLineEdit *e = new QLineEdit( container );
                 listValue.append( e );
                 gl->addWidget( e, i, 1);
         }
 
         l = new QLabel( tr("Gender"), container );
-        gl->addWidget( l, slDynamicEntries->count(), 0 );
+        gl->addWidget( l, slDynamicEntries.count(), 0 );
         cmbGender = new QComboBox( container );
         cmbGender->insertItem( "", 0 );
         cmbGender->insertItem( tr("Male"), 1);
         cmbGender->insertItem( tr("Female"), 2);
-        gl->addWidget( cmbGender, slDynamicEntries->count(), 1 );
-        
+        gl->addWidget( cmbGender, slDynamicEntries.count(), 1 );
+
         tabMain->insertTab( tabViewport, tr( "Details" ) );
-        
+
         dlgNote = new QDialog( this, "Note Dialog", TRUE );
         dlgNote->setCaption( tr("Enter Note") );
         QVBoxLayout *vbNote = new QVBoxLayout( dlgNote );
         txtNote = new QMultiLineEdit( dlgNote );
         vbNote->addWidget( txtNote );
         connect( btnNote, SIGNAL(clicked()), this, SLOT(slotNote()) );
 
         dlgName = new QDialog( this, "Name Dialog", TRUE );
         dlgName->setCaption( tr("Edit Name") );
-        gl = new QGridLayout( dlgName, 4, 2, 2, 3 );
+        gl = new QGridLayout( dlgName, 5, 2, 2, 3 );
 
         l = new QLabel( tr("First Name"), dlgName );
         gl->addWidget( l, 0, 0 );
         txtFirstName = new QLineEdit( dlgName );
         gl->addWidget( txtFirstName, 0, 1 );
 
         l = new QLabel( tr("Middle Name"), dlgName );
         gl->addWidget( l, 1, 0 );
         txtMiddleName = new QLineEdit( dlgName );
         gl->addWidget( txtMiddleName, 1, 1 );
 
         l = new QLabel( tr("Last Name"), dlgName );
         gl->addWidget( l, 2, 0 );
         txtLastName = new QLineEdit( dlgName );
         gl->addWidget( txtLastName, 2, 1 );
 
         l = new QLabel( tr("Suffix"), dlgName );
         gl->addWidget( l, 3, 0 );
         txtSuffix = new QLineEdit( dlgName );
         gl->addWidget( txtSuffix, 3, 1 );
+        space = new QSpacerItem(1,1,
+                                QSizePolicy::Maximum,
+                                QSizePolicy::MinimumExpanding );
+        gl->addItem( space, 4,  0 );
 
-        cmbChooserField1->insertStringList( *slChooserNames );
-        cmbChooserField2->insertStringList( *slChooserNames );
-        cmbChooserField3->insertStringList( *slChooserNames );
-        cmbChooserField4->insertStringList( *slChooserNames );
+        cmbChooserField1->insertStringList( slChooserNames );
+        cmbChooserField2->insertStringList( slChooserNames );
+        cmbChooserField3->insertStringList( slChooserNames );
+        cmbChooserField4->insertStringList( slChooserNames );
 
         cmbChooserField1->setCurrentItem( 0 );
         cmbChooserField2->setCurrentItem( 1 );
         cmbChooserField3->setCurrentItem( 2 );
-        
+
         connect( btnFullName, SIGNAL(clicked()), this, SLOT(slotName()) );
-                        
+
         connect( txtFullName, SIGNAL(textChanged(const QString &)), this, SLOT(slotFullNameChange(const QString &)) );
-        
-        connect( txtChooserField1, SIGNAL(textChanged(const QString &)), this, SLOT(slotChooser1Change(const QString &)) ); 
-        connect( txtChooserField2, SIGNAL(textChanged(const QString &)), this, SLOT(slotChooser2Change(const QString &)) ); 
-        connect( txtChooserField3, SIGNAL(textChanged(const QString &)), this, SLOT(slotChooser3Change(const QString &)) ); 
-        connect( txtChooserField4, SIGNAL(textChanged(const QString &)), this, SLOT(slotChooser4Change(const QString &)) ); 
-        connect( txtAddress, SIGNAL(textChanged(const QString &)), this, SLOT(slotAddressChange(const QString &)) ); 
-        //connect( txtAddress2, SIGNAL(textChanged(const QString &)), this, SLOT(slotAddress2Change(const QString &)) ); 
-        //connect( txtPOBox, SIGNAL(textChanged(const QString &)), this, SLOT(slotPOBoxChange(const QString &)) ); 
-        connect( txtCity, SIGNAL(textChanged(const QString &)), this, SLOT(slotCityChange(const QString &)) ); 
-        connect( txtState, SIGNAL(textChanged(const QString &)), this, SLOT(slotStateChange(const QString &)) ); 
-        connect( txtZip, SIGNAL(textChanged(const QString &)), this, SLOT(slotZipChange(const QString &)) ); 
-        connect( cmbCountry, SIGNAL(textChanged(const QString &)), this, SLOT(slotCountryChange(const QString &)) ); 
-        connect( cmbCountry, SIGNAL(activated(const QString &)), this, SLOT(slotCountryChange(const QString &)) ); 
-        connect( cmbChooserField1, SIGNAL(activated(int)), this, SLOT(slotCmbChooser1Change(int)) );
-        connect( cmbChooserField2, SIGNAL(activated(int)), this, SLOT(slotCmbChooser2Change(int)) );
-        connect( cmbChooserField3, SIGNAL(activated(int)), this, SLOT(slotCmbChooser3Change(int)) );
-        connect( cmbChooserField4, SIGNAL(activated(int)), this, SLOT(slotCmbChooser4Change(int)) );
-        connect( cmbAddress, SIGNAL(activated(int)), this, SLOT(slotAddressTypeChange(int)) );
+
+        connect( txtChooserField1, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotChooser1Change(const QString &)) );
+        connect( txtChooserField2, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotChooser2Change(const QString &)) );
+        connect( txtChooserField3, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotChooser3Change(const QString &)) );
+        connect( txtChooserField4, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotChooser4Change(const QString &)) );
+        connect( txtAddress, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotAddressChange(const QString &)) );
+        //connect( txtAddress2, SIGNAL(textChanged(const QString &)), this, SLOT(slotAddress2Change(const QString &)) );
+        //connect( txtPOBox, SIGNAL(textChanged(const QString &)), this, SLOT(slotPOBoxChange(const QString &)) );
+        connect( txtCity, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotCityChange(const QString &)) );
+        connect( txtState, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotStateChange(const QString &)) );
+        connect( txtZip, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotZipChange(const QString &)) );
+        connect( cmbCountry, SIGNAL(textChanged(const QString &)),
+                 this, SLOT(slotCountryChange(const QString &)) );
+        connect( cmbCountry, SIGNAL(activated(const QString &)),
+                 this, SLOT(slotCountryChange(const QString &)) );
+        connect( cmbChooserField1, SIGNAL(activated(int)),
+                 this, SLOT(slotCmbChooser1Change(int)) );
+        connect( cmbChooserField2, SIGNAL(activated(int)),
+                 this, SLOT(slotCmbChooser2Change(int)) );
+        connect( cmbChooserField3, SIGNAL(activated(int)),
+                 this, SLOT(slotCmbChooser3Change(int)) );
+        connect( cmbChooserField4, SIGNAL(activated(int)),
+                 this, SLOT(slotCmbChooser4Change(int)) );
+        connect( cmbAddress, SIGNAL(activated(int)),
+                 this, SLOT(slotAddressTypeChange(int)) );
+
         new QPEDialogListener(this);
 }
 
 void ContactEditor::initMap()
 {
     /*
     // since the fields and the XML fields exist, create a map
     // between them...
     Config cfg1( "AddressBook" );
     Config cfg2( "AddressBook" );
     QString strCfg1,
             strCfg2;
@@ -730,190 +756,192 @@ o    cfg2.setGroup( "XMLFields" );
         strCfg1 = cfg1.readEntry( "Field" + QString::number(i),
                                   QString::null );
         strCfg2 = cfg2.readEntry( "XMLField" + QString::number(i++),
                                   QString::null );
     }
     */
 }
 
 void ContactEditor::slotChooser1Change( const QString &textChanged ) {
 
         int index = cmbChooserField1->currentItem();
 
-        (*slChooserValues)[index] = textChanged;
+        slChooserValues[index] = textChanged;
 
 }
 
 void ContactEditor::slotChooser2Change( const QString &textChanged ) {
 
         int index = cmbChooserField2->currentItem();
 
-        (*slChooserValues)[index] = textChanged;
+        slChooserValues[index] = textChanged;
 
 }
 
 void ContactEditor::slotChooser3Change( const QString &textChanged ) {
 
         int index = cmbChooserField3->currentItem();
 
-        (*slChooserValues)[index] = textChanged;
+        slChooserValues[index] = textChanged;
 
 }
 
 void ContactEditor::slotChooser4Change( const QString &textChanged ) {
 
         int index = cmbChooserField4->currentItem();
 
-        (*slChooserValues)[index] = textChanged;
+        slChooserValues[index] = textChanged;
 
 }
 
 void ContactEditor::slotAddressChange( const QString &textChanged ) {
 
         if ( cmbAddress->currentItem() == 0 ) {
-                (*slBusinessAddress)[0] = textChanged;
+                slBusinessAddress[0] = textChanged;
         } else {
-                (*slHomeAddress)[0] = textChanged;
+                slHomeAddress[0] = textChanged;
         }
 }
 
 void ContactEditor::slotAddress2Change( const QString &textChanged ) {
 
         if ( cmbAddress->currentItem() == 0 ) {
-                (*slBusinessAddress)[1] = textChanged;
+                slBusinessAddress[1] = textChanged;
         } else {
-                (*slHomeAddress)[1] = textChanged;
+                slHomeAddress[1] = textChanged;
         }
 }
 
 void ContactEditor::slotPOBoxChange( const QString &textChanged ) {
 
         if ( cmbAddress->currentItem() == 0 ) {
-                (*slBusinessAddress)[2] = textChanged;
+                slBusinessAddress[2] = textChanged;
         } else {
-                (*slHomeAddress)[2] = textChanged;
+                slHomeAddress[2] = textChanged;
         }
 }
 
 void ContactEditor::slotCityChange( const QString &textChanged ) {
 
         if ( cmbAddress->currentItem() == 0 ) {
-                (*slBusinessAddress)[3] = textChanged;
+                slBusinessAddress[3] = textChanged;
         } else {
-                (*slHomeAddress)[3] = textChanged;
+                slHomeAddress[3] = textChanged;
         }
 }
 
 void ContactEditor::slotStateChange( const QString &textChanged ) {
 
-        
+
         if ( cmbAddress->currentItem() == 0 ) {
-                (*slBusinessAddress)[4] = textChanged;
+                slBusinessAddress[4] = textChanged;
         } else {
-                (*slHomeAddress)[4] = textChanged;
+                slHomeAddress[4] = textChanged;
         }
 }
 
 void ContactEditor::slotZipChange( const QString &textChanged ) {
 
         if ( cmbAddress->currentItem() == 0 ) {
-                (*slBusinessAddress)[5] = textChanged;
+                slBusinessAddress[5] = textChanged;
         } else {
-                (*slHomeAddress)[5] = textChanged;
+                slHomeAddress[5] = textChanged;
         }
 }
 
 void ContactEditor::slotCountryChange( const QString &textChanged ) {
 
         if ( cmbAddress->currentItem() == 0 ) {
-                (*slBusinessAddress)[6] = textChanged;
+                slBusinessAddress[6] = textChanged;
         } else {
-                (*slHomeAddress)[6] = textChanged;
+                slHomeAddress[6] = textChanged;
         }
 }
 
 void ContactEditor::slotCmbChooser1Change( int index ) {
 
-        txtChooserField1->setText( (*slChooserValues)[index] );
-
+        txtChooserField1->setText( slChooserValues[index] );
+        txtChooserField1->setFocus();
 }
 
 void ContactEditor::slotCmbChooser2Change( int index ) {
 
-        txtChooserField2->setText( (*slChooserValues)[index] );
-
+        txtChooserField2->setText( slChooserValues[index] );
+        txtChooserField2->setFocus();
 }
 
 void ContactEditor::slotCmbChooser3Change( int index ) {
 
-        txtChooserField3->setText( (*slChooserValues)[index] );
+        txtChooserField3->setText( slChooserValues[index] );
+        txtChooserField3->setFocus();
 
 }
 
 void ContactEditor::slotCmbChooser4Change( int index ) {
 
-        txtChooserField4->setText( (*slChooserValues)[index] );
+        txtChooserField4->setText( slChooserValues[index] );
+        txtChooserField4->setFocus();
 
 }
 
 void ContactEditor::slotAddressTypeChange( int index ) {
 
         if ( index == 0 ) {
 
-                txtAddress->setText( (*slBusinessAddress)[0] );
+                txtAddress->setText( slBusinessAddress[0] );
                 //txtAddress2->setText( (*slBusinessAddress)[1] );
                 //txtPOBox->setText( (*slBusinessAddress)[2] );
-                txtCity->setText( (*slBusinessAddress)[3] );
-                txtState->setText( (*slBusinessAddress)[4] );
-                txtZip->setText( (*slBusinessAddress)[5] );
+                txtCity->setText( slBusinessAddress[3] );
+                txtState->setText( slBusinessAddress[4] );
+                txtZip->setText( slBusinessAddress[5] );
                 QLineEdit *txtTmp = cmbCountry->lineEdit();
-                txtTmp->setText( (*slBusinessAddress)[6] );
+                txtTmp->setText( slBusinessAddress[6] );
 
         } else {
 
-                txtAddress->setText( (*slHomeAddress)[0] );
+                txtAddress->setText( slHomeAddress[0] );
                 //txtAddress2->setText( (*slHomeAddress)[1] );
                 //txtPOBox->setText( (*slHomeAddress)[2] );
-                txtCity->setText( (*slHomeAddress)[3] );
-                txtState->setText( (*slHomeAddress)[4] );
-                txtZip->setText( (*slHomeAddress)[5] );
+                txtCity->setText( slHomeAddress[3] );
+                txtState->setText( slHomeAddress[4] );
+                txtZip->setText( slHomeAddress[5] );
                 QLineEdit *txtTmp = cmbCountry->lineEdit();
-                txtTmp->setText( (*slHomeAddress)[6] );
+                txtTmp->setText( slHomeAddress[6] );
 
         }
 
 }
 
 void ContactEditor::slotFullNameChange( const QString &textChanged ) {
 
         int index = cmbFileAs->currentItem();
 
         cmbFileAs->clear();
 
         cmbFileAs->insertItem( parseName( textChanged, 0 ) );
         cmbFileAs->insertItem( parseName( textChanged, 1 ) );
         cmbFileAs->insertItem( parseName( textChanged, 2 ) );
         cmbFileAs->insertItem( parseName( textChanged, 3 ) );
 
         cmbFileAs->setCurrentItem( index );
 
         useFullName = TRUE;
 
-        }
+}
 
 void ContactEditor::loadFields() {
 
         QStringList::ConstIterator it;
         QListIterator<QLabel> lit( listName );
-        for ( it = slDynamicEntries->begin(); *lit; ++lit, ++it) {
+        for ( it = slDynamicEntries.begin(); *lit; ++lit, ++it) {
                 (*lit)->setText( *it );
         }
 }
 
 void ContactEditor::accept() {
 
         if ( isEmpty() ) {
                 cleanupFields();
                 reject();
         } else {
                 saveEntry();
                 cleanupFields();
@@ -950,54 +978,54 @@ void ContactEditor::slotName() {
 
 void ContactEditor::setNameFocus() {
 
         txtFullName->setFocus();
 
 }
 
 bool ContactEditor::isEmpty() {
         // Test and see if the record should be saved.
         // More strict than the original qtopia, needs name or fileas to save
 
         QString t = txtFullName->text();
-        if ( !t.isEmpty() && containsAlphaNum( t ) ) 
+        if ( !t.isEmpty() && containsAlphaNum( t ) )
                 return false;
-        
+
         t = cmbFileAs->currentText();
         if ( !t.isEmpty() && containsAlphaNum( t ) )
                 return false;
 
         return true;
 
 }
 
 QString ContactEditor::parseName( const QString fullName, int type ) {
 
         QString simplifiedName( fullName.simplifyWhiteSpace() );
         QString strFirstName;
         QString strMiddleName;
         QString strLastName;
         QString strSuffix;
         QString strTitle;
         int commapos;
         int spCount;
         int spPos;
         int spPos2;
-        
+
 
         commapos = simplifiedName.find( ',', 0, TRUE);
         spCount = simplifiedName.contains( ' ', TRUE );
 
         if ( commapos == -1 ) {
-                
+
                 switch (spCount) {
                         case 0:
                                 return simplifiedName;
 
                         case 1:
                                 spPos = simplifiedName.find( ' ', 0, TRUE );
                                 strFirstName = simplifiedName.left( spPos );
                                 strLastName = simplifiedName.mid( spPos + 1 );
                                 break;
 
                         case 2:
                                 spPos = simplifiedName.find( ' ', 0, TRUE );
@@ -1035,25 +1063,25 @@ QString ContactEditor::parseName( const QString fullName, int type ) {
                                 spPos2 = simplifiedName.find( ' ', spPos + 1, TRUE );
                                 strFirstName = simplifiedName.mid( spPos + 1, (spPos2 - 1) - spPos );
                                 spPos = simplifiedName.find( ' ', spPos2 + 1, TRUE );
                                 strMiddleName = simplifiedName.mid( spPos2 + 1, (spPos - 1) - spPos2 );
                                 spPos2 = simplifiedName.find( ' ', spPos + 1, TRUE );
                                 strLastName = simplifiedName.mid( spPos + 1, (spPos2 - 1) - spPos );
                                 strSuffix = simplifiedName.mid( spPos2 + 1 );
                                 break;
                 }
         } else {
                 simplifiedName.replace( commapos, 1, " " );
                 simplifiedName = simplifiedName.simplifyWhiteSpace();
-                
+
                 switch (spCount) {
                         case 0:
                                 return simplifiedName;
 
                         case 1:
                                 spPos = simplifiedName.find( ' ', 0, TRUE );
                                 strLastName = simplifiedName.left( spPos );
                                 strFirstName = simplifiedName.mid( spPos + 1 );
                                 break;
 
                         case 2:
                                 spPos = simplifiedName.find( ' ', 0, TRUE );
@@ -1089,25 +1117,25 @@ QString ContactEditor::parseName( const QString fullName, int type ) {
                                 spPos = simplifiedName.find( ' ', 0, TRUE );
                                 strLastName = simplifiedName.left( spPos );
                                 spPos2 = simplifiedName.find( ' ', spPos + 1, TRUE );
                                 strTitle = simplifiedName.mid( spPos + 1, (spPos2 - 1) - spPos );
                                 spPos = simplifiedName.find( ' ', spPos2 + 1, TRUE );
                                 strFirstName = simplifiedName.mid( spPos2 + 1, (spPos - 1) - spPos );
                                 spPos2 = simplifiedName.find( ' ', spPos + 1, TRUE );
                                 strMiddleName = simplifiedName.mid( spPos + 1, (spPos2 - 1) - spPos );
                                 strSuffix = simplifiedName.mid( spPos2 + 1 );
                                 break;
                 }
         }
-        
+
         switch (type) {
                 case NAME_FL:
                         return strFirstName + " " + strLastName;
 
                 case NAME_LF:
                         return strLastName + ", " + strFirstName;
 
                 case NAME_LFM:
                         return strLastName + ", " + strFirstName + " " + strMiddleName;
 
                 case NAME_FMLS:
                         return strFirstName + " " + strMiddleName + " " + strLastName + " " + strSuffix;
@@ -1121,37 +1149,37 @@ QString ContactEditor::parseName( const QString fullName, int type ) {
                 case NAME_L:
                         return strLastName;
 
                 case NAME_S:
                         return strSuffix;
 
         }
         return QString::null;
 }
 
 void ContactEditor::cleanupFields() {
 
-        QStringList::Iterator it = slChooserValues->begin();
-        for ( int i = 0; it != slChooserValues->end(); i++, ++it ) {
+        QStringList::Iterator it = slChooserValues.begin();
+        for ( int i = 0; it != slChooserValues.end(); i++, ++it ) {
                 (*it) = "";
         }
 
         for ( int i = 0; i < 7; i++ ) {
-                (*slHomeAddress)[i] = "";
-                (*slBusinessAddress)[i] = "";
+                slHomeAddress[i] = "";
+                slBusinessAddress[i] = "";
         }
 
         QStringList::ConstIterator cit;
         QListIterator<QLineEdit> itLE( listValue );
-        for ( cit = slDynamicEntries->begin(); cit != slDynamicEntries->end(); ++cit, ++itLE) {
+        for ( cit = slDynamicEntries.begin(); cit != slDynamicEntries.end(); ++cit, ++itLE) {
                 (*itLE)->setText( "" );
         }
 
         txtFirstName->setText("");
         txtMiddleName->setText("");
         txtLastName->setText("");
         txtSuffix->setText("");
         txtNote->setText("");
         txtFullName->setText("");
         txtJobTitle->setText("");
         txtOrganization->setText("");
         txtChooserField1->setText("");
@@ -1165,88 +1193,88 @@ void ContactEditor::cleanupFields() {
         txtZip->setText("");
         QLineEdit *txtTmp = cmbCountry->lineEdit();
         txtTmp->setText("");
         txtTmp = cmbFileAs->lineEdit();
         txtTmp->setText("");
 
 }
 
 void ContactEditor::setEntry( const Contact &entry ) {
 
         cleanupFields();
 
-        
+
         ent = entry;
 
         useFullName = FALSE;
         txtFirstName->setText( ent.firstName() );
         txtMiddleName->setText( ent.middleName() );
         txtLastName->setText( ent.lastName() );
         txtSuffix->setText( ent.suffix() );
 
         QString *tmpString = new QString;
-        *tmpString = ent.firstName() + " " + ent.middleName() + 
+        *tmpString = ent.firstName() + " " + ent.middleName() +
                 + " " + ent.lastName() + " " + ent.suffix();
-        
+
         txtFullName->setText( tmpString->simplifyWhiteSpace() );
 
         cmbFileAs->setEditText( ent.fileAs() );
 
         if (hasTitle)
                 txtJobTitle->setText( ent.jobTitle() );
 
-        if (hasCompany) 
+        if (hasCompany)
                 txtOrganization->setText( ent.company() );
 
         if (hasNotes)
                 txtNote->setText( ent.notes() );
 
         if (hasStreet) {
-                (*slHomeAddress)[0] = ent.homeStreet();
-                (*slBusinessAddress)[0] = ent.businessStreet();
+                slHomeAddress[0] = ent.homeStreet();
+                slBusinessAddress[0] = ent.businessStreet();
         }
 /*
         if (hasStreet2) {
                 (*slHomeAddress)[1] = ent.homeStreet2();
                 (*slBusinessAddress)[1] = ent.businessStreet2();
         }
 
         if (hasPOBox) {
                 (*slHomeAddress)[2] = ent.homePOBox();
                 (*slBusinessAddress)[2] = ent.businessPOBox();
         }
 */
         if (hasCity) {
-                (*slHomeAddress)[3] = ent.homeCity();
-                (*slBusinessAddress)[3] = ent.businessCity();
+                slHomeAddress[3] = ent.homeCity();
+                slBusinessAddress[3] = ent.businessCity();
         }
 
         if (hasState) {
-                (*slHomeAddress)[4] = ent.homeState();
-                (*slBusinessAddress)[4] = ent.businessState();
+                slHomeAddress[4] = ent.homeState();
+                slBusinessAddress[4] = ent.businessState();
         }
 
         if (hasZip) {
-                (*slHomeAddress)[5] = ent.homeZip();
-                (*slBusinessAddress)[5] = ent.businessZip();
+                slHomeAddress[5] = ent.homeZip();
+                slBusinessAddress[5] = ent.businessZip();
         }
 
         if (hasCountry) {
-                (*slHomeAddress)[6] = ent.homeCountry();
-                (*slBusinessAddress)[6] = ent.businessCountry();
+                slHomeAddress[6] = ent.homeCountry();
+                slBusinessAddress[6] = ent.businessCountry();
         }
 
         QStringList::ConstIterator it;
         QListIterator<QLineEdit> itLE( listValue );
-        for ( it = slDynamicEntries->begin(); it != slDynamicEntries->end(); ++it, ++itLE) {
+        for ( it = slDynamicEntries.begin(); it != slDynamicEntries.end(); ++it, ++itLE) {
                 if ( *it == "Department" )
                         (*itLE)->setText( ent.department() );
 
                 if ( *it == "Company" )
                         (*itLE)->setText( ent.company() );
 
                 if ( *it == "Office" )
                         (*itLE)->setText( ent.office() );
 
                 if ( *it == "Profession" )
                         (*itLE)->setText( ent.profession() );
 
@@ -1265,25 +1293,25 @@ void ContactEditor::setEntry( const Contact &entry ) {
                 if ( *it == "Anniversary" )
                         (*itLE)->setText( ent.anniversary() );
 
                 if ( *it == "Nickname" )
                         (*itLE)->setText( ent.nickname() );
 
                 if ( *it == "Children" )
                         (*itLE)->setText( ent.children() );
 
         }
 
         QStringList::Iterator itV;
-        for ( it = slChooserNames->begin(), itV = slChooserValues->begin(); it != slChooserNames->end(); ++it, ++itV ) {
+        for ( it = slChooserNames.begin(), itV = slChooserValues.begin(); it != slChooserNames.end(); ++it, ++itV ) {
 
                 if ( *it == "Business Phone" || *it == "Work Phone" )
                         *itV = ent.businessPhone();
 /*
                 if ( *it == "Business 2 Phone" )
                         *itV = ent.business2Phone();
 */
                 if ( *it == "Business Fax" || *it == "Work Fax" )
                         *itV = ent.businessFax();
 
                 if ( *it == "Business Mobile" || *it == "work Mobile" )
                         *itV = ent.businessMobile();
@@ -1337,113 +1365,113 @@ void ContactEditor::setEntry( const Contact &entry ) {
                         *itV = ent.MSNIM();
 
                 if ( *it == "Yahoo IM" )
                         *itV = ent.yahooIM();
 */
                 if ( *it == "Home Web Page" )
                         *itV = ent.homeWebpage();
                 if ( *it == "Business Web Page" || *it == "Work Web Page" )
                         *itV = ent.businessWebpage();
 
 
         }
-        
+
         cmbCat->setCategories( ent.categories(), "Contacts", tr("Contacts") );
 
         QString gender = ent.gender();
         cmbGender->setCurrentItem( gender.toInt() );
 
         txtNote->setText( ent.notes() );
 
         slotCmbChooser1Change( cmbChooserField1->currentItem() );
         slotCmbChooser2Change( cmbChooserField2->currentItem() );
         slotCmbChooser3Change( cmbChooserField3->currentItem() );
 
         slotAddressTypeChange( cmbAddress->currentItem() );
 
 }
 
 void ContactEditor::saveEntry() {
 
         if ( useFullName == TRUE ) {
                 txtFirstName->setText( parseName( txtFullName->text(), NAME_F ) );
                 txtMiddleName->setText( parseName( txtFullName->text(), NAME_M ) );
                 txtLastName->setText( parseName( txtFullName->text(), NAME_L ) );
                 txtSuffix->setText( parseName( txtFullName->text(), NAME_S ) );
-                
+
                 useFullName = FALSE;
         }
 
-        /*if ( ent.firstName() != txtFirstName->text() || 
+        /*if ( ent.firstName() != txtFirstName->text() ||
                         ent.lastName != txtLastName->text() ||
                         ent.middleName != txtMiddleName->text() ) {
         */
                 ent.setFirstName( txtFirstName->text() );
                 ent.setLastName( txtLastName->text() );
                 ent.setMiddleName( txtMiddleName->text() );
                 ent.setSuffix( txtSuffix->text() );
 
         //}
 
         ent.setFileAs( cmbFileAs->currentText() );
-        
+
         ent.setCategories( cmbCat->currentCategories() );
 
         if (hasTitle)
                 ent.setJobTitle( txtJobTitle->text() );
 
-        if (hasCompany) 
+        if (hasCompany)
                 ent.setCompany( txtOrganization->text() );
 
         if (hasNotes)
                 ent.setNotes( txtNote->text() );
 
         if (hasStreet) {
-                ent.setHomeStreet( (*slHomeAddress)[0] );
-                ent.setBusinessStreet( (*slBusinessAddress)[0] );
+                ent.setHomeStreet( slHomeAddress[0] );
+                ent.setBusinessStreet( slBusinessAddress[0] );
         }
 /*
         if (hasStreet2) {
                 ent.setHomeStreet2( (*slHomeAddress)[1] );
                 ent.setBusinessStreet2( (*slBusinessAddress)[1] );
         }
 
         if (hasPOBox) {
                 ent.setHomePOBox( (*slHomeAddress)[2] );
                 ent.setBusinessPOBox( (*slBusinessAddress)[2] );
         }
 */
         if (hasCity) {
-                ent.setHomeCity( (*slHomeAddress)[3] );
-                ent.setBusinessCity( (*slBusinessAddress)[3] );
+                ent.setHomeCity( slHomeAddress[3] );
+                ent.setBusinessCity( slBusinessAddress[3] );
         }
 
         if (hasState) {
-                ent.setHomeState( (*slHomeAddress)[4] );
-                ent.setBusinessState( (*slBusinessAddress)[4] );
+                ent.setHomeState( slHomeAddress[4] );
+                ent.setBusinessState( slBusinessAddress[4] );
         }
 
         if (hasZip) {
-                ent.setHomeZip( (*slHomeAddress)[5] );
-                ent.setBusinessZip( (*slBusinessAddress)[5] );
+                ent.setHomeZip( slHomeAddress[5] );
+                ent.setBusinessZip( slBusinessAddress[5] );
         }
-        
+
         if (hasCountry) {
-                ent.setHomeCountry( (*slHomeAddress)[6] );
-                ent.setBusinessCountry( (*slBusinessAddress)[6] );
+                ent.setHomeCountry( slHomeAddress[6] );
+                ent.setBusinessCountry( slBusinessAddress[6] );
         }
 
         QStringList::ConstIterator it;
         QListIterator<QLineEdit> itLE( listValue );
-        for ( it = slDynamicEntries->begin(); it != slDynamicEntries->end(); ++it, ++itLE) {
+        for ( it = slDynamicEntries.begin(); it != slDynamicEntries.end(); ++it, ++itLE) {
                 if ( *it == "Department" )
                         ent.setDepartment( (*itLE)->text() );
 
                 if ( *it == "Company" )
                         ent.setCompany( (*itLE)->text() );
 
                 if ( *it == "Office" )
                         ent.setOffice( (*itLE)->text() );
 
                 if ( *it == "Profession" )
                         ent.setProfession( (*itLE)->text() );
 
@@ -1462,25 +1490,25 @@ void ContactEditor::saveEntry() {
                 if ( *it == "Anniversary" )
                         ent.setAnniversary( (*itLE)->text() );
 
                 if ( *it == "Nickname" )
                         ent.setNickname( (*itLE)->text() );
 
                 if ( *it == "Children" )
                         ent.setChildren( (*itLE)->text() );
 
         }
 
         QStringList::ConstIterator itV;
-        for ( it = slChooserNames->begin(), itV = slChooserValues->begin(); it != slChooserNames->end(); ++it, ++itV ) {
+        for ( it = slChooserNames.begin(), itV = slChooserValues.begin(); it != slChooserNames.end(); ++it, ++itV ) {
 
                 if ( *it == "Business Phone" || *it == "Work Phone" )
                         ent.setBusinessPhone( *itV );
 /*
                 if ( *it == "Business 2 Phone" )
                         ent.setBusiness2Phone( *itV );
 */
                 if ( *it == "Business Fax" || *it == "Work Fax" )
                         ent.setBusinessFax( *itV );
 
                 if ( *it == "Business Mobile" || *it == "Work Mobile" )
                         ent.setBusinessMobile( *itV );
@@ -1539,25 +1567,25 @@ void ContactEditor::saveEntry() {
                         ent.setMSNIM( *itV );
 
                 if ( *it == "Yahoo IM" )
                         ent.setYahooIM( *itV );
 */
                 if ( *it == "Home Web Page" )
                         ent.setHomeWebpage( *itV );
                 if ( *it == "Business Web Page" || *it == "Work Web Page" )
                         ent.setBusinessWebpage( *itV );
 
 
         }
-        
+
         int gender = cmbGender->currentItem();
         ent.setGender( QString::number( gender ) );
 
         QString str = txtNote->text();
         if ( !str.isNull() )
                 ent.setNotes( str );
 
 }
 
 void parseEmailFrom( const QString &txt, QString &strDefaultEmail,
                      QString &strAll )
 {
diff --git a/core/pim/addressbook/contacteditor.h b/core/pim/addressbook/contacteditor.h
index a58a860..4f3b56e 100644
--- a/core/pim/addressbook/contacteditor.h
+++ b/core/pim/addressbook/contacteditor.h
@@ -41,25 +41,25 @@ const int NAME_S = 7;
 
 class QScrollView;
 class QTabWidget;
 class QMultiLineEdit;
 class QLineEdit;
 class QComboBox;
 class QPushButton;
 class CategorySelect;
 class QLabel;
 
 class ContactEditor : public QDialog {
         Q_OBJECT
-        
+
         public:
                         ContactEditor(const Contact &entry,
                                 const QValueList<int> *newOrderedValues,
                                 QStringList *slNewOrdered,
                                 QWidget *parent = 0,
                                 const char *name = 0,
                                 WFlags fl = 0 );
                 ~ContactEditor();
                 void loadFields();
                 void setNameFocus();
                 Contact entry() const { return ent; }
 
@@ -69,25 +69,25 @@ class ContactEditor : public QDialog {
                 void setEntry(const Contact &entry);
 
         protected slots:
                 void accept();
 
         private:
                 void init();
                 void initMap();
                 void saveEntry();
                 bool isEmpty();
                 void cleanupFields();
                 QString parseName( QString fullName, int type );
-                
+
         private slots:
                 void slotChooser1Change( const QString &textChanged );
                 void slotChooser2Change( const QString &textChanged );
                 void slotChooser3Change( const QString &textChanged );
                 void slotChooser4Change( const QString &textChanged );
                 void slotCmbChooser1Change( int index );
                 void slotCmbChooser2Change( int index );
                 void slotCmbChooser3Change( int index );
                 void slotCmbChooser4Change( int index );
                 void slotAddressTypeChange( int index );
                 void slotAddressChange( const QString &textChanged );
                 void slotAddress2Change( const QString &textChanged );
@@ -100,31 +100,31 @@ class ContactEditor : public QDialog {
 
         private:
                 bool useFullName, hasGender, hasTitle, hasCompany, hasNotes, hasStreet, hasStreet2, hasPOBox, hasCity, hasState, hasZip, hasCountry;
 
                 Contact ent;
 
                 QDialog *dlgNote;
                 QDialog *dlgName;
 
                 QList<QLineEdit> listValue;
                 QList<QLabel> listName;
                 const QValueList<int> *orderedValues;
-                QStringList *slOrdered;
-                QStringList *slDynamicEntries;
+                QStringList slOrdered;
+                QStringList slDynamicEntries;
 
-                QStringList *slHomeAddress;
-                QStringList *slBusinessAddress;
-                QStringList *slChooserNames;
-                QStringList *slChooserValues;
+                QStringList slHomeAddress;
+                QStringList slBusinessAddress;
+                QStringList slChooserNames;
+                QStringList slChooserValues;
 
                 QMultiLineEdit *txtNote;
                 QLabel *lblNote;
 
                 //QLineEdit *txtTitle;
                 QLineEdit *txtFirstName;
                 QLineEdit *txtMiddleName;
                 QLineEdit *txtLastName;
                 QLineEdit *txtSuffix;
 
                 QTabWidget *tabMain;
                 QScrollView *svGeneral;
@@ -134,25 +134,25 @@ class ContactEditor : public QDialog {
                 QLineEdit *txtJobTitle;
                 QLineEdit *txtOrganization;
                 QLineEdit *txtChooserField1;
                 QLineEdit *txtChooserField2;
                 QLineEdit *txtChooserField3;
                 QLineEdit *txtChooserField4;
                 QComboBox *cmbChooserField1;
                 QComboBox *cmbChooserField2;
                 QComboBox *cmbChooserField3;
                 QComboBox *cmbChooserField4;
                 QComboBox *cmbFileAs;
                 CategorySelect *cmbCat;
-                
+
                 QScrollView *svAddress;
                 QLineEdit *txtAddress;
                 //QLineEdit *txtAddress2;
                 //QLineEdit *txtPOBox;
                 QLineEdit *txtCity;
                 QLineEdit *txtState;
                 QLineEdit *txtZip;
                 QComboBox *cmbAddress;
                 QComboBox *cmbCountry;
 
                 QScrollView *svDetails;
                 QComboBox *cmbGender;