Fixed reload after sync- bug

4 files changed, 25 insertions, 65 deletions

diff --git a/core/pim/addressbook/TODO b/core/pim/addressbook/TODO
index 32d2838..7bc906d 100644
--- a/core/pim/addressbook/TODO
+++ b/core/pim/addressbook/TODO
@@ -3,3 +3,4 @@ Stuff todo:
 Urgent:
-- Syncing: abtable not reloaded after sync.
+
+- Font menu is invisible using german translation
 
@@ -22,5 +23,6 @@ Less important:
 
-Should be Fixed:
+Should be Fixed (not absolute sure, need validation):
 - "Nonenglish" translation bug has to be fixed.
 
-Fixed:
\ No newline at end of file
+Fixed:
+- Syncing: abtable not reloaded after sync.
diff --git a/core/pim/addressbook/abtable.cpp b/core/pim/addressbook/abtable.cpp
index 70c070f..91a7171 100644
--- a/core/pim/addressbook/abtable.cpp
+++ b/core/pim/addressbook/abtable.cpp
@@ -472,3 +472,3 @@ void AbTable::resizeRows() {
 
-bool AbTable::save( const QString& /* fn */ )
+bool AbTable::save()
 {
@@ -477,8 +477,7 @@ bool AbTable::save( const QString& /* fn */ )
 	qWarning("abtable:Save data");
-	m_contactdb.save();
-	
-	return true;
+
+	return m_contactdb.save();
 }
 
-void AbTable::load( const QString& /* fn */ )
+void AbTable::load()
 {
@@ -496,8 +495,6 @@ void AbTable::load( const QString& /* fn */ )
 	
-	resort();
-	
 	setUpdatesEnabled( TRUE );
-	
+
 	setSorting( true );
-	//resort();
+	resort();
 }
@@ -505,2 +502,8 @@ void AbTable::load( const QString& /* fn */ )
 
+void AbTable::reload()
+{
+	m_contactdb.reload();
+	load();
+}
+
 void AbTable::realignTable( int row )
diff --git a/core/pim/addressbook/abtable.h b/core/pim/addressbook/abtable.h
index 84aef1e..80c3ca0 100644
--- a/core/pim/addressbook/abtable.h
+++ b/core/pim/addressbook/abtable.h
@@ -79,4 +79,5 @@ public:
     void refresh();
-    bool save( const QString &fn );
-    void load( const QString &fn );
+    bool save();
+    void load();
+    void reload();
 
diff --git a/core/pim/addressbook/addressbook.cpp b/core/pim/addressbook/addressbook.cpp
index 2eb9ddf..93581e8 100644
--- a/core/pim/addressbook/addressbook.cpp
+++ b/core/pim/addressbook/addressbook.cpp
@@ -69,14 +69,2 @@
 #include "picker.h"
-static QString addressbookOldXMLFilename()
-{
-	QString filename = QPEApplication::documentDir() + "addressbook.xml";
-	return filename;
-}
-
-static QString addressbookXMLFilename()
-{
-	QString filename = Global::applicationFileName("addressbook",
-						       "addressbook.xml");
-	return filename;
-}
 
@@ -204,17 +192,2 @@ AddressbookWindow::AddressbookWindow( QWidget *parent, const char *name,
 	// Create Views
-	
-	// This is safe to call without checking to see if it exists...
-	// 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 );
@@ -232,7 +205,3 @@ AddressbookWindow::AddressbookWindow( QWidget *parent, const char *name,
 	
-	abList->load( addressbookXMLFilename() );
-	if ( QFile::exists(addressbookOldXMLFilename()) ) {
-		abList->load( addressbookOldXMLFilename() );
-		QFile::remove(addressbookOldXMLFilename());
-	}
+	abList->load();
 	
@@ -711,3 +680,3 @@ void AddressbookWindow::reload()
 	abList->clear();
-	abList->load( addressbookXMLFilename() );
+	abList->reload();
 }
@@ -717,3 +686,3 @@ void AddressbookWindow::flush()
 	syncing = TRUE;
-	abList->save( addressbookXMLFilename() );
+	abList->save();
 }
@@ -757,5 +726,4 @@ bool AddressbookWindow::save()
 {
-	QString str = addressbookXMLFilename();
-	if ( str.isNull() ) {
-		if ( QMessageBox::critical( 0, tr("Out of space"),
+	if ( !abList->save() ) {
+		if ( QMessageBox::critical( 0, tr( "Out of space" ),
 					    tr("Unable to save information.\n"
@@ -770,16 +738,2 @@ bool AddressbookWindow::save()
 			return FALSE;
-	} else {
-		if ( !abList->save( str ) ) {
-			if ( QMessageBox::critical( 0, tr( "Out of space" ),
-						    tr("Unable to save information.\n"
-						       "Free up some space\n"
-						       "and try again.\n"
-						       "\nQuit anyway?"),
-						    QMessageBox::Yes|QMessageBox::Escape,
-						    QMessageBox::No|QMessageBox::Default )
-			     != QMessageBox::No )
-				return TRUE;
-			else
-				return FALSE;
-		}
 	}