davidedmundson added inline comments. INLINE COMMENTS
> comicupdater.cpp:56 > + } else { > + mUpdateIntervall = 3; > } wouldn't you still need to call checkForUpdate() if you went into this path? Especially as checkForUpdate is the one that sets up the timer to call itself again > comicupdater.cpp:86 > if ( !mLastUpdate.isValid() || ( mLastUpdate.addDays( mUpdateIntervall ) > < QDateTime::currentDateTime() ) ) { > - mGroup.writeEntry( "lastUpdate", QDateTime::currentDateTime() ); > + mLastUpdate = QDateTime::currentDateTime(); > + mGroup.writeEntry( "lastUpdate", mLastUpdate ); I assume this line is the important change? +1 to this bit. REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D9338 To: anthonyfieroni, davidedmundson, broulik, #plasma, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart