Bugfix: Datebook shouldn't crash if we modify prefences when dayview isn't initialized. :)

1 files changed, 97 insertions, 92 deletions

diff --git a/core/pim/datebook/datebook.cpp b/core/pim/datebook/datebook.cpp
index 1bc82a5..829f8f8 100644
--- a/core/pim/datebook/datebook.cpp
+++ b/core/pim/datebook/datebook.cpp
@@ -216,34 +216,34 @@ DateBook::DateBook( QWidget *parent, const char *, WFlags f )
     channel = new QCopChannel( "QPE/Datebook", this );
     connect( channel, SIGNAL(received(const QCString&, const QByteArray&)),
 	     this, SLOT(receive(const QCString&, const QByteArray&)) );
     qDebug("olle\n");
 #endif
 #endif
 
     qDebug("done t=%d", t.elapsed() );
-    
+
     /*
      *  Here is a problem description:
      *  When Weekview is the default view
      *  a DateBookWeekView get's created
      *  redraw() get's called. So what?
      *  Remember that we're still in the c'tor
      *  and no final layout has happened? Ok
      *  now all Events get arranged. Their x
      *  position get's determined by a QHeader
      *  position. But the QHeader isn't layouted or
      *  at the right position. redraw() is a slot
      *  so we'll call it then via a singleShot
      *  from view()
      */
-    if( needEvilHack ){
-	QTimer::singleShot( 500, this, SLOT(viewWeek()) );
-    }
+	if( needEvilHack ){
+		QTimer::singleShot( 500, this, SLOT(viewWeek()) );
+	}
 }
 
 void DateBook::receive( const QCString &msg, const QByteArray &data )
 {
     QDataStream stream( data, IO_ReadOnly );
     if ( msg == "timeChange(QString)" ) {
 	// update active view!
 	if ( dayAction->isOn() )
@@ -271,38 +271,42 @@ void DateBook::slotSettings()
     frmSettings.setStartTime( startTime );
     frmSettings.setAlarmPreset( aPreset, presetTime );
     frmSettings.setJumpToCurTime( bJumpToCurTime );
     frmSettings.setRowStyle( rowStyle );
 #if defined (Q_WS_QWS) || defined(_WS_QWS_)
     frmSettings.showMaximized();
 #endif
 
-    if ( frmSettings.exec() ) {
-	aPreset = frmSettings.alarmPreset();
-	presetTime = frmSettings.presetTime();
-	startTime = frmSettings.startTime();
-	bJumpToCurTime = frmSettings.jumpToCurTime();
-	rowStyle = frmSettings.rowStyle();
-	if ( dayView )
-	    dayView->setStartViewTime( startTime );
-	    dayView->setJumpToCurTime( bJumpToCurTime );
-	    dayView->setRowStyle( rowStyle );
-	if ( weekView )
-	    weekView->setStartViewTime( startTime );
-	saveSettings();
-
-	// make the change obvious
-	if ( views->visibleWidget() ) {
-	    if ( views->visibleWidget() == dayView )
-		dayView->redraw();
-	    else if ( views->visibleWidget() == weekView )
-		weekView->redraw();
+	if ( frmSettings.exec() ) {
+
+		aPreset = frmSettings.alarmPreset();
+		presetTime = frmSettings.presetTime();
+		startTime = frmSettings.startTime();
+		bJumpToCurTime = frmSettings.jumpToCurTime();
+		rowStyle = frmSettings.rowStyle();
+
+		if ( dayView ) {
+			dayView->setStartViewTime( startTime );
+			dayView->setJumpToCurTime( bJumpToCurTime );
+			dayView->setRowStyle( rowStyle );
+		}
+		if ( weekView ) {
+			weekView->setStartViewTime( startTime );
+		}
+		saveSettings();
+
+		// make the change obvious
+		if ( views->visibleWidget() ) {
+			if ( views->visibleWidget() == dayView )
+				dayView->redraw();
+			else if ( views->visibleWidget() == weekView )
+				weekView->redraw();
+		}
 	}
-    }
 }
 
 void DateBook::fileNew()
 {
     slotNewEventFromKey("");
 }
 
 QString DateBook::checkEvent(const Event &e)
@@ -346,41 +350,41 @@ QDate DateBook::currentDate()
     } else if ( monthView && views->visibleWidget() == monthView ) {
 	d = monthView->selectedDate();
     }
 
     return d;
 }
 
 void DateBook::view(int v, const QDate &d) {
-    if (v==DAY) {
-	initDay();
-	dayAction->setOn( TRUE );
-	dayView->setDate( d );
-	views->raiseWidget( dayView );
-	dayView->redraw();
-    } else if (v==WEEK) {
-	initWeek();
-	weekAction->setOn( TRUE );
-	weekView->setDate( d );
-	views->raiseWidget( weekView );
-	weekView->redraw();
-    } else if (v==WEEKLST) {
-	initWeekLst();
-	weekLstAction->setOn( TRUE );
-	weekLstView->setDate(d);
-	views->raiseWidget( weekLstView );
-	weekLstView->redraw();
-    } else if (v==MONTH) {
-	initMonth();
-	monthAction->setOn( TRUE );
-	monthView->setDate( d.year(), d.month(), d.day() );
-	views->raiseWidget( monthView );
-	monthView->redraw();
-    }
+	if (v==DAY) {
+		initDay();
+		dayAction->setOn( TRUE );
+		dayView->setDate( d );
+		views->raiseWidget( dayView );
+		dayView->redraw();
+	} else if (v==WEEK) {
+		initWeek();
+		weekAction->setOn( TRUE );
+		weekView->setDate( d );
+		views->raiseWidget( weekView );
+		weekView->redraw();
+	} else if (v==WEEKLST) {
+		initWeekLst();
+		weekLstAction->setOn( TRUE );
+		weekLstView->setDate(d);
+		views->raiseWidget( weekLstView );
+		weekLstView->redraw();
+	} else if (v==MONTH) {
+		initMonth();
+		monthAction->setOn( TRUE );
+		monthView->setDate( d.year(), d.month(), d.day() );
+		views->raiseWidget( monthView );
+		monthView->redraw();
+	}
 }
 
 void DateBook::viewDefault(const QDate &d) {
   Config config("DateBook");
   config.setGroup("Main");
   int current=config.readNumEntry("defaultview", DAY);
 
   view(current,d);
@@ -509,67 +513,68 @@ void DateBook::addEvent( const Event &e )
 void DateBook::showDay( int year, int month, int day )
 {
     QDate d(year, month, day);
     view(DAY,d);
 }
 
 void DateBook::initDay()
 {
-    if ( !dayView ) {
-	dayView = new DateBookDay( ampm, onMonday, db, views, "day view" );
-	views->addWidget( dayView, DAY );
-	dayView->setStartViewTime( startTime );
-	dayView->setJumpToCurTime( bJumpToCurTime );
-	dayView->setRowStyle( rowStyle );
-	connect( this, SIGNAL( newEvent() ),
-		 dayView, SLOT( redraw() ) );
-	connect( dayView, SIGNAL( newEvent() ),
-		 this, SLOT( fileNew() ) );
-	connect( dayView, SIGNAL( removeEvent( const Event & ) ),
-		 this, SLOT( removeEvent( const Event & ) ) );
-	connect( dayView, SIGNAL( editEvent( const Event & ) ),
-		 this, SLOT( editEvent( const Event & ) ) );
-	connect( dayView, SIGNAL( duplicateEvent( const Event & ) ),
-		 this, SLOT( duplicateEvent( const Event & ) ) );
-	connect( dayView, SIGNAL( beamEvent( const Event & ) ),
-		 this, SLOT( beamEvent( const Event & ) ) );
-	connect( dayView, SIGNAL(sigNewEvent(const QString &)),
-		 this, SLOT(slotNewEventFromKey(const QString &)) );
+	if ( !dayView ) {
+		dayView = new DateBookDay( ampm, onMonday, db, views, "day view" );
+		views->addWidget( dayView, DAY );
+		dayView->setStartViewTime( startTime );
+		dayView->setJumpToCurTime( bJumpToCurTime );
+		dayView->setRowStyle( rowStyle );
+		connect( this, SIGNAL( newEvent() ),
+			dayView, SLOT( redraw() ) );
+		connect( dayView, SIGNAL( newEvent() ),
+			this, SLOT( fileNew() ) );
+		connect( dayView, SIGNAL( removeEvent( const Event & ) ),
+			this, SLOT( removeEvent( const Event & ) ) );
+		connect( dayView, SIGNAL( editEvent( const Event & ) ),
+			this, SLOT( editEvent( const Event & ) ) );
+		connect( dayView, SIGNAL( duplicateEvent( const Event & ) ),
+			this, SLOT( duplicateEvent( const Event & ) ) );
+		connect( dayView, SIGNAL( beamEvent( const Event & ) ),
+			this, SLOT( beamEvent( const Event & ) ) );
+		connect( dayView, SIGNAL(sigNewEvent(const QString &)),
+			this, SLOT(slotNewEventFromKey(const QString &)) );
     }
 }
 
 void DateBook::initWeek()
 {
-    if ( !weekView ) {
-	weekView = new DateBookWeek( ampm, onMonday, db, views, "week view" );
-	weekView->setStartViewTime( startTime );
-	views->addWidget( weekView, WEEK );
-	connect( weekView, SIGNAL( showDate( int, int, int ) ),
-		 this, SLOT( showDay( int, int, int ) ) );
-	connect( this, SIGNAL( newEvent() ),
-		 weekView, SLOT( redraw() ) );
-    }
-    //But also get it right: the year that we display can be different
-    //from the year of the current date. So, first find the year
-    //number of the current week.
-
-    int yearNumber, totWeeks;
-    calcWeek( currentDate(), totWeeks, yearNumber, onMonday );
+	if ( !weekView ) {
+		weekView = new DateBookWeek( ampm, onMonday, db, views, "week view" );
+		weekView->setStartViewTime( startTime );
+		views->addWidget( weekView, WEEK );
+		connect( weekView, SIGNAL( showDate( int, int, int ) ),
+			this, SLOT( showDay( int, int, int ) ) );
+		connect( this, SIGNAL( newEvent() ),
+			weekView, SLOT( redraw() ) );
+	}
 
-    QDate d = QDate( yearNumber, 12, 31 );
-    calcWeek( d, totWeeks, yearNumber, onMonday );
+	//But also get it right: the year that we display can be different
+	//from the year of the current date. So, first find the year
+	//number of the current week.
+	int yearNumber, totWeeks;
+	calcWeek( currentDate(), totWeeks, yearNumber, onMonday );
 
-    while ( totWeeks == 1 ) {
-	d = d.addDays( -1 );
+	QDate d = QDate( yearNumber, 12, 31 );
 	calcWeek( d, totWeeks, yearNumber, onMonday );
-    }
-    if ( totWeeks != weekView->totalWeeks() )
-	weekView->setTotalWeeks( totWeeks );
+
+	while ( totWeeks == 1 ) {
+		d = d.addDays( -1 );
+		calcWeek( d, totWeeks, yearNumber, onMonday );
+	}
+	if ( totWeeks != weekView->totalWeeks() )
+		weekView->setTotalWeeks( totWeeks );
 }
+
 void DateBook::initWeekLst() {
   if ( !weekLstView ) {
       weekLstView = new DateBookWeekLst( ampm, onMonday, db,
 					 views, "weeklst view" );
       views->addWidget( weekLstView, WEEKLST );
 
       //weekLstView->setStartViewTime( startTime );
       connect( weekLstView, SIGNAL( showDate( int, int, int ) ),