Opened 11 years ago

Closed 11 years ago

#7384 closed patch (fixed)

allocine.pl (french imdb) update (allocine.com have change their templates)

Reported by: anonymous Owned by: robertm
Priority: trivial Milestone: unknown
Component: Plugin - MythVideo Version: head
Severity: low Keywords: allocine imdb
Cc: Ticket locked: no

Description

Hi,

I've made some changes on the allocine.pl file (this is the french imdb). allocine.com have change their website and nothing worked anymore.

So i've change many parts of the script so now this new version seems to work. (i test it on my own french movies).

really sorry for my "english"

here is the new version of the script

http://www.soslinux.net/down/allocine.pl

Regards,

Geoffroy Geerseau jamdess@…

Attachments (6)

allocine.pl.diff.gz (3.3 KB) - added by jamdess@… 11 years ago.
allocine diff unified
allocine.pl.gz (3.4 KB) - added by jamdess@… 11 years ago.
last version with the correction for the title and the posters
allocine.pl.diff.between.now.and.file.changes.20090618.gz (3.5 KB) - added by jamdess@… 11 years ago.
diff between the last version (with title & poster debug) and the version of june 18, 2009
allocine.pl.diff.between.now.and.file.changes.20091020.gz (1.9 KB) - added by jamdess@… 11 years ago.
diff between the last version (with title & poster debug) and the version of october 20, 2009
allocine.pl.2.gz (3.5 KB) - added by jamdess@… 11 years ago.
last allocine.pl script (poster correction && user rating correction)
allocine.diff.gz (1.1 KB) - added by jamdess@… 11 years ago.
diff unified

Download all attachments as: .zip

Change History (14)

comment:1 Changed 11 years ago by robertm

Owner: changed from Anduin Withers to robertm
Priority: majortrivial
Severity: mediumlow
Status: newassigned
Type: defectpatch
Version: unknownhead

Geoffroy,

Thank you very much for the script-- can you please provide it as a unified diff against the existing copy?

comment:2 Changed 11 years ago by jamdess@…

oups i'm really really sorry for the duplicates posts. (i've some troubles with my webbrowser)

you can find the diff file at http://www.soslinux.net/down/allocine.pl.diff

regards

comment:3 Changed 11 years ago by robertm

Can you please attach it here so that it doesn't appear from some third party web server before I can look at it?

comment:4 Changed 11 years ago by jamdess@…

i'd like to but when I try to attach a file to a ticket, I got this error : "Submission rejected as potential spam (Maximum number of external links per post exceeded)" groumph.

comment:5 Changed 11 years ago by robertm

gzip or tar it.

Changed 11 years ago by jamdess@…

Attachment: allocine.pl.diff.gz added

allocine diff unified

comment:6 Changed 11 years ago by jamdess@…

Hi

I've found 2 "bugs". The first one was a charset problem and the second was a problem when no "big" poster was founded.

So I'll attach the last version of the script "allocine.pl", and 2 diff files.

Regards,

Geoffroy Geerseau

Changed 11 years ago by jamdess@…

Attachment: allocine.pl.gz added

last version with the correction for the title and the posters

Changed 11 years ago by jamdess@…

diff between the last version (with title & poster debug) and the version of june 18, 2009

Changed 11 years ago by jamdess@…

diff between the last version (with title & poster debug) and the version of october 20, 2009

comment:7 Changed 11 years ago by anonymous

Last correction

Posters download user rating

Changed 11 years ago by jamdess@…

Attachment: allocine.pl.2.gz added

last allocine.pl script (poster correction && user rating correction)

Changed 11 years ago by jamdess@…

Attachment: allocine.diff.gz added

diff unified

comment:8 Changed 11 years ago by robertm

Resolution: fixed
Status: assignedclosed

(In [22620]) Closes #7384. Update allocine.pl grabber for recent site changes. Patch is slightly modified from that submitted-- we should not be doing downloads in the grabber, the url should always be presented in the results. Also changed the output format to return the cover in the -D query as is required for .23/trunk. Left -P query for now for backward compatibility. Thank you to Geoffroy Geerseau for the patch.

Note: See TracTickets for help on using tickets.