MediaPortal Forums HTPC/MediaCenter

Go Back   MediaPortal Forum » MediaPortal 1 » Help on Development » Quality Assurance (Bugreports) » Archive » Bug Reports » 0.2.3.0 final and SVN Builds


0.2.3.0 final and SVN Builds Post bugs you have found in 0.2.3.0 final or any SVN-Snapshot here.

Reply
 
Thread Tools Display Modes
Old 2007-12-15, 01:46   #1 (permalink)
Portal Member
 
Join Date: Nov 2005
Location: Rijswijk
Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts

Country:


Default Creating folder.jpg within MP -> Movies -> IMDB lookup

When looking up movie information in MP (IMDB option), MP automatically creates a folder.jpg from the downloaded coverart. This does not occur when looking up movie information through the MP configuration btw.

The created folder.jpg is low on quality where as it used to be high in quality. I don't know from which SVN it doesnt work anymore.

bug description:

1. start mediaportal
2. go to MyMovies
3. set movie view to share!!!
4. go to a Movie folder without coverart
5. click right mousebutton and choose IMDB from menu
6. let it download movie info and pick a coverart
7. go back with ESCAPE
8. see that the folder coverart is a low quality image where as the IMDB screen showed a high quality image
9. check the movie folder in explorer. The folder.jpg is just a couple KB in size and small in resolution

Fix:

I've dusted off some coding skills (LOL) and downloaded the sourcecode.
The code handling this is located in GUIVideoInfo.cs. The code does a ConvertToLargeCoverArt from the normal coverImageArt which is (I think) not needed to perform as the good quality coverart is also downloaded and saved in the corresponding Thumb folder.

I changed the code so that it can use both normal coverImageArt and largeCoverImageArt where the largeCoverImageArt has priority.
Now it saves a folder.jpg based on the better quality coverart and if not present it uses the lower quality based one.

I only have tested it to see if it creates the high quality coverart which it does. Couldnt check what happens when the largeCoverArt isnt available but it should then work with the normal coverArt. Hoping to see this fixed with my adjustments or in another way (as long as it gets fixed ).

attachments:

I've attached a pre-build WindowsPlugins.dll (for easy testing) and the editted GUIVideoInfo.cs

EDIT/UPDATE! New WindowsPlugin.dll added compiled against SVN Revision:17388 (SVN-Snapshot: 02-14-2008 05-04h - Revision:17388)

Just replace or rename the old WindowsPlugin.dll with the new one in the archive.

GUIVideoInfo.zip contains the source code for the change!
Attached Files
File Type: zip WindowPlugins.zip (578.1 KB, 71 views)
File Type: zip GUIVideoInfo.zip (5.1 KB, 32 views)
__________________
Veni, Vidi, Visa
I came, I saw, I bought

Last edited by BoelShit; 2008-02-15 at 16:07. Reason: Update!
BoelShit is offline   Reply With Quote
Old 2007-12-16, 08:57   #2 (permalink)
Super Moderator
 
rekenaar's Avatar
 
Join Date: Dec 2006
Location: OFS
Age: 37
Posts: 901
Thanks: 30
Thanked 41 Times in 38 Posts

Country:

My System

Default

I will download and test later today. Thanks for the suggestion.

I hope the developers can pick up and implement your suggestion - I have also suggested a codefix a week ago, but it was not yet implemented. I guess (hope) the devs are too busy with infinity
__________________
Ondersteun Afrikaans!
******************

Rekenaar
rekenaar is offline   Reply With Quote
Old 2007-12-19, 10:17   #3 (permalink)
Portal Member
 
Join Date: Nov 2005
Location: Rijswijk
Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts

Country:


Default

I noticed... Hope they will fix this though... saves me the manual copying from Thumbs folder to the movie folder and renaming it. I hate to do that for 150+ movies
__________________
Veni, Vidi, Visa
I came, I saw, I bought
BoelShit is offline   Reply With Quote
Old 2008-01-02, 17:02   #4 (permalink)
Portal Member
 
emphatic's Avatar
 
Join Date: Aug 2006
Location: Alingsås
Age: 34
Posts: 625
Thanks: 192
Thanked 34 Times in 31 Posts

Country:


Default

Has this been added yet to a recent SVN build? And btw, is MediPortal supposed to overwrite the thumb in a folder if there's already a folder.jpg present?

I will also download your fix and test in my setup. Thanks.
Emph
__________________
You're not the content of your wallet.
emphatic is online now   Reply With Quote
Old 2008-01-02, 18:33   #5 (permalink)
Portal Member
 
Join Date: Nov 2005
Location: Rijswijk
Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts

Country:


Default

This hasnt been added to SVN yet. I guess due to MP2 and the lack of testers confirming the "bug". If you need a newer WindowsPlugin.dll compiled to the latest MP source let me know and I'll add it.

The folder.jpg is as I can see is only overwritten when you do an IMDB lookup and change the cover image from the IMDB screen and then close that screen. If you already have a folder.jpg in the movie folder the IMDB screen shows that folder.jpg as cover nr.1 and won't overwrite it.

Hope this fix works for you also so it can be added to the source.
__________________
Veni, Vidi, Visa
I came, I saw, I bought
BoelShit is offline   Reply With Quote
Old 2008-01-02, 23:10   #6 (permalink)
Portal Member
 
emphatic's Avatar
 
Join Date: Aug 2006
Location: Alingsås
Age: 34
Posts: 625
Thanks: 192
Thanked 34 Times in 31 Posts

Country:


Default

I've downloaded your .zip but I don't really understand where to put the GUIVideoInfo.cs? I've searched my MediaPortal directory, but couldn't find it.

Emph
__________________
You're not the content of your wallet.
emphatic is online now   Reply With Quote
Old 2008-01-02, 23:13   #7 (permalink)
Portal Member
 
Join Date: Nov 2005
Location: Rijswijk
Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts

Country:


Default

The GUIVideoInfo.cs is the source file for the devs which I have edited. You dont need to put it anywhere. The WindowsPlugin.dll has to be put in MPinstalldir\Plugins\Windows.
__________________
Veni, Vidi, Visa
I came, I saw, I bought
BoelShit is offline   Reply With Quote
Old 2008-01-03, 00:18   #8 (permalink)
Portal Member
 
emphatic's Avatar
 
Join Date: Aug 2006
Location: Alingsås
Age: 34
Posts: 625
Thanks: 192
Thanked 34 Times in 31 Posts

Country:


Default

Okay. Tested and working! Great job!
Emph
__________________
You're not the content of your wallet.
emphatic is online now   Reply With Quote
Old 2008-01-07, 20:34   #9 (permalink)
Portal User
 
Join Date: Oct 2007
Posts: 3
Thanks: 0
Thanked 0 Times in 0 Posts

Country:


Default

I'm experiencing the same problem on my MP machine.
Very low quality IMDB images, but the problem seems to be solved after implementing Boelshits fix.

Perhaps if more people notice the bug it could get included into the next SVN?
snoekieboe is offline   Reply With Quote
Old 2008-01-09, 23:52   #10 (permalink)
Portal User
 
Join Date: Dec 2007
Posts: 1
Thanks: 0
Thanked 0 Times in 0 Posts

Country:


Default

Bug confirmed by me too...
__________________
kind regards,

Dennis
DennisNL is offline   Reply With Quote
Reply

Bookmarks

Tags
andgt, creating, folderjpg, imdb, lookup

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are Off

Similar Threads
Thread Thread Starter Forum Replies Last Post
My Movies: imdb lookup causes flickering McSeven 0.2.2.0 Final and SVN Builds 4 2007-04-14 21:09
Black screen after on/off on tv MP 0.2.2.0 (Mac mini -> DVI -> HDMI -> Samsung LCD) FlexyZ MediaPortal 1 Talk 1 2007-01-03 00:00
Right click AVI and press 3 for IMDB lookup => MP crashes druden fixed 0.2.0.0 bugs 3 2006-12-05 19:31
File>change device and configuration>general>allway Lolita The old Bugreport Forum 3 2006-07-10 21:39
[DVB] MP Standby -> Resume -> No encrypted channels? ziphnor General Support 0 2006-06-14 21:18


All times are GMT +1. The time now is 13:00.


Powered by vBulletin® Version 3.7.3
Copyright ©2000 - 2009, Jelsoft Enterprises Ltd.
Search Engine Optimization by vBSEO 3.2.0 Protected by Akismet Blog with WordPress
Advertisement System V2.6 By   Branden