[Bugs] #3709 Jukebox UNSP: Jukebox should safely ignore when referenced media is not found
Sugar Labs Bugs
bugtracker-noreply at sugarlabs.org
Fri Jun 29 15:05:13 EDT 2012
#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
More information about the Bugs
mailing list