Bug 14024 - Importer Scan failing- playlists missing
: Importer Scan failing- playlists missing
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.4.0
: PC Other
: P1 major (vote)
: 7.4.0
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-09-14 09:17 UTC by Matt Cuson
Modified: 2009-10-05 14:26 UTC (History)
4 users (show)

See Also:
Category: ---


Attachments
Matt's work itunes xml file (258.33 KB, application/xml)
2009-09-23 08:31 UTC, Matt Cuson
Details
Proposed patch (989 bytes, patch)
2009-09-23 09:52 UTC, Alan Young
Details | Diff
current itunes playlist (257.04 KB, application/xml)
2009-09-23 11:02 UTC, Matt Cuson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Cuson 2009-09-14 09:17:13 UTC
[09-09-14 09:15:28.2802] main::main (184) Starting Squeezebox Server scanner (v7.4, r28505, Sun Sep 13 04:01:02 PDT 2009) perl 5.008008
[09-09-14 09:15:28.6864] Slim::Music::Import::runImporter (552) Starting Slim::Music::MusicFolderScan scan
[09-09-14 09:15:29.1135] Slim::Utils::Scanner::scanDirectory (315) Found 2 files in /Users/mcuson/Music/iTunes/iTunes Music
[09-09-14 09:15:29.1146] Slim::Utils::Scanner::scanDirectory (328) Scanning: /Users/mcuson/Music/iTunes/iTunes Music/www.freeplaymusic.com/Action and Adventure Vol 3/Stormfront.mp3
[09-09-14 09:15:29.1198] Slim::Utils::Scanner::scanDirectory (328) Scanning: /Users/mcuson/Music/iTunes/iTunes Music/www.freeplaymusic.com/Action And Adventure Vol 7/Lower Ground 60.mp3
[09-09-14 09:15:29.1249] Slim::Music::Import::endImporter (686) Completed Slim::Music::MusicFolderScan Scan in 0 seconds.
[09-09-14 09:15:29.1256] Slim::Music::Import::runImporter (552) Starting Slim::Music::PlaylistFolderScan scan
[09-09-14 09:15:29.1375] Slim::Utils::Scanner::scanDirectory (315) Found 2 files in /Users/mcuson/Music/Playlists
[09-09-14 09:15:29.1396] Slim::Music::Import::runImporter (552) Starting Slim::Plugin::iTunes::Importer scan
[09-09-14 09:15:29.9360] main::main (300) Error: Failed when running main scan: [Usage: artist(self) at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Schema/RemoteTrack.pm line 197.
]
[09-09-14 09:15:29.9363] main::main (301) Error: Skipping post-process & Not updating lastRescanTime!
Comment 1 James Richardson 2009-09-14 09:46:52 UTC
Scanner does not pick up Playlists when doing a first time scan (after install) or change and update (after install)
Comment 2 SVN Bot 2009-09-14 09:49:21 UTC
 == Auto-comment from SVN commit #28511 to the slim repo by andy ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28511 ==

Fixed bug 14024, make artist a rw accessor on RemoteTrack
Comment 3 Alan Young 2009-09-15 09:30:48 UTC
The fix is not right as it does not make sense to set the artist attribute on a RemoteTrack object. Reverting the fix.

Can QA please narrow down the circumstance under which this occurs?
Comment 4 SVN Bot 2009-09-15 09:35:02 UTC
 == Auto-comment from SVN commit #28526 to the slim repo by ayoung ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28526 ==

bug 14024: Importer Scan failing- playlists missing 
revert change 28511
Comment 5 James Richardson 2009-09-17 10:16:16 UTC
QA verified fixed with latest chagnes
Comment 6 Matt Cuson 2009-09-21 16:26:04 UTC
Sorry but it's still failing on my system. Reopening.
Comment 7 Alan Young 2009-09-21 22:31:35 UTC
Matt, can you attach the playlists(s) in question to the bug.
Comment 8 Michael Herger 2009-09-22 02:26:37 UTC
*** Bug 13906 has been marked as a duplicate of this bug. ***
Comment 9 Matt Cuson 2009-09-22 12:11:02 UTC
Just updated 28589 and got another itunes scan failure.  Here's the scanner log:

[09-09-22 12:04:18.8277] main::main (184) Starting Squeezebox Server scanner (v7.4, r28589, Tue Sep 22 04:05:59 PDT 2009) perl 5.008008
[09-09-22 12:04:19.5258] main::main (260) Removing artwork cache...
[09-09-22 12:04:19.9240] Slim::Music::Import::runImporter (562) Starting Slim::Plugin::iTunes::Importer scan
[09-09-22 12:08:01.0399] Slim::Formats::Movie::getTag (71) Warning: Use of uninitialized value in string eq at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Formats/Movie.pm line 71.
[09-09-22 12:08:01.0403] Slim::Formats::Movie::getTag (71) Warning: Use of uninitialized value in string eq at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Formats/Movie.pm line 71.
[09-09-22 12:08:02.6118] Slim::Formats::Movie::getTag (71) Warning: Use of uninitialized value in string eq at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Formats/Movie.pm line 71.
[09-09-22 12:08:02.6122] Slim::Formats::Movie::getTag (71) Warning: Use of uninitialized value in string eq at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Formats/Movie.pm line 71.
[09-09-22 12:08:06.5482] Slim::Formats::Movie::getTag (71) Warning: Use of uninitialized value in string eq at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Formats/Movie.pm line 71.
[09-09-22 12:08:06.5486] Slim::Formats::Movie::getTag (71) Warning: Use of uninitialized value in string eq at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Formats/Movie.pm line 71.
[09-09-22 12:08:09.9026] main::main (300) Error: Failed when running main scan: [Usage: artist(self) at /Library/PreferencePanes/Squeezebox.prefPane/Contents/server/Slim/Schema/RemoteTrack.pm line 197.
]
[09-09-22 12:08:09.9033] main::main (301) Error: Skipping post-process & Not updating lastRescanTime!
Comment 10 Michael Herger 2009-09-22 13:29:17 UTC
> How do I do that?  The playlists, are iTunes playlists.

Please contact James or Steven. They will be able to help you.

BTW: if you want to update a bug report the easy way, don't forward the  
mail to some developer, but simply reply to the bugzilla mail. It will  
automatically add your mail as a comment to the bug.
Comment 11 Matt Cuson 2009-09-22 19:41:42 UTC
Deleted all 7.4 files and did a clean install of 7.3.3.  Everything worked fine.  Including access to playlist.  It also included the smart playlists (purchased, most recent, most frequent, itunes dj etc.).

Then performed an upgrade to 7.4 (build 28603).  Itunes failed import.  Playlists showed empty.  But I was able to navigate the music library

Scanner log showed same point of failure as my earlier posts (remotetrack.pm line 197).

Scanner console showed repeated attempts to scan with immediate failures.  

it would appear that the path names for music and playlists were preserved from the upgrade but the itunes function didn't work.

Not sure what else i can do at this point.
Comment 12 Alan Young 2009-09-23 06:46:01 UTC
My guess is that we are tripping over the default assumption that all URLs that do not start "file:" must be remote. Nonetheless, I'm not sure why it shoudl matter.
Comment 13 Alan Young 2009-09-23 08:26:58 UTC
Matt, pelase attach you itues xml file to this bug.
Comment 14 Matt Cuson 2009-09-23 08:31:00 UTC
Created attachment 5899 [details]
Matt's work itunes xml file

This file works properly with 7.3.3.  However it causes a scan failure with 7.4 and the scanner never properly captures the playlists.
Comment 15 Alan Young 2009-09-23 09:46:18 UTC
x
Comment 16 Alan Young 2009-09-23 09:52:53 UTC
Created attachment 5900 [details]
Proposed patch
Comment 17 SVN Bot 2009-09-23 10:03:19 UTC
 == Auto-comment from SVN commit #28617 to the slim repo by ayoung ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28617 ==

Fixed bug 14024: Importer Scan failing- playlists missing 
Need to map contributer track attributes.
Comment 18 Michael Herger 2009-09-23 10:10:01 UTC
Matt - I triggered a new build (28618). It should be up on  
http://downloads.slimdevices.com/nightly/?ver=7.4 in a few minutes.
Comment 19 Matt Cuson 2009-09-23 11:02:08 UTC
Created attachment 5901 [details]
current itunes playlist

Sorry but the new build didn't work.  I installed over the old 7.4 and when that didn't work, I deleted everything and did a clean install.  that didn't work.  I then deleted a song from itunes to force a rescan.  that didn't work.

Also, that version of 7.4 had "use itunes" checkbox OFF.  I was under the impression that "use itunes" is to be checked on by default.

BTW, I'm not seeing the iTunes scan error anymore but I'm still not getting itunes playlists in my playlists folder (only infobrowser.opml)
Comment 20 Alan Young 2009-09-23 11:42:14 UTC
Can someone from QA please work with Matt to see if you can diagnose what is going on.

Matt, I presume that you did check the "use itunes" checkbox?

Obviously, I do not have your complete library, so I could not use your complete xml file. I cut it down to just a single (remote) track and a single playlist that included that remote track. I was able to reproduce the error before the fix and not afterwards. I was able to see the playlist in My Music / Playlists and to play it.
Comment 21 Matt Cuson 2009-09-23 14:51:04 UTC
uploaded my itunes folder here

http://files.me.com/mattcuson/153zd7

Only up for a couple days if you want it.
Comment 22 Matt Cuson 2009-09-23 15:58:56 UTC
I think I discovered the problem.  Might be what Anoop saw also.  It would appear that the installer doesn't set "use itunes" on by default (or somehow deep hidden files on my system insist that it be turned off).  When i select "use itunes" in the control panel, the check box changes but the itunes plug-in isn't actually loaded.  I have to go into Advanced > plug-ins > enable itunes > restart etc. and then step 7 finally happens.
Comment 23 Matt Cuson 2009-09-23 16:05:06 UTC
repeated the process from a clean install and got the same results.  After initial scan, music was available but no playlists.  Then go to advanced > plug-ins and enable itunes.  On restart there's a new scan and the playlists are picked up.

at least we have a workaround!
Comment 24 Anoop Mehta 2009-09-23 16:26:37 UTC
also see bug 14237
Comment 25 Michael Herger 2009-09-23 22:57:24 UTC
I'm closing this bug as playlist scanning from iTunes is working fine when iTunes is enabled. Bug 14237 should cover the question why it is disabled on your computer. Please follow up on bug 14327. Thanks!
Comment 26 James Richardson 2009-10-05 14:26:42 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.