[booblyrics] cleaning

This commit is contained in:
Julien Veyssier 2013-03-07 20:06:10 +01:00
commit 595e33b1fa
5 changed files with 6 additions and 19 deletions

View file

@ -23,8 +23,6 @@ from weboob.tools.backend import BaseBackend
from .browser import SeeklyricsBrowser from .browser import SeeklyricsBrowser
from urllib import quote_plus from urllib import quote_plus
from contextlib import closing
from gzip import GzipFile
__all__ = ['SeeklyricsBackend'] __all__ = ['SeeklyricsBackend']

View file

@ -18,13 +18,6 @@
# along with weboob. If not, see <http://www.gnu.org/licenses/>. # along with weboob. If not, see <http://www.gnu.org/licenses/>.
try:
from urlparse import parse_qs
except ImportError:
from cgi import parse_qs # NOQA
from urlparse import urlsplit
from weboob.capabilities.lyrics import SongLyrics from weboob.capabilities.lyrics import SongLyrics
from weboob.capabilities.base import NotAvailable, NotLoaded from weboob.capabilities.base import NotAvailable, NotLoaded
from weboob.tools.browser import BasePage from weboob.tools.browser import BasePage
@ -42,10 +35,10 @@ class ResultsPage(BasePage):
continue continue
artist = NotAvailable artist = NotAvailable
ftitle = self.parser.select(tr,'a > font > font',1) ftitle = self.parser.select(tr,'a > font > font',1)
title = ftitle.getparent().getparent().text_content() title = unicode(ftitle.getparent().getparent().text_content())
id = ftitle.getparent().getparent().attrib.get('href','').replace('/lyrics/','').replace('.html','') id = ftitle.getparent().getparent().attrib.get('href','').replace('/lyrics/','').replace('.html','')
aartist = self.parser.select(tr,'a')[-1] aartist = self.parser.select(tr,'a')[-1]
artist = aartist.text artist = unicode(aartist.text)
songlyrics = SongLyrics(id, title) songlyrics = SongLyrics(id, title)
songlyrics.artist = artist songlyrics.artist = artist
songlyrics.content = NotLoaded songlyrics.content = NotLoaded
@ -59,9 +52,9 @@ class SonglyricsPage(BasePage):
l_artitle = self.parser.select(self.document.getroot(),'table.text td > b > h2') l_artitle = self.parser.select(self.document.getroot(),'table.text td > b > h2')
if len(l_artitle) > 0: if len(l_artitle) > 0:
artitle = l_artitle[0].text.split(' Lyrics by ') artitle = l_artitle[0].text.split(' Lyrics by ')
artist = artitle[1] artist = unicode(artitle[1])
title = artitle[0] title = unicode(artitle[0])
content = self.parser.select(self.document.getroot(),'div#songlyrics',1).text_content().strip() content = unicode(self.parser.select(self.document.getroot(),'div#songlyrics',1).text_content().strip())
songlyrics = SongLyrics(id, title) songlyrics = SongLyrics(id, title)
songlyrics.artist = artist songlyrics.artist = artist
songlyrics.content = content songlyrics.content = content

View file

@ -20,8 +20,6 @@
from weboob.tools.test import BackendTest from weboob.tools.test import BackendTest
from weboob.capabilities.base import NotLoaded from weboob.capabilities.base import NotLoaded
from random import choice
class SeeklyricsTest(BackendTest): class SeeklyricsTest(BackendTest):
BACKEND = 'seeklyrics' BACKEND = 'seeklyrics'

View file

@ -25,7 +25,6 @@ from weboob.capabilities.lyrics import ICapLyrics
from weboob.tools.application.repl import ReplApplication from weboob.tools.application.repl import ReplApplication
from weboob.capabilities.base import NotAvailable,NotLoaded from weboob.capabilities.base import NotAvailable,NotLoaded
from weboob.tools.application.formatters.iformatter import IFormatter, PrettyFormatter from weboob.tools.application.formatters.iformatter import IFormatter, PrettyFormatter
from weboob.core import CallErrors
__all__ = ['Booblyrics'] __all__ = ['Booblyrics']

View file

@ -18,8 +18,7 @@
# along with weboob. If not, see <http://www.gnu.org/licenses/>. # along with weboob. If not, see <http://www.gnu.org/licenses/>.
from .base import IBaseCap, CapBaseObject, Field, StringField, FloatField, \ from .base import IBaseCap, CapBaseObject, StringField
DateField, IntField, UserError
__all__ = ['SongLyrics', 'ICapLyrics'] __all__ = ['SongLyrics', 'ICapLyrics']