Bugfix: (bug #0000833) Duplicating an event should also duplicate the category.

1 files changed, 6 insertions, 27 deletions

diff --git a/core/pim/datebook/datebook.cpp b/core/pim/datebook/datebook.cpp
index 3e18374..1bc82a5 100644
--- a/core/pim/datebook/datebook.cpp
+++ b/core/pim/datebook/datebook.cpp
@@ -404,28 +404,17 @@ void DateBook::viewMonth() {
 
 void DateBook::duplicateEvent( const Event &e )
 {
         qWarning("Hmmm...");
     // Alot of code duplication, as this is almost like editEvent();
         if (syncing) {
-        QMessageBox::warning( this, tr("Calendar"),
-                              tr( "Can not edit data, currently syncing") );
+                QMessageBox::warning( this, tr("Calendar"), tr( "Can not edit data, currently syncing") );
         return;
     }
         
-        Event dupevent;
-        dupevent.setStart(e.start());
-        dupevent.setEnd(e.end());
-        dupevent.setDescription(e.description());
-        dupevent.setLocation(e.location());
-        //      dupevent.setCategory(e.category());// how is this done??
-        dupevent.setNotes(e.notes());
-        dupevent.setAllDay(e.isAllDay());
-        dupevent.setTimeZone(e.timeZone());
-        if(e.hasAlarm()) dupevent.setAlarm(e.alarmDelay(),e.alarmSound());
-        if(e.hasRepeat()) dupevent.setRepeat(e.repeatPattern());
+                Event dupevent(e);// Make a duplicate.
         
         // workaround added for text input.
     QDialog editDlg( this, 0, TRUE );
     DateEntry *entry;
     editDlg.setCaption( tr("Duplicate Event") );
     QVBoxLayout *vb = new QVBoxLayout( &editDlg );
@@ -440,33 +429,27 @@ void DateBook::duplicateEvent( const Event &e )
 
 #if defined(Q_WS_QWS) || defined(_WS_QWS_)
     editDlg.showMaximized();
 #endif
     while (editDlg.exec() ) {
         Event newEv = entry->event();
-        if(newEv.description().isEmpty() && newEv.notes().isEmpty() )
-            break;
-        newEv.setUid(e.uid()); // FIXME: Hack not to clear uid
         QString error = checkEvent(newEv);
         if (!error.isNull()) {
-            if (QMessageBox::warning(this, "error box",
-                                     error, "Fix it", "Continue",
-                                     0, 0, 1) == 0)
+                        if (QMessageBox::warning(this, "error box", error, "Fix it", "Continue", 0, 0, 1) == 0)
                 continue;
         }
         db->addEvent(newEv);
         emit newEvent();
         break;
     }
 }
 
 void DateBook::editEvent( const Event &e )
 {
     if (syncing) {
-        QMessageBox::warning( this, tr("Calendar"),
-                              tr( "Can not edit data, currently syncing") );
+                QMessageBox::warning( this, tr("Calendar"), tr( "Can not edit data, currently syncing") );
         return;
     }
 
     // workaround added for text input.
     QDialog editDlg( this, 0, TRUE );
     DateEntry *entry;
@@ -488,28 +471,24 @@ void DateBook::editEvent( const Event &e )
         Event newEv = entry->event();
         if(newEv.description().isEmpty() && newEv.notes().isEmpty() )
             break;
         newEv.setUid(e.uid()); // FIXME: Hack not to clear uid
         QString error = checkEvent(newEv);
         if (!error.isNull()) {
-            if (QMessageBox::warning(this, "error box",
-                                     error, "Fix it", "Continue",
-                                     0, 0, 1) == 0)
-                continue;
+                        if (QMessageBox::warning(this, "error box", error, "Fix it", "Continue", 0, 0, 1) == 0) continue;
         }
         db->editEvent(e, newEv);
         emit newEvent();
         break;
     }
 }
 
 void DateBook::removeEvent( const Event &e )
 {
     if (syncing) {
-        QMessageBox::warning( this, tr("Calendar"),
-                              tr( "Can not edit data, currently syncing") );
+                QMessageBox::warning( this, tr("Calendar"), tr( "Can not edit data, currently syncing") );
         return;
     }
 
     QString strName = e.description();
 
     if ( !QPEMessageBox::confirmDelete( this, tr( "Calendar" ),strName ) )