| |||||||
| 0.2.3.0 final and SVN Builds Post bugs you have found in 0.2.3.0 final or any SVN-Snapshot here. |
![]() |
| | Thread Tools | Display Modes |
| | #1 (permalink) |
| Portal Member Join Date: Nov 2005 Location: Rijswijk Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts
Country: | 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!
__________________ Veni, Vidi, Visa I came, I saw, I bought Last edited by BoelShit; 2008-02-15 at 16:07. Reason: Update! |
| | |
| | #2 (permalink) |
| Super Moderator Join Date: Dec 2006 Location: OFS Age: 37
Posts: 901
Thanks: 30
Thanked 41 Times in 38 Posts
Country: | 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 |
| | |
| | #3 (permalink) |
| Portal Member Join Date: Nov 2005 Location: Rijswijk Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts
Country: | 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 |
| | |
| | #4 (permalink) |
| Portal Member Join Date: Aug 2006 Location: Alingsås Age: 34
Posts: 625
Thanks: 192
Thanked 34 Times in 31 Posts
Country: | 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. |
| | |
| | #5 (permalink) |
| Portal Member Join Date: Nov 2005 Location: Rijswijk Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts
Country: | 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 |
| | |
| | #6 (permalink) |
| Portal Member Join Date: Aug 2006 Location: Alingsås Age: 34
Posts: 625
Thanks: 192
Thanked 34 Times in 31 Posts
Country: | 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. |
| | |
| | #7 (permalink) |
| Portal Member Join Date: Nov 2005 Location: Rijswijk Age: 28
Posts: 225
Thanks: 4
Thanked 8 Times in 4 Posts
Country: | 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 |
| | |
| | #9 (permalink) |
| Portal User Join Date: Oct 2007
Posts: 3
Thanks: 0
Thanked 0 Times in 0 Posts
Country: | 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? |
| | |
![]() |
| Bookmarks |
| Tags |
| andgt, creating, folderjpg, imdb, lookup |
| Thread Tools | |
| Display Modes | |
| |
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 |