I fixed the Friday Times recipe

[ireadtheinternet]

I had been using the Friday Times recipe as a template, because it was about the simplest parse_index recipe (that is, a recipe not based on a RSS feed) I could find. However I eventually noticed the recipe itself was broken, and I had to take a break from the other recipe I am working on. So I fixed the Friday Times recipe. Let me know any criticisms or suggestions for the fix.

from calibre.web.feeds.news import BasicNewsRecipe

class TheFridayTimes(BasicNewsRecipe):
    language       = 'en_PK'
    encoding       = 'utf8'
    version        = 1.1
    
    title          = u'The Friday Times'
    category       = u'news, Pakistan'
    description    = u"Pakistan's First Independent Weekly Paper"

    no_stylesheets            = True
    no_javascript             = True
    ignore_duplicate_articles = {'url'}

    keep_only_tags = [
        dict(name='div', attrs={'class':'sidebar_content'}),
        dict(name='div', attrs={'class':'comment_inner'})  
    ]

    remove_tags = [
        dict(name='p', attrs={'class':'no-break'}),
        dict(name='div', attrs={'class':'related_posts'}),
        dict(name='div', attrs={'id':'respond'})
    ]

    def parse_index(self):      
        toc_page = self.index_to_soup('http://www.thefridaytimes.com/tft/') 
        toc = toc_page.find('div', attrs={'class':'sidebar_left_home_wrapper'})

        articles = []
        for story in toc.findAll('a'):
            # skip the links with an image, they are repeated further down
            if story.find('img') is not None:
                continue
            url = story['href']
            # If no title, use url as title
            title = story.get('title', url)
            self.log('Found article:', story)
            self.log('\t', url)
            articles.append({'title':title, 'url':url, 'date':'','description':''})

        return [('Current Issue', articles)]

[ireadtheinternet]

If the [s]articles[/s] comments are not wanted, you can comment out the line

dict(name='div', attrs={'class':'comment_inner'})

The builtin currently has an issue where the comments were displaying but they were in a giant font, so I wasn't sure if the original intent had been to include or exclude the comments.

EDIT: Noticed this is an official edit now, thanks.