iplex - nieprawidlowa ilosc list, dodanie opisu i oceny filmu

[GoogleCodeExporter]

Iplex na swojej stronie podaje nieprawidlowa ilosc filmow w danej kategorii, 
np. Kino Dokumentalne pokazuje 282 filmow, ale tak naprawde jest 8 stron po 20 
filmow na kazdje stronie. Komedia pokazuje 699 filmow ale jest tylko 16 stron z 
20 tytulami.

iplex.py pobiera ta nieprawidlowa wartosc, dzieli przez 20 (ilosc tytulow na 
stornie) i generuje odpowiednia ilosc list, czyli w przypadku Kina 
Dokumentalnego 15 list, a powinno byc tylko 8.

Tu jest nowa fukncja ktora zwraca numer ostatniej strony:
def getLastPageNumber(self, url):
    numLastPage = 0
    openURL = urllib2.urlopen(url)
    readURL = openURL.read()
    openURL.close()
    match = re.compile('page=(.+?)">ostatnia strona</a>').findall(readURL)
    if len(match) > 0:
        numLastPage = match[0]
    return numLastPage

w "listsItemsPage" zamiast:
a = math.ceil(float(self.getSizeAllItems(url)) / 
float(self.getSizeItemsPerPage(url)))

mozemy teraz dac:
a = self.getLastPageNumber(url)

Przy tej okazji dorzucilem opis i ocene do funkcji "add":
liz.setInfo('video', {'Title': title, 'plot': desc, 'rating': rating} )

pozdrawiam

Original issue reported on code.google.com by si...@jatrn.com on 4 Aug 2012 at 3:18

[GoogleCodeExporter]

jesli kogos iteresuja moje zmiany, to zalaczam moj iplex.py, a w nim:
1. poprawiona ilosc list,
2. dodane informacje o filmach (info): plot, rating, votes, year duration.

pozdrawiam

Original comment by si...@jatrn.com on 4 Aug 2012 at 10:40

Attachments:

• iplex.py

[GoogleCodeExporter]

Ciekawe zmiany :). W chwili wolnego czasu sprawdzę i dodam do repozytorium :)

Original comment by ples...@gmail.com on 5 Aug 2012 at 6:39

[GoogleCodeExporter]

w pierwszym iplex.py, zabraklo ELSE w "getLastPageNumber", powinno byc:
    if len(match) > 0:
        numLastPage = match[0]
    else:
        numLastPage = 1

Wstrzymaj sie z dodawaniem czegokolwiek narazie, bo planuje kilka innych zmian. 
Zawsze sie zastanawialem dlaczego w kazdej kategorii sa "Lista1, 2, 3..." i 
dopiero w nich poszczegolne filmy. Pomyslalem ze mozna by ominac "listy" i po 
wejsciu do kazdej kategorii od razu wyszczegolnic tytuly z pierwszej strony i 
rownoczesnie dodac linki do "nastepna" i "poprzednia" strona.

Zrobilem cos takiego wczoraj wieczorem (narazie tylko dla glownych kategorii, 
nie dla serii), wiec rzuc na to okiem i daj znac czy to ma jakis sens. Jesli 
sie podoba to dokoncze w najblizszych dniach.
Planuje rowniez do glownej listy obok "Kategorie" i "Szukaj" dodac "Kolekcje", 
"Kanaly" i "Bajki", zobaczymy co z tego wyjdzie.

Plik ktory zalaczam, to wersja BETA, tylko i wylacznie do testowania.

pozdrawiam i czekam na komentarze.

Original comment by si...@jatrn.com on 6 Aug 2012 at 2:05

Attachments:

• iplex.py

[GoogleCodeExporter]

Też tak chciałem przerobić, ale niestety brak czasu. Może lepiej abym Cię 
dodał do projektu i po skończeniu tego co chcesz zrobiłbyś commita? Tak 
chyba by było lepiej :).
Co Ty na to?

Original comment by ples...@gmail.com on 6 Aug 2012 at 6:37

[GoogleCodeExporter]

brzmi calkiem nie zle :)

Original comment by si...@jatrn.com on 6 Aug 2012 at 10:34

[GoogleCodeExporter]

wysłałem Ci maila z prośbą o googlowego maila abym Cię dodał. dostałeś?

Original comment by ples...@gmail.com on 8 Aug 2012 at 7:42

[GoogleCodeExporter]

dostalem i odpisalem dopiero dzisiaj. 

Original comment by si...@jatrn.com on 9 Aug 2012 at 2:09

[GoogleCodeExporter]

Original comment by jatrn69 on 11 Aug 2012 at 1:35

• Changed state: Done
• Added labels: Type-Enhancement
• Removed labels: Type-Defect