[Kde-pim] Review Request: Akregator: don't fetch some feeds times and times again (if it cannot access them).

[Torgny Nyblom]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5384/#review7691
-----------------------------------------------------------



/branches/KDE/4.5/kdepim/akregator/src/feed.h
<http://svn.reviewboard.kde.org/r/5384/#comment7805>

    Don't really like the function name as "set" is more or less the standard prefix.
    Perhaps something along the lines of setFetchedTime().
    
    Other then that I say "ship it", but I'm not the maintainer :)


- Torgny


On 2010-09-20 17:24:44, Shlomi Fish wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5384/
> -----------------------------------------------------------
> 
> (Updated 2010-09-20 17:24:44)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> This resolves https://bugs.kde.org/show_bug.cgi?id=225851 :
> 
> Akregator fetches some feeds times and times again (possibly ones it cannot
> access). Despite the fact that I specifically told it to cancel the feed
> fetches, it still goes to the Internet and tries to fetch the following feeds:
> 
> http://www.sparkthis.com/atom.xml
> 
> I'm on Mandriva Linux Cooker, but it's been an ongoing problem for a long time
> and is very annoying. Please fix it.
> 
> Regards,
> 
> -- Shlomi Fish
> 
> 
> Diffs
> -----
> 
>   /branches/KDE/4.5/kdepim/akregator/src/feed.h 1160922 
>   /branches/KDE/4.5/kdepim/akregator/src/feed.cpp 1160922 
> 
> Diff: http://svn.reviewboard.kde.org/r/5384/diff
> 
> 
> Testing
> -------
> 
> I've been using akregator with this patch applied for many months now.
> 
> 
> Thanks,
> 
> Shlomi
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/