damienhaynes / moving-pictures

Moving Pictures is a movies plug-in for the MediaPortal media center application. The goal of the plug-in is to create a very focused and refined experience that requires minimal user interaction. The plug-in emphasizes usability and ease of use in managing a movie collection consisting of ripped DVDs, and movies reencoded in common video formats supported by MediaPortal.
12 stars 6 forks source link

kinopoisk grabber doesn't import movie description #1058

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.Import a movie using kinopoisk grabber.
2.See it's summary
3.There is no summary...

What is the expected output? What do you see instead?
Expected output is full movie data import, including summary.

What version of Moving Pictures are you using? 1.1.3
What version of MediaPortal? 1.1.5
What Skin? SMP

Please attach a screenshot and logs if possible. When submitting log files
please submit DEBUG logs. See here for details:
http://tinyurl.com/aj65su

Original issue reported on code.google.com by Journ...@gmail.com on 8 Oct 2011 at 2:38

Attachments:

GoogleCodeExporter commented 9 years ago
I managed to get summary working, modifying regex.
But i still cant get actors. I hate regex, to be honest.

Original comment by Journ...@gmail.com on 9 Oct 2011 at 6:29

Attachments:

GoogleCodeExporter commented 9 years ago
Here is a patch to 1.4.2. It fixes all known parsing problems - summary, score 
and actor list.

Original comment by leonid.k...@gmail.com on 6 Nov 2011 at 1:21

Attachments:

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Awesome, thanks. :) Will get in the next release.

Original comment by conrad.john on 6 Nov 2011 at 4:51

GoogleCodeExporter commented 9 years ago
Updated version. Fix score parsing.

Original comment by leonid.k...@gmail.com on 12 Nov 2011 at 12:04

Attachments:

GoogleCodeExporter commented 9 years ago
Issue 1052 has been merged into this issue.

Original comment by conrad.john on 30 Dec 2011 at 5:20

GoogleCodeExporter commented 9 years ago
Thanks guys, got this included in r1339.

Original comment by conrad.john on 30 Dec 2011 at 8:13