Update allmusic.com scraper (2 Viewers)

styrsven

Portal Pro
January 6, 2013
115
51
63
Home Country
Sweden Sweden
Finally got around to testing this again.
I got errors for "Kjell Alinge & Janne Forsell", see the attached log file.

Another thing I noticed is that groups like "Beatles,The" and "Beach Boys,The" were not found.
 

jameson_uk

Retired Team Member
  • Premium Supporter
  • January 27, 2005
    7,258
    2,528
    Birmingham
    Home Country
    United Kingdom United Kingdom
    Updated with v2b.
    This should sort out the artist prefixes (eg=> Beatles, The => The Beatles)
     

    HomeY

    Test Group
  • Team MediaPortal
  • February 23, 2008
    6,475
    4,645
    49
    ::1
    Home Country
    Netherlands Netherlands
    But... i have 1 artist that always seems to cause problems :)
    Frédéric Chopin
    OK this was because he has no discography page. New version in first post which will now check that pages exist before loading them.
    Can you try and break this again please?

    Unable to breakt it at the moment. Working as expected now. (y)
     

    HomeY

    Test Group
  • Team MediaPortal
  • February 23, 2008
    6,475
    4,645
    49
    ::1
    Home Country
    Netherlands Netherlands
    Version 2 removed as I think I have broken it...

    Can confirm. Although the change does fix scraping for this Artist, i'm now seeing some side effects, for example:
    - All long text fields that were scrolling, are now flickering/blinking
     
    Last edited:

    SilentBob

    Portal Pro
    January 1, 2011
    67
    88
    Home Country
    Germany Germany
    When I try to get Info I get NullReference Exceptions in log

    Code:
    [2013-08-18 15:03:27,464] [Log	] [MPMain  ] [ERROR] - Exception: System.NullReferenceException: Object reference not set to an instance of an object.
      at MediaPortal.Music.Database.MusicAlbumInfo.Parse(String strUrl) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\Databases\Music\InternetLookups\MusicAlbumInfo.cs:line 211
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(Int32 parentWindowID, String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1584
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1471
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnInfo(Int32 iItem) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1456
      at MediaPortal.GUI.Music.GUIMusicGenres.OnShowContextMenu() in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 723
      at MediaPortal.GUI.Library.GUIWindow.OnAction(Action action)
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 493
      at MediaPortal.GUI.Music.GUIMusicGenres.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 336
      at MediaPortal.GUI.Library.GUIWindowManager.OnAction(Action action)
      at MediaPortalApp.OnAction(Action action)  Message: Object reference not set to an instance of an object.  Site  : Boolean Parse(System.String)  Source : Databases  Stack Trace:	at MediaPortal.Music.Database.MusicAlbumInfo.Parse(String strUrl) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\Databases\Music\InternetLookups\MusicAlbumInfo.cs:line 211
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(Int32 parentWindowID, String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1584
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1471
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnInfo(Int32 iItem) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1456
      at MediaPortal.GUI.Music.GUIMusicGenres.OnShowContextMenu() in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 723
      at MediaPortal.GUI.Library.GUIWindow.OnAction(Action action)
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 493
      at MediaPortal.GUI.Music.GUIMusicGenres.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 336
      at MediaPortal.GUI.Library.GUIWindowManager.OnAction(Action action)
      at MediaPortalApp.OnAction(Action action)
    [2013-08-18 15:03:27,465] [Error  ] [MPMain  ] [ERROR] - Exception: Object reference not set to an instance of an object. Databases	at MediaPortal.Music.Database.MusicAlbumInfo.Parse(String strUrl) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\Databases\Music\InternetLookups\MusicAlbumInfo.cs:line 211
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(Int32 parentWindowID, String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1584
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1471
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnInfo(Int32 iItem) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1456
      at MediaPortal.GUI.Music.GUIMusicGenres.OnShowContextMenu() in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 723
      at MediaPortal.GUI.Library.GUIWindow.OnAction(Action action)
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 493
      at MediaPortal.GUI.Music.GUIMusicGenres.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 336
      at MediaPortal.GUI.Library.GUIWindowManager.OnAction(Action action)
      at MediaPortalApp.OnAction(Action action)
    [2013-08-18 15:03:27,467] [Log	] [MPMain  ] [ERROR] - Exception: System.Exception: exception occurred ---> System.NullReferenceException: Object reference not set to an instance of an object.
      at MediaPortal.Music.Database.MusicAlbumInfo.Parse(String strUrl) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\Databases\Music\InternetLookups\MusicAlbumInfo.cs:line 211
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(Int32 parentWindowID, String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1584
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.ShowAlbumInfo(String artistName, String albumName, String strPath, MusicTag tag) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1471
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnInfo(Int32 iItem) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 1456
      at MediaPortal.GUI.Music.GUIMusicGenres.OnShowContextMenu() in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 723
      at MediaPortal.GUI.Library.GUIWindow.OnAction(Action action)
      at MediaPortal.GUI.Music.GUIMusicBaseWindow.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicBaseWindow.cs:line 493
      at MediaPortal.GUI.Music.GUIMusicGenres.OnAction(Action action) in C:\Users\james_000\Documents\Code\MediaPortal-1\mediaportal\WindowPlugins\GUIMusic\GUIMusicGenres.cs:line 336
      at MediaPortal.GUI.Library.GUIWindowManager.OnAction(Action action)
      at MediaPortalApp.OnAction(Action action)
      --- End of inner exception stack trace ---
      at MediaPortalApp.OnAction(Action action)
      at MediaPortal.GUI.Library.OnActionHandler.Invoke(Action action)
      at MediaPortal.GUI.Library.GUIWindowManager.DispatchThreadMessages()
      at MediaPortalApp.FrameMove()  Message: exception occurred  Site  : Void OnAction(MediaPortal.GUI.Library.Action)  Source : MediaPortal  Inner Exception(s):  -> exception occurred  -> Object reference not set to an instance of an object.  Stack Trace:	at MediaPortalApp.OnAction(Action action)
      at MediaPortal.GUI.Library.OnActionHandler.Invoke(Action action)
      at MediaPortal.GUI.Library.GUIWindowManager.DispatchThreadMessages()
      at MediaPortalApp.FrameMove()
     

    jameson_uk

    Retired Team Member
  • Premium Supporter
  • January 27, 2005
    7,258
    2,528
    Birmingham
    Home Country
    United Kingdom United Kingdom
    Is this something that needs to be included in 1.5.0?

    Well it is totally broken at the moment so including it can not makes things much worse ;)
    I will one day get around to making this like the IMDB scrapers but I never seem to find the time for that.
     

    styrsven

    Portal Pro
    January 6, 2013
    115
    51
    63
    Home Country
    Sweden Sweden
    I tested the new version, and it is an improvement. It found info for 'Beach Boys, The'. But it didn't find the album 'Pet Sounds' by the same group. I'm attaching debug logs.
     

    Users who are viewing this thread

    Top Bottom