Fanart Handler Plugin v2.2.3.197 (and music fanart scraper) (4 Viewers)

Status
Not open for further replies.

emphatic

Design Group
  • Team MediaPortal
  • August 25, 2006
    3,736
    1,196
    Alingsås
    Home Country
    Sweden Sweden
    Would it be possible to add fanart manually (for Various Artists)?

    Cheers
    Emph
     

    cul8er

    Retired Team Member
  • Premium Supporter
  • August 31, 2006
    2,159
    2,679
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #62
    Hi,

    Please explain what you mean by this and I will see what I can do.

    best regards
    cul8er
     

    emphatic

    Design Group
  • Team MediaPortal
  • August 25, 2006
    3,736
    1,196
    Alingsås
    Home Country
    Sweden Sweden
    Hi,

    Please explain what you mean by this and I will see what I can do.

    best regards
    cul8er

    Hi cul8er,

    What I mean is that I would like to be able to add fanart manually from a local source. So instead of the skin's default fanart being shown for a track from a compilation by the Album Artist "Various Artists" or for a film soundtrack where it would look great to have a fanart from the movie. Having the plugin do this automatically wouldn't be possible, but selecting this yourself would be pretty quick from the FanArt Handler's config app. The automatic functionality of the plugin is mindblowing for single artist/band art, but for Various Artists it is somewhat lacking.

    Automatic functionality could be implemented if a checkbox can be added to the FanArt database - "Compilation". Then a compilation tab with these options:

    First rule:
    [ ] Select FanArt by Genre
    [ ] Select FanArt by Track Artist (the ID3 Tag for Artist, not Album Artist)
    [x] Use skin's "music" backdrop (default)

    Second rule:
    [ ] User defined (when the user has edited an album like say, the soundtrack to Casino Royale, this will be used before any of the rules above is used)
    [x] Don't use custom selections (first rule in use only - default)

    I hope this makes sense and is something you will consider. :D

    Emph
     

    cul8er

    Retired Team Member
  • Premium Supporter
  • August 31, 2006
    2,159
    2,679
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #64
    Hi emphatic,

    I have added a new button to the"Manage Fanart" tab in FanartHandler configuration. The new button is called "Edit Image Path" (see attached screenshot). This button enables you to edit an automatically matched fanart image or add a manual image to an artist that has no fanart. Hope this is somewhat what you asked for;)

    For example;
    1. The artist Alan Parsons in the screenshot has no fanart today. Press the Edit Image Path button and manually select an image. Can be an actual image of the artist or a general image reflecting the genre of the artist or similar.
    2. The artist Anastacia already has several fanart images but the button allows you to alter the image to another one.

    I will also add a button so that you can add more images to the selected artist.

    cheers
    cul8er
     

    Attachments

    • Bild1.jpg
      Bild1.jpg
      476.9 KB

    seco

    Retired Team Member
  • Premium Supporter
  • August 7, 2007
    1,575
    1,239
    Home Country
    Finland Finland
    Please fix the logging configuration of this plugin.

    Following line which is used two times by the Fanart Handler breaks every other plugin logging configuration:

    LoggingConfiguration config = new LoggingConfiguration();

    Instead, this should be

    LoggingConfiguration config = LogManager.Configuration ?? new LoggingConfiguration();

    Hi cul8er,

    I hope you can fix this before releasing new version for MP 1.2.0 Beta since this affects all plugins using NLog LogManager.

    This issue was fixed a long time ago in Moving Pictures which is the source of the code.
     

    cul8er

    Retired Team Member
  • Premium Supporter
  • August 31, 2006
    2,159
    2,679
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #66
    seco,

    This was fixed the day after your original post. Thanks for making me aware of the problem. I had not noticed as that code was from Moving Pictures as you correctly say. So I have not paid any attention to it and had missed that it was wrong. Will be part of the release as soon as MP 1.2 Beta is out (will create versions for both Beta and none Beta users).

    cheers
    cul8er
     

    emphatic

    Design Group
  • Team MediaPortal
  • August 25, 2006
    3,736
    1,196
    Alingsås
    Home Country
    Sweden Sweden
    That is a very nice implementation and will make my dreams come true!

    Thanks a lot for your great and awesome work.
    Emph
     

    cul8er

    Retired Team Member
  • Premium Supporter
  • August 31, 2006
    2,159
    2,679
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #68
    Hi emphatic,

    Thank you for a great suggestion.

    best regards
    cul8er
     
    Status
    Not open for further replies.

    Users who are viewing this thread

    Top Bottom