1. datebookweeklist - a week has SEVEN days, not SIX. I'm wondering why

   events on sunday never showed there ;)
2. extra file datebooktypes contains some special classes used by datebook
   so other classes must not include the big mainheader file.
3. added support for holiday-plugins to week-views. Month view is a little
   bit more tricky 'cause TT has never heard about virtual methods and so
   I have the choice between complete copy the monthview and make the modifications
   or changing the interface of datebookdb-class to virtual. both isn't nice.

1 files changed, 245 insertions, 244 deletions

diff --git a/core/pim/datebook/datebookweek.cpp b/core/pim/datebook/datebookweek.cpp
index 7503751..a509d89 100644
--- a/core/pim/datebook/datebookweek.cpp
+++ b/core/pim/datebook/datebookweek.cpp
@@ -21,2 +21,3 @@
 #include "datebookweekheaderimpl.h"
+#include "datebooktypes.h"
 
@@ -54,3 +55,3 @@ void DateBookWeekItem::setGeometry( int x, int y, int w, int h )
 DateBookWeekView::DateBookWeekView( bool ap, bool startOnMonday,
-                                    QWidget *parent, const char *name )
+                    QWidget *parent, const char *name )
     : QScrollView( parent, name ), ampm( ap ), bOnMonday( startOnMonday ),
@@ -74,3 +75,3 @@ DateBookWeekView::DateBookWeekView( bool ap, bool startOnMonday,
     QObject::connect(qApp, SIGNAL(clockChanged(bool)),
-            this, SLOT(slotChangeClock(bool)));
+        this, SLOT(slotChangeClock(bool)));
 
@@ -87,3 +88,3 @@ void DateBookWeekView::initNames()
     if ( bFirst ) {
-        if ( bOnMonday ) {
+    if ( bOnMonday ) {
                     for ( int i = 1; i<=7; i++  )  {
@@ -92,3 +93,3 @@ void DateBookWeekView::initNames()
 
-        } else {
+    } else {
                     header->addLabel( Calendar::nameOfDay( 7 ) );
@@ -98,6 +99,6 @@ void DateBookWeekView::initNames()
                 }
-        bFirst = false;
+    bFirst = false;
     } else {
-        // we are change things...
-        if ( bOnMonday ) {
+    // we are change things...
+    if ( bOnMonday ) {
                     for ( int i = 1; i<=7; i++  )  {
@@ -106,3 +107,3 @@ void DateBookWeekView::initNames()
 
-        } else {
+    } else {
                     header->setLabel( 1, Calendar::nameOfDay( 7 ) );
@@ -112,3 +113,3 @@ void DateBookWeekView::initNames()
 
-        }
+    }
     }
@@ -120,12 +121,12 @@ void DateBookWeekView::showEvents( QValueList<EffectiveEvent> &ev )
 {
-        items.clear();
-        QValueListIterator<EffectiveEvent> it;
-        for ( it = ev.begin(); it != ev.end(); ++it ) {
-                DateBookWeekItem *i = new DateBookWeekItem( *it );
-                        if(!((i->event().end().hour()==0) && (i->event().end().minute()==0) && (i->event().startDate()!=i->event().date()))) {// Skip events ending at 00:00 starting at another day.
-                        positionItem( i );
-                        items.append( i );
-                }
-        }
-        viewport()->update();
+    items.clear();
+    QValueListIterator<EffectiveEvent> it;
+    for ( it = ev.begin(); it != ev.end(); ++it ) {
+        DateBookWeekItem *i = new DateBookWeekItem( *it );
+        if(!((i->event().end().hour()==0) && (i->event().end().minute()==0) && (i->event().startDate()!=i->event().date()))) {  // Skip events ending at 00:00 starting at another day.
+            positionItem( i );
+            items.append( i );
+        }
+    }
+    viewport()->update();
 }
@@ -152,7 +153,7 @@ static inline int db_round30min( int m )
     if ( m < 15 )
-        m = 0;
+    m = 0;
     else if ( m < 45 )
-        m = 1;
+    m = 1;
     else
-        m = 2;
+    m = 2;
 
@@ -164,3 +165,3 @@ void DateBookWeekView::alterDay( int day )
     if ( !bOnMonday ) {
-            day--;
+        day--;
     }
@@ -186,7 +187,7 @@ void DateBookWeekView::positionItem( DateBookWeekItem *i )
     if ( ev.event().type() == Event::AllDay ) {
-                h = (48 * rowHeight) / 2;
-                y = 0;
-        } else {
-                h=y2-y;
-                if ( h < (1*rowHeight)/2 ) h = (1*rowHeight)/2;
+        h = (48 * rowHeight) / 2;
+        y = 0;
+    } else {
+        h=y2-y;
+        if ( h < (1*rowHeight)/2 ) h = (1*rowHeight)/2;
     }
@@ -195,6 +196,6 @@ void DateBookWeekView::positionItem( DateBookWeekItem *i )
     if ( !bOnMonday ) {
-        if ( dow == 7 )
-            dow = 1;
-        else
-            dow++;
+    if ( dow == 7 )
+        dow = 1;
+    else
+        dow++;
     }
@@ -204,5 +205,5 @@ void DateBookWeekView::positionItem( DateBookWeekItem *i )
     do {
-        i->setGeometry( x, y, Width, h );
-        isect = intersects( i );
-        x += Width - 1;
+    i->setGeometry( x, y, Width, h );
+    isect = intersects( i );
+    x += Width - 1;
     } while ( isect && x < xlim );
@@ -220,8 +221,8 @@ DateBookWeekItem *DateBookWeekView::intersects( const DateBookWeekItem *item )
     for ( ; it.current(); ++it ) {
-        DateBookWeekItem *i = it.current();
-        if ( i != item ) {
-            if ( i->geometry().intersects( geom ) ) {
-                return i;
-            }
-        }
+    DateBookWeekItem *i = it.current();
+    if ( i != item ) {
+        if ( i->geometry().intersects( geom ) ) {
+        return i;
+        }
+    }
     }
@@ -235,8 +236,8 @@ void DateBookWeekView::contentsMousePressEvent( QMouseEvent *e )
     for ( ; it.current(); ++it ) {
-        DateBookWeekItem *i = it.current();
-        if ( i->geometry().contains( e->pos() ) ) {
-            showingEvent = true;
-            emit signalShowEvent( i->event() );
-            break;
-        }
+    DateBookWeekItem *i = it.current();
+    if ( i->geometry().contains( e->pos() ) ) {
+        showingEvent = true;
+        emit signalShowEvent( i->event() );
+        break;
+    }
     }
@@ -247,11 +248,11 @@ void DateBookWeekView::contentsMouseReleaseEvent( QMouseEvent *e )
     if ( showingEvent ) {
-        showingEvent = false;
-        emit signalHideEvent();
+    showingEvent = false;
+    emit signalHideEvent();
     } else {
-        int d = header->sectionAt( e->pos().x() );
-        if ( d > 0 ) {
-        //     if ( !bOnMonday )
-        //      d--;
-            emit showDay( d );
-        }
+    int d = header->sectionAt( e->pos().x() );
+    if ( d > 0 ) {
+//      if ( !bOnMonday )
+//      d--;
+        emit showDay( d );
+    }
     }
@@ -264,3 +265,3 @@ void DateBookWeekView::drawContents( QPainter *p, int cx, int cy, int cw, int ch
     for ( int i = 1; i <= 7; i++ )
-        p->drawLine( header->sectionPos(i)-2, cy, header->sectionPos(i)-2, cy+ch );
+    p->drawLine( header->sectionPos(i)-2, cy, header->sectionPos(i)-2, cy+ch );
 
@@ -268,30 +269,30 @@ void DateBookWeekView::drawContents( QPainter *p, int cx, int cy, int cw, int ch
     for ( int t = 0; t < 24; t++ ) {
-        int y = t*rowHeight;
-        if ( QRect( 1, y, 20, rowHeight ).intersects( ur ) ) {
-            QString s;
-            if ( ampm ) {
-                if ( t == 0 )
-                    s = QString::number( 12 );
-                else if ( t == 12 )
-                    s = QString::number(12) + tr( "p" );
-                else if ( t > 12 ) {
-                    if ( t - 12 < 10 )
-                        s = " ";
-                    else
-                        s = "";
-                    s += QString::number( t - 12 ) + tr("p");
-                } else {
-                    if ( 12 - t < 3 )
-                        s = "";
-                    else
-                        s = " ";
-                    s += QString::number( t );
-                }
-            } else {
-                s = QString::number( t );
-                if ( s.length() == 1 )
-                    s.prepend( "0" );
-            }
-            p->drawText( 1, y+p->fontMetrics().ascent()+1, s );
-        }
+    int y = t*rowHeight;
+    if ( QRect( 1, y, 20, rowHeight ).intersects( ur ) ) {
+        QString s;
+        if ( ampm ) {
+        if ( t == 0 )
+            s = QString::number( 12 );
+        else if ( t == 12 )
+            s = QString::number(12) + tr( "p" );
+        else if ( t > 12 ) {
+            if ( t - 12 < 10 )
+            s = " ";
+            else
+            s = "";
+            s += QString::number( t - 12 ) + tr("p");
+        } else {
+            if ( 12 - t < 3 )
+            s = "";
+            else
+            s = " ";
+            s += QString::number( t );
+        }
+        } else {
+        s = QString::number( t );
+        if ( s.length() == 1 )
+            s.prepend( "0" );
+        }
+        p->drawText( 1, y+p->fontMetrics().ascent()+1, s );
+    }
     }
@@ -300,7 +301,7 @@ void DateBookWeekView::drawContents( QPainter *p, int cx, int cy, int cw, int ch
     for ( ; it.current(); ++it ) {
-        DateBookWeekItem *i = it.current();
-        if ( i->geometry().intersects( ur ) ) {
-            p->setBrush( i->color() );
-            p->drawRect( i->geometry() );
-        }
+    DateBookWeekItem *i = it.current();
+    if ( i->geometry().intersects( ur ) ) {
+        p->setBrush( i->color() );
+        p->drawRect( i->geometry() );
+    }
     }
@@ -318,3 +319,3 @@ void DateBookWeekView::resizeEvent( QResizeEvent *e )
     header->setGeometry( leftMargin()+frameWidth()+frameRect().left() , frameWidth(),
-                          visibleWidth(), header->sizeHint().height() );
+              visibleWidth(), header->sizeHint().height() );
     setMargins( 0, header->sizeHint().height(), 0, 0 );
@@ -330,3 +331,3 @@ void DateBookWeekView::resizeEvent( QResizeEvent *e )
     for ( int i = 1; i < 7; i++ )
-        header->resizeSection( i, sw );
+    header->resizeSection( i, sw );
     header->resizeSection( 7, avail - hourWidth - sw*6 );
@@ -342,4 +343,4 @@ void DateBookWeekView::setStartOfWeek( bool bStartOnMonday )
 
-DateBookWeek::DateBookWeek( bool ap, bool startOnMonday, DateBookDB *newDB,
-                            QWidget *parent, const char *name )
+DateBookWeek::DateBookWeek( bool ap, bool startOnMonday, DateBookDBHoliday *newDB,
+                QWidget *parent, const char *name )
     : QWidget( parent, name ),
@@ -350,24 +351,24 @@ DateBookWeek::DateBookWeek( bool ap, bool startOnMonday, DateBookDB *newDB,
 {
-        setFocusPolicy(StrongFocus);
-        QVBoxLayout *vb = new QVBoxLayout( this );
-        header = new DateBookWeekHeader( bStartOnMonday, this );
-        view = new DateBookWeekView( ampm, startOnMonday, this );
-        vb->addWidget( header );
-        vb->addWidget( view );
+    setFocusPolicy(StrongFocus);
+    QVBoxLayout *vb = new QVBoxLayout( this );
+    header = new DateBookWeekHeader( bStartOnMonday, this );
+    view = new DateBookWeekView( ampm, startOnMonday, this );
+    vb->addWidget( header );
+    vb->addWidget( view );
 
-        lblDesc = new QLabel( this, "event label" );
-        lblDesc->setFrameStyle( QFrame::Plain | QFrame::Box );
-        lblDesc->setBackgroundColor( yellow );
-        lblDesc->hide();
+    lblDesc = new QLabel( this, "event label" );
+    lblDesc->setFrameStyle( QFrame::Plain | QFrame::Box );
+    lblDesc->setBackgroundColor( yellow );
+    lblDesc->hide();
 
-        tHide = new QTimer( this );
+    tHide = new QTimer( this );
 
-        connect( view, SIGNAL( showDay(int) ), this, SLOT( showDay(int) ) );
-        connect( view, SIGNAL(signalShowEvent(const EffectiveEvent&)), this, SLOT(slotShowEvent(const EffectiveEvent&)) );
-        connect( view, SIGNAL(signalHideEvent()), this, SLOT(slotHideEvent()) );
-        connect( header, SIGNAL( dateChanged(QDate&) ), this, SLOT( dateChanged(QDate&) ) );
-        connect( tHide, SIGNAL( timeout() ), lblDesc, SLOT( hide() ) );
-        connect( qApp, SIGNAL(weekChanged(bool)), this, SLOT(slotWeekChanged(bool)) );
-        connect( qApp, SIGNAL(clockChanged(bool)), this, SLOT(slotClockChanged(bool)));
-        setDate(QDate::currentDate());
+    connect( view, SIGNAL( showDay(int) ), this, SLOT( showDay(int) ) );
+    connect( view, SIGNAL(signalShowEvent(const EffectiveEvent&)), this, SLOT(slotShowEvent(const EffectiveEvent&)) );
+    connect( view, SIGNAL(signalHideEvent()), this, SLOT(slotHideEvent()) );
+    connect( header, SIGNAL( dateChanged(QDate&) ), this, SLOT( dateChanged(QDate&) ) );
+    connect( tHide, SIGNAL( timeout() ), lblDesc, SLOT( hide() ) );
+    connect( qApp, SIGNAL(weekChanged(bool)), this, SLOT(slotWeekChanged(bool)) );
+    connect( qApp, SIGNAL(clockChanged(bool)), this, SLOT(slotClockChanged(bool)));
+    setDate(QDate::currentDate());
 }
@@ -377,16 +378,16 @@ void DateBookWeek::keyPressEvent(QKeyEvent *e)
     switch(e->key()) {
-        case Key_Up:
-            view->scrollBy(0, -20);
-            break;
-        case Key_Down:
-            view->scrollBy(0, 20);
-            break;
-        case Key_Left:
-               setDate(date().addDays(-7));
-            break;
-        case Key_Right:
-            setDate(date().addDays(7));
-            break;
-        default:
-            e->ignore();
+    case Key_Up:
+        view->scrollBy(0, -20);
+        break;
+    case Key_Down:
+        view->scrollBy(0, 20);
+        break;
+    case Key_Left:
+        setDate(date().addDays(-7));
+        break;
+    case Key_Right:
+        setDate(date().addDays(7));
+        break;
+    default:
+        e->ignore();
     }
@@ -396,12 +397,12 @@ void DateBookWeek::showDay( int day )
 {
-        QDate d=bdate;
+    QDate d=bdate;
 
-        // Calculate offset to first day of week.
-        int dayoffset=d.dayOfWeek() % 7;
+    // Calculate offset to first day of week.
+    int dayoffset=d.dayOfWeek() % 7;
 
-        if(bStartOnMonday) dayoffset--;
+    if(bStartOnMonday) dayoffset--;
 
-        day--;
-        d=d.addDays(day-dayoffset);
-        emit showDate( d.year(), d.month(), d.day() );
+    day--;
+    d=d.addDays(day-dayoffset);
+    emit showDate( d.year(), d.month(), d.day() );
 }
@@ -410,3 +411,3 @@ void DateBookWeek::setDate( int y, int m, int d )
 {
-        setDate(QDate(y, m, d));
+    setDate(QDate(y, m, d));
 }
@@ -415,5 +416,5 @@ void DateBookWeek::setDate(QDate newdate)
 {
-        bdate=newdate;
-        dow = newdate.dayOfWeek();
-        header->setDate( newdate );
+    bdate=newdate;
+    dow = newdate.dayOfWeek();
+    header->setDate( newdate );
 }
@@ -422,4 +423,4 @@ void DateBookWeek::dateChanged( QDate &newdate )
 {
-        bdate=newdate;
-        getEvents();
+    bdate=newdate;
+    getEvents();
 }
@@ -428,3 +429,3 @@ QDate DateBookWeek::date() const
 {
-        return bdate;
+    return bdate;
 }
@@ -433,8 +434,8 @@ void DateBookWeek::getEvents()
 {
-        QDate startWeek = weekDate();
+    QDate startWeek = weekDate();
 
-        QDate endWeek = startWeek.addDays( 6 );
-        QValueList<EffectiveEvent> eventList = db->getEffectiveEvents(startWeek, endWeek);
-        view->showEvents( eventList );
-        view->moveToHour( startTime );
+    QDate endWeek = startWeek.addDays( 6 );
+    QValueList<EffectiveEvent> eventList = db->getEffectiveEvents(startWeek, endWeek);
+    view->showEvents( eventList );
+    view->moveToHour( startTime );
 }
@@ -442,3 +443,3 @@ void DateBookWeek::getEvents()
 void DateBookWeek::generateAllDayTooltext( QString& text ) {
-        text += "<b>" + tr("This is an all day event.") + "</b><br>";
+    text += "<b>" + tr("This is an all day event.") + "</b><br>";
 }
@@ -446,13 +447,13 @@ void DateBookWeek::generateAllDayTooltext( QString& text ) {
 void DateBookWeek::generateNormalTooltext( QString& str, const EffectiveEvent &ev ) {
-        str += "<b>" + QObject::tr("Start") + "</b>: ";
-        str += TimeString::timeString( ev.event().start().time(), ampm, FALSE );
-        if( ev.startDate()!=ev.endDate() ) {
-                str += " <i>" + TimeString::longDateString( ev.startDate() )+"</i>";
-        }
-        str += "<br>";
-        str += "<b>" + QObject::tr("End") + "</b>: ";
-        str += TimeString::timeString( ev.event().end().time(), ampm, FALSE );
-        if( ev.startDate()!=ev.endDate() ) {
-                str += " <i>" + TimeString::longDateString( ev.endDate() ) + "</i>";
-        }
+    str += "<b>" + QObject::tr("Start") + "</b>: ";
+    str += TimeString::timeString( ev.event().start().time(), ampm, FALSE );
+    if( ev.startDate()!=ev.endDate() ) {
+        str += " <i>" + TimeString::longDateString( ev.startDate() )+"</i>";
+    }
+    str += "<br>";
+    str += "<b>" + QObject::tr("End") + "</b>: ";
+    str += TimeString::timeString( ev.event().end().time(), ampm, FALSE );
+    if( ev.startDate()!=ev.endDate() ) {
+        str += " <i>" + TimeString::longDateString( ev.endDate() ) + "</i>";
+    }
 }
@@ -469,5 +470,5 @@ void DateBookWeek::slotShowEvent( const EffectiveEvent &ev )
     while ( where != -1 ) {
-        strDesc.remove( where, 1 );
-        strDesc.insert( where, "&#60;" );
-        where = strDesc.find( "<", where );
+    strDesc.remove( where, 1 );
+    strDesc.insert( where, "&#60;" );
+    where = strDesc.find( "<", where );
     }
@@ -479,5 +480,5 @@ void DateBookWeek::slotShowEvent( const EffectiveEvent &ev )
 //     while ( where != -1 ) {
-        // strCat.remove( where, 1 );
-        // strCat.insert( where, "&#60;" );
-        // where = strCat.find( "<", where );
+//  strCat.remove( where, 1 );
+//  strCat.insert( where, "&#60;" );
+//  where = strCat.find( "<", where );
 //     }
@@ -486,5 +487,5 @@ void DateBookWeek::slotShowEvent( const EffectiveEvent &ev )
     while ( where != -1 ) {
-        strLocation.remove( where, 1 );
-        strLocation.insert( where, "&#60;" );
-        where = strLocation.find( "<", where );
+    strLocation.remove( where, 1 );
+    strLocation.insert( where, "&#60;" );
+    where = strLocation.find( "<", where );
     }
@@ -494,5 +495,5 @@ void DateBookWeek::slotShowEvent( const EffectiveEvent &ev )
     while ( where != -1 ) {
-        strNote.remove( where, 1 );
-        strNote.insert( where, "&#60;" );
-        where = strNote.find( "<", where );
+    strNote.remove( where, 1 );
+    strNote.insert( where, "&#60;" );
+    where = strNote.find( "<", where );
     }
@@ -503,3 +504,3 @@ void DateBookWeek::slotShowEvent( const EffectiveEvent &ev )
                   + "<br>" + TimeString::longDateString( ev.date() )
-                  + "<br>";
+              + "<br>";
 
@@ -542,10 +543,10 @@ void DateBookWeek::slotYearChanged( int y )
 {
-        int totWeek;
-        QDate d( y, 12, 31 );
-        int throwAway;
-        calcWeek( d, totWeek, throwAway, bStartOnMonday );
-        while ( totWeek == 1 ) {
-                d = d.addDays( -1 );
-                calcWeek( d, totWeek, throwAway, bStartOnMonday );
-        }
+    int totWeek;
+    QDate d( y, 12, 31 );
+    int throwAway;
+    calcWeek( d, totWeek, throwAway, bStartOnMonday );
+    while ( totWeek == 1 ) {
+        d = d.addDays( -1 );
+        calcWeek( d, totWeek, throwAway, bStartOnMonday );
+    }
 }
@@ -554,6 +555,6 @@ void DateBookWeek::slotWeekChanged( bool onMonday )
 {
-        bStartOnMonday = onMonday;
-        view->setStartOfWeek( bStartOnMonday );
-        header->setStartOfWeek( bStartOnMonday );
-        redraw();
+    bStartOnMonday = onMonday;
+    view->setStartOfWeek( bStartOnMonday );
+    header->setStartOfWeek( bStartOnMonday );
+    redraw();
 }
@@ -562,3 +563,3 @@ void DateBookWeek::slotClockChanged( bool ap )
 {
-        ampm = ap;
+    ampm = ap;
 }
@@ -568,11 +569,11 @@ QDate DateBookWeek::weekDate() const
 {
-        QDate d=bdate;
+    QDate d=bdate;
 
-        // Calculate offset to first day of week.
-        int dayoffset=d.dayOfWeek();
-        if(bStartOnMonday) dayoffset--;
-        else if( dayoffset == 7 )
-            dayoffset = 0;
+    // Calculate offset to first day of week.
+    int dayoffset=d.dayOfWeek();
+    if(bStartOnMonday) dayoffset--;
+    else if( dayoffset == 7 )
+        dayoffset = 0;
 
-        return d.addDays(-dayoffset);
+    return d.addDays(-dayoffset);
 }
@@ -602,3 +603,3 @@ QDate DateBookWeek::weekDate() const
 bool calcWeek( const QDate &d, int &week, int &year,
-               bool startOnMonday )
+           bool startOnMonday )
 {
@@ -613,58 +614,58 @@ bool calcWeek( const QDate &d, int &week, int &year,
     if ( !d.isValid() )
-        return false;
+    return false;
 
     if ( startOnMonday ) {
-        // find the Jan1Weekday;
-        if ( d.dayOfYear() <= ( 8 - jan1WeekDay) && jan1WeekDay > 4 ) {
-            yearNumber = d.year() - 1;
-            if ( jan1WeekDay == 5 || ( jan1WeekDay == 6 && QDate::leapYear(yearNumber) ) )
-                weekNumber = 53;
-            else
-                weekNumber = 52;
-        } else
-            yearNumber = d.year();
-        if ( yearNumber == d.year() ) {
-            int totalDays = 365;
-            if ( QDate::leapYear(yearNumber) )
-                totalDays++;
-            if ( ((totalDays - d.dayOfYear()) < (4 - dayOfWeek) )
-                 || (jan1WeekDay == 7) && (totalDays - d.dayOfYear()) < 3) {
-                yearNumber++;
-                weekNumber = 1;
-            }
-        }
-        if ( yearNumber == d.year() ) {
-            int j = d.dayOfYear() + (7 - dayOfWeek) + ( jan1WeekDay - 1 );
-            weekNumber = j / 7;
-            if ( jan1WeekDay > 4 )
-                weekNumber--;
-        }
+    // find the Jan1Weekday;
+    if ( d.dayOfYear() <= ( 8 - jan1WeekDay) && jan1WeekDay > 4 ) {
+        yearNumber = d.year() - 1;
+        if ( jan1WeekDay == 5 || ( jan1WeekDay == 6 && QDate::leapYear(yearNumber) ) )
+        weekNumber = 53;
+        else
+        weekNumber = 52;
+    } else
+        yearNumber = d.year();
+    if ( yearNumber == d.year() ) {
+        int totalDays = 365;
+        if ( QDate::leapYear(yearNumber) )
+        totalDays++;
+        if ( ((totalDays - d.dayOfYear()) < (4 - dayOfWeek) )
+         || (jan1WeekDay == 7) && (totalDays - d.dayOfYear()) < 3) {
+        yearNumber++;
+        weekNumber = 1;
+        }
+    }
+    if ( yearNumber == d.year() ) {
+        int j = d.dayOfYear() + (7 - dayOfWeek) + ( jan1WeekDay - 1 );
+        weekNumber = j / 7;
+        if ( jan1WeekDay > 4 )
+        weekNumber--;
+    }
     } else {
-        // it's better to keep these cases separate...
-        if ( d.dayOfYear() <= (7 - jan1WeekDay) && jan1WeekDay > 4
-             && jan1WeekDay != 7 ) {
-            yearNumber = d.year() - 1;
-            if ( jan1WeekDay == 6
-                 || (jan1WeekDay == 7 && QDate::leapYear(yearNumber) ) ) {
-                weekNumber = 53;
-            }else
-                weekNumber = 52;
-        } else
-            yearNumber = d.year();
-        if ( yearNumber == d.year() ) {
-            int totalDays = 365;
-            if ( QDate::leapYear( yearNumber ) )
-                totalDays++;
-            if ( ((totalDays - d.dayOfYear()) < (4 - dayOfWeek % 7)) ) {
-                yearNumber++;
-                weekNumber = 1;
-            }
-        }
-        if ( yearNumber == d.year() ) {
-            int j = d.dayOfYear() + (7 - dayOfWeek % 7) + ( jan1WeekDay - 1 );
-            weekNumber = j / 7;
-            if ( jan1WeekDay > 4 ) {
-                weekNumber--;
-            }
-        }
+    // it's better to keep these cases separate...
+    if ( d.dayOfYear() <= (7 - jan1WeekDay) && jan1WeekDay > 4
+         && jan1WeekDay != 7 ) {
+        yearNumber = d.year() - 1;
+        if ( jan1WeekDay == 6
+         || (jan1WeekDay == 7 && QDate::leapYear(yearNumber) ) ) {
+        weekNumber = 53;
+        }else
+        weekNumber = 52;
+    } else
+        yearNumber = d.year();
+    if ( yearNumber == d.year() ) {
+        int totalDays = 365;
+        if ( QDate::leapYear( yearNumber ) )
+        totalDays++;
+        if ( ((totalDays - d.dayOfYear()) < (4 - dayOfWeek % 7)) ) {
+        yearNumber++;
+        weekNumber = 1;
+        }
+    }
+    if ( yearNumber == d.year() ) {
+        int j = d.dayOfYear() + (7 - dayOfWeek % 7) + ( jan1WeekDay - 1 );
+        weekNumber = j / 7;
+        if ( jan1WeekDay > 4 ) {
+        weekNumber--;
+        }
+    }
     }