{"action":"created","issue":{"url":"https://api.github.com/repos/sampsyo/beets/issues/1475","labels_url":"https://api.github.com/repos/sampsyo/beets/issues/1475/labels{/name}","comments_url":"https://api.github.com/repos/sampsyo/beets/issues/1475/comments","events_url":"https://api.github.com/repos/sampsyo/beets/issues/1475/events","html_url":"https://github.com/sampsyo/beets/pull/1475","id":80347147,"number":1475,"title":"Always have a non-degenerate path to display for *Task","user":{"login":"jamesob","id":73197,"avatar_url":"https://avatars.githubusercontent.com/u/73197?v=3","gravatar_id":"","url":"https://api.github.com/users/jamesob","html_url":"https://github.com/jamesob","followers_url":"https://api.github.com/users/jamesob/followers","following_url":"https://api.github.com/users/jamesob/following{/other_user}","gists_url":"https://api.github.com/users/jamesob/gists{/gist_id}","starred_url":"https://api.github.com/users/jamesob/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamesob/subscriptions","organizations_url":"https://api.github.com/users/jamesob/orgs","repos_url":"https://api.github.com/users/jamesob/repos","events_url":"https://api.github.com/users/jamesob/events{/privacy}","received_events_url":"https://api.github.com/users/jamesob/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-05-25T02:18:27Z","updated_at":"2015-05-25T19:55:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/sampsyo/beets/pulls/1475","html_url":"https://github.com/sampsyo/beets/pull/1475","diff_url":"https://github.com/sampsyo/beets/pull/1475.diff","patch_url":"https://github.com/sampsyo/beets/pull/1475.patch"},"body":"As I started to run an import, I noticed both in the autotagging prompts and logfile that certain filenames were instead printed as `None`, e.g.\r\n```\r\nNone (1 items)\r\nFinding tags for album \"MF DOOM - MM..FOOD\".\r\nCandidates:\r\n```\r\n\r\nIt turns out that `ImportTask.paths` is referenced a bunch of places as an assumed-truthy value. To support these references, I've added a property called `paths_for_display` which will fall back to the task's `items` should its `paths` be falsey.\r\n\r\nAfter changes, we have\r\n\r\n```\r\n/Users/job/Music/collection/4ize/MM..Food_/Guinnesses.mp3 (1 items)\r\nFinding tags for album \"MF DOOM - MM..FOOD\".\r\nCandidates:\r\n```\r\n\r\nPlease advise in terms of any tests that need to be updated."},"comment":{"url":"https://api.github.com/repos/sampsyo/beets/issues/comments/105299853","html_url":"https://github.com/sampsyo/beets/pull/1475#issuecomment-105299853","issue_url":"https://api.github.com/repos/sampsyo/beets/issues/1475","id":105299853,"user":{"login":"sampsyo","id":188033,"avatar_url":"https://avatars.githubusercontent.com/u/188033?v=3","gravatar_id":"","url":"https://api.github.com/users/sampsyo","html_url":"https://github.com/sampsyo","followers_url":"https://api.github.com/users/sampsyo/followers","following_url":"https://api.github.com/users/sampsyo/following{/other_user}","gists_url":"https://api.github.com/users/sampsyo/gists{/gist_id}","starred_url":"https://api.github.com/users/sampsyo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sampsyo/subscriptions","organizations_url":"https://api.github.com/users/sampsyo/orgs","repos_url":"https://api.github.com/users/sampsyo/repos","events_url":"https://api.github.com/users/sampsyo/events{/privacy}","received_events_url":"https://api.github.com/users/sampsyo/received_events","type":"User","site_admin":false},"created_at":"2015-05-25T19:55:47Z","updated_at":"2015-05-25T19:55:47Z","body":"Great. Thanks for the extra detail.\r\n\r\nI think it was an oversight not to set the task's paths in the first place. The `paths` field has two purposes: display/logging, as you found here, and also progress/history tracking. For both purposes, I'm pretty sure we want grouped-album tasks to have paths that reflect the individual files involved. So I'm pushing a change now that initializes the field with that in mind.\r\n\r\nSee also #825, which brought up the same logging issue.\r\n\r\nWould you mind trying this out to make sure I didn't break anything?"}}