Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#3118 closed defect (fixed)

imdb.pl patch to more consistently scrape movie runtime

Reported by: rsalern5@… Owned by: Anduin Withers
Priority: minor Milestone: unknown
Component: mythvideo Version: 0.20
Severity: medium Keywords:
Cc: Ticket locked: no

Description

If the first attempt at determining movie runtime fails, make a second attempt to grab the runtime.

This works when the imdb site displays runtime like this:

Runtime: Argentina:99 min / Norway:55 min / USA:98 min

Prior versions will result in a runtime of zero in this instance. The attached patch looks for "USA:" and attempts to use the subsequent number to determine runtime. "USA:" was used since it was most consistently present among lists of multiple countries.

Attachments (1)

imdb.pl.runtime-fix.patch​ (121 bytes ) - added by rsalern5@… 19 years ago.

Download all attachments as: .zip

Change History (6)

by rsalern5@…, 19 years ago

Attachment: imdb.pl.runtime-fix.patch​ added

comment:1 by Anduin Withers, 19 years ago

Component: mythtv → mythvideo
Owner: changed from Isaac Richards to Anduin Withers

comment:2 by rsalern5@…, 19 years ago

Here are two example movie titles:

"Why We Fight", imdb no.: 0436971, ​http://www.imdb.com/title/tt0436971/

"Little Man", imdb no.: 0430304, ​http://www.imdb.com/title/tt0430304/

comment:3 by Anduin Withers, 19 years ago

Resolution: → fixed
Status: new → closed

(In [12856]) Closes #3118

Extract USA runtime if more than one is provided.

Thanks to rsalern5 for the patch.

comment:4 by Anduin Withers, 19 years ago

(In [12880]) References #3118 Closes #3111

Apply imdb.com format changes, plus better runtime grabbing.

comment:5 by Anduin Withers, 19 years ago

(In [13194]) References #3118 #3285

Merge in changes from [13192] and [12856]

Thanks to Jose M for [13192] and rsalern5 for [12856]

  • Fix format changes for directory and writer
  • Try to get run length better
Note: See TracTickets for help on using tickets.