[Bugs] #3709 Jukebox UNSP: Jukebox should safely ignore when referenced media is not found

[Sugar Labs Bugs]

#3709: Jukebox should safely ignore when referenced media is not found
------------------------------------------+---------------------------------
    Reporter:  manuq                      |          Owner:  godiard          
        Type:  defect                     |         Status:  assigned         
    Priority:  Unspecified by Maintainer  |      Milestone:  0.98             
   Component:  Jukebox                    |        Version:  Unspecified      
    Severity:  Unspecified                |       Keywords:  patch, screenshot
Distribution:  Unspecified                |   Status_field:  Unconfirmed      
------------------------------------------+---------------------------------

Comment(by godiard):

 Replying to [comment:12 humitos]:
 > Replying to [comment:11 godiard]:
 > > I prefer go for the non complicate route :)
 >
 > What's is the way that you prefer? The: "N tracks not found."?

 No, show one message by media.

 >
 > > Personally, I am not sure "Track ''words2.ogg'' from ''HUMITOS'' not
 found." is a good message. What is HUMITOS? And the logic to know this is
 a path pointing to a non munted device can be tricky (now are in
 /run/media/olpc/ but in older versions were in /media). We already are
 showing the path in the mounted device palette. Is not nice, but is
 information needed.
 >
 > "HUMITOS" is the volume label of the Stick and is given by the dbus (or
 GIO) API. It's something that Sugar itself already known. I mean, Sugar
 shows the volume label on the palette.
 >

 I know that, but will the user understand it? If we do this, will need:

  * Check if the path is in one of the directories use to mount devices
 ('/mnt', '/media', '/run/media/$USER/') and keep the remaining part in the
 path.

  * If there are any error show the path.

  * If is not in any of this directories show the path.

  * If the media was in the journal will have a file name as
 3ae5362afe6af66aae23a366, what we should show in that case?

  * Any other case?

 > > > I guess it might have been easier to just disable/grey out missing
 tracks in the playlist ;)
 > >
 > > Disable and don't show a message? And the user should guess what
 happened?
 >
 > I think we should show a message here AND disable the tracks that can't
 be played. Going on this way will show a good explanation message and will
 show a good GUI that reflects the message shown

 I have not tried disable elements in a listview, but if is possible, we
 can do it in the process where we try to get the title with the metadata.

 I think we can do another patch with the message changes, if there are not
 other objections about the patch, I will push it.

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/3709#comment:13>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system