Bug 4059 – Browse Genres > GENRE level don't work when group compilations set

Bug 4059 - Browse Genres > GENRE level don't work when group compilations set

Summary:

Browse Genres > GENRE level don't work when group compilations set

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	6.5b1
Platform:	PC Windows XP

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	KDF

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2006-09-06 10:16 UTC by Nigel Birch
Modified:	2008-09-15 14:39 UTC (History)
CC List:	2 users (show)

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Nigel Birch 2006-09-06 10:16:52 UTC

This is with SlimServer Version: 6.5b1 - 9424 - Windows XP - EN - cp1252, Perl Version: v5.8.7 MSWin32-x86-multi-thread, MySQL Version: 5.0.22-community-nt.

1. Using the Default skin, go to Browse Genres.
2. Choose a Genre, e.g. Electronica.
3. Attempt to play or add any artist, e.g. Labradford.
4. Play will clear the Now Playing playlist but not add the selected tracks.  Add won't do anything.

I'm speculating, but perhaps this is related to bug 2983. I'm also seeing the broken breadcrumb list for genres.

Thanks.

Comment 1 Nigel Birch 2006-09-06 10:19:32 UTC

Opps. I mean bug 3983.

Comment 2 KDF 2006-09-06 14:33:56 UTC

I'm seeing this on one system, but not the other.  urls for the commands should include genre.id, but instead are using genreTracks.genre

Comment 3 KDF 2006-09-06 15:05:01 UTC

Ah, it has a direct link to the Group Compilations setting.

set as group, and it's buggered.
set to list under each artist, and it's fine.

same exact thing for bug 3983

I think we're missing a clone of the $find hash here, and it's getting buggered by the VA count.

Pages/BrowseDB, line 353 is the problem for both bugs

Comment 4 KDF 2006-09-06 15:50:57 UTC

got a fix for this, will commit this evening.

Comment 5 KDF 2006-09-06 18:10:23 UTC

fixed at change 9463

Comment 6 Nigel Birch 2006-09-07 10:19:20 UTC

Works for me.  Thanks again.