Update allmusic.com scraper (1 Viewer)

[styrsven]

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]

Updated with v2b.
This should sort out the artist prefixes (eg=> Beatles, The => The Beatles)

 

[HomeY]

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.

 

[jameson_uk]

Version 2 removed as I think I have broken it...

 

[HomeY]

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

 

[SilentBob]

When I try to get Info I get NullReference Exceptions in log

[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]

Added v3 to first post. I have been testing a fair bit and this seems to resolve most of the issues I could find.

 

[elliottmc]

Is this something that needs to be included in 1.5.0?

 

[jameson_uk]

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]

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.