[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [kdepim/akregator_port] akregator2/src: Fix handling of "disable fetch" and "no custom interval"
From:       Frank Osterfeld <osterfeld () kde ! org>
Date:       2013-01-24 20:19:38
Message-ID: 20130124201938.DD360A6091 () git ! kde ! org
[Download RAW message or body]

Git commit 4bc1434daf459cfceef85efce67fa11388130583 by Frank Osterfeld.
Committed on 05/01/2013 at 13:55.
Pushed by osterfeld into branch 'akregator_port'.

Fix handling of "disable fetch" and "no custom interval"

M  +2    -2    akregator2/src/editfeedcommand.cpp

http://commits.kde.org/kdepim/4bc1434daf459cfceef85efce67fa11388130583

diff --git a/akregator2/src/editfeedcommand.cpp b/akregator2/src/editfeedcommand.cpp
index 7917d98..a419435 100644
--- a/akregator2/src/editfeedcommand.cpp
+++ b/akregator2/src/editfeedcommand.cpp
@@ -80,7 +80,7 @@ public:
             QPointer<FeedPropertiesDialog> dlg( new FeedPropertiesDialog( q->parentWidget() ) );
             dlg->setFeedTitle( fc.title() );
             dlg->setUrl( fc.xmlUrl() );
-            dlg->setCustomFetchInterval( fc.fetchInterval() > 0 ); //PENDING(frank) correct?
+            dlg->setCustomFetchInterval( fc.fetchInterval() >= 0 );
             dlg->setFetchInterval( fc.fetchInterval() );
 
             if ( dlg->exec() != QDialog::Accepted ) {
@@ -91,7 +91,7 @@ public:
             fc.setName( dlg->feedTitle() + KRandom::randomString( 8 ) );
             fc.setTitle( dlg->feedTitle() );
             fc.setXmlUrl( dlg->url() );
-            fc.setFetchInterval( dlg->hasCustomFetchInterval() ? dlg->fetchInterval() : 0 );
+            fc.setFetchInterval( dlg->hasCustomFetchInterval() ? dlg->fetchInterval() : -1 );
             delete dlg;
             CollectionModifyJob* job = new CollectionModifyJob( fc, session );
             connect( job, SIGNAL(finished(KJob*)), q, SLOT(collectionModified(KJob*)) );
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic