Fanart Handler Plugin v4.0.X.000 (4 Viewers)

jameson_uk

Retired Team Member
  • Premium Supporter
  • January 27, 2005
    7,258
    2,528
    Birmingham
    Home Country
    United Kingdom United Kingdom
    I have uploaded a version of MusicInfoHandler that allows you to whatever folders you want now.

    If you are looking at this can I suggest you take a look at the matching / naming criteria as I remember this being an issue. IIRC FH would not match bands like "+44" and would display would do a like rather than = so on the A-Z page it would match the U2 image for the letter U and the M83 image for M. Also IIRC it would do something like match the image for Gun for Guns 'N' Roses. I made the naming artist_{x} which for some reason I thought worked better.
     

    ajs

    Development Group
  • Team MediaPortal
  • February 29, 2008
    16,039
    11,112
    Kyiv
    Home Country
    Ukraine Ukraine
    IIRC FH would not match bands like "+44" and would display would do a like rather than = so on the A-Z page it would match the U2 image for the letter U and the M83 image for M. Also IIRC it would do something like match the image for Gun for Guns 'N' Roses.
    Is it true? Or am I already corrected?
     

    HomeY

    Test Group
  • Team MediaPortal
  • February 23, 2008
    6,475
    4,645
    49
    ::1
    Home Country
    Netherlands Netherlands
    @ajs seems resume from standby is creating multiple instances of FH to run again?
    Code:
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler is starting. 
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler is starting. 
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler is starting. 
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler version is 4.0.0.151 
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler version is 4.0.0.151 
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler version is 4.0.0.151 
    03-Jan-2015 11:31:20  Info [               Utils]: Fanart Handler folder initialize starting. 
    03-Jan-2015 11:31:20  Info [               Utils]: Fanart Handler folder initialize starting.
    Wasn't this fixed in a previous version? ;)
     

    HomeY

    Test Group
  • Team MediaPortal
  • February 23, 2008
    6,475
    4,645
    49
    ::1
    Home Country
    Netherlands Netherlands
    FanartHandler.cs (line 1024)
    C:
    internal void Start()
    seems to do a lot of stuff. Maybe it's not supposed to do all that, except when the plugin starts?

    My loglines show:
    Code:
    03-Jan-2015 11:30:27  Info [       FanartHandler]: Fanart Handler: is suspended/hibernated.
    03-Jan-2015 11:30:27  Info [       FanartHandler]: Fanart Handler: is suspended/hibernated.
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler: is resuming from standby/hibernate.
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler: is resuming from standby/hibernate.
    And then it goes from 2 lines to 3:
    Code:
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler is starting.
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler is starting.
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler is starting.
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler version is 4.0.0.151
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler version is 4.0.0.151
    03-Jan-2015 11:31:20  Info [       FanartHandler]: Fanart Handler version is 4.0.0.151

    It seems OnSystemPowerModeChanged() is triggering start() on resume.
    C:
        private void OnSystemPowerModeChanged(object sender, PowerModeChangedEventArgs e)
        {
          try
          {
            if (e.Mode == PowerModes.Resume)
            {
              logger.Info("Fanart Handler: is resuming from standby/hibernate.");
              StopTasks(false);
              Start();
            }
    Stop() seems to try to release some of those resources that Start() created, but maybe there are some references left and it fails to release the instances?
     

    ajs

    Development Group
  • Team MediaPortal
  • February 29, 2008
    16,039
    11,112
    Kyiv
    Home Country
    Ukraine Ukraine
    seems to do a lot of stuff. Maybe it's not supposed to do all that, except when the plugin starts?
    In fact, everything is worse, if the MP in the configurator, run in and out of their configurator FH, it will also multiply. He can not behave normally close and destroy. I think from there and feet grow.
     

    Users who are viewing this thread

    Top Bottom