Bug 6006 - Favorites - can't fully rename 'By Genre' favorite
: Favorites - can't fully rename 'By Genre' favorite
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.4.0
: PC Windows XP
: P3 minor (vote)
: 8.1.0
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-11-02 15:02 UTC by Jim McAtee
Modified: 2014-10-16 22:42 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2007-11-02 15:02:13 UTC
Well, you can rename it, but it's not reflected in the breadcrumbs, remaining 'By Genre'.
Comment 1 KDF 2007-11-02 16:33:15 UTC
this is an xmlbrowser issue.  $feed->{'title'} for @crumb isn't updated when an item is changed.
Comment 2 Adrian Smith 2007-11-03 04:14:45 UTC
Is this a problem with the Default skin only?

I'm trying to understand the process to reproduce this - if you edit the title of a favorites folder, at that point you should not see that folder on the breakcrumb list?  Is this occuring when you subsequently select it?
Comment 3 Jim McAtee 2007-11-03 09:48:41 UTC
> Is this a problem with the Default skin only?

No, all skins.

> I'm trying to understand the process to reproduce this - if you edit the title
> of a favorites folder, at that point you should not see that folder on the
> breakcrumb list?  Is this occuring when you subsequently select it?

Yes.  It appears correctly on the page, but not in the breadcrumb list.  And it appears to only be a problem with the 'By Genre' favorite that was included/installed with 7.0.  If I create a new folder and subsequently rename it, it appears correctly in the breadcrumbs.

Comment 4 Adrian Smith 2007-11-03 10:33:06 UTC
What - "By Genre" favorite?  I've not seen one or intended for there to be one... 

If you look at the favorites.opml file does it include this entry?

Michael - is there something you have added in the install process creating favorites entries?
Comment 5 Jim McAtee 2007-11-03 11:16:18 UTC
> What - "By Genre" favorite?  I've not seen one or intended for there to be
> one... 

I could be mistaken that it was added with the install.  My SVN of 7.0 has been around for a while, so maybe I added it when playing with the Favorites interface at some point in the past.  It's from Slim Picks > Internet Radio > By Genre.  When I rename it to 'Slim Picks by Genre' I see the breadcrumb behavior described above.

> If you look at the favorites.opml file does it include this entry?

Yes, here's the entry:

    <outline URL="http://www.slimdevices.com/picks/split/By%20Genre.opml" text="Slim Picks by Genre" />

Could it be beacause it points to another .opml file?
Comment 6 Adrian Smith 2007-11-03 12:42:42 UTC
That's a valid favorite and would be added when you click on the favorite icon in slimpicks by genre.  (I suspect it was added by you doing that rather than the install.)

The reason it does not change the name is that xmlbrowser uses the title from the referred to opml file for the breadcrumb rather than the text entry in the refering opml file.  Given that you can't edit the slimpicks file itself you can't change the title for the page..

We probably need to to think what is the right thing to do here - at present it is operating as xmlbrowser has always done ie when it fetches a subfeed it is put into the data structure in place of the url and title - this results in what you are seeing.
Comment 7 KDF 2007-11-03 12:51:32 UTC
yes, the slim picks entries must have come from the early days of the new opml favourites.  One of the early examples was to load a copy of the Slim Picks opml file, so that's probably a hold over.
Comment 8 Jim McAtee 2007-11-03 13:08:40 UTC
Fair enough.  It's seems fairly minor, so I'm changing the severity.  See if someone can come up with a better bug title for future reference.
Comment 9 Michael Herger 2008-01-07 07:09:00 UTC
Triode - is this the same as the second part of bug 6544?
Comment 10 Adrian Smith 2008-01-07 11:13:50 UTC
I suggest this one is kept open as it remains, but there's a question of what the right thing to do here is.

I believe 6544 should be closed as I think its fixed.
Comment 11 Michael Herger 2008-11-06 08:07:51 UTC
We planned to rework the favorites/presets/radio stations/podcasts etc. functionality.
Comment 12 Michael Herger 2009-07-28 03:09:55 UTC
The favorites redesign is not going to happen for 7.4/8.0
Comment 13 Jim McAtee 2014-10-16 22:42:50 UTC
This is now working properly in both 7.8 and 7.9.