Issue Details (XML | Word | Printable)

Key: MBS-3554
Type: Bug Bug
Status: Closed Closed
Resolution: Won't Fix
Priority: Low Low
Assignee: Oliver Charles
Reporter: voiceinsideyou
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
MusicBrainz Server

Taglookup for artist/release/duration should do a release search rather than recording search

Created: 29/Sep/11 05:01 PM   Updated: 20/May/12 03:14 AM   Resolved: 03/Feb/12 03:12 PM
Component/s: Search
Affects Version/s: None
Fix Version/s: None

Issue Links:
Relates
 



Sort Order: Ascending order - Click to sort in descending order
Oliver Charles added a comment - 29/Sep/11 05:46 PM

Why should it do a release search?


Oliver Charles added a comment - 29/Sep/11 05:46 PM

Oh, I see, that's the duration of the entire release. Hrm. Is that actually the normal use of it?


voiceinsideyou added a comment - 30/Sep/11 12:25 AM

I don't actually know Also not sure if the NGS search has the information to make use of it - especially with multiple mediums. It's a bit weird; as surely using n-tracks would be a more useful parameter - but that's not passed through.

Just something I noticed that was different than classic really.


voiceinsideyou added a comment - 28/Jan/12 10:28 AM

Having taken a look at this, I think it definitely was supposed to work (it was the old behaviour), however this isn't really (easily) fixable currently I believe since album/release length isn't part of the search server's indexes and we're using indexed search for tag lookup. In any case, there'd probably be just as good results for such a search using #tracks rather than duration; so doesn't really make sense to fix this.

As part of PICARD-77 I've fixed this in Picard so it doesn't send the duration for release/cluster lookups [1] so if it gets merged this problem will go away to some extent (people may then just complain that the results suck, which they indeed do )

(Assigning to you just so you're aware Ollie, unassign afterwards!)

[1] https://github.com/voiceinsideyou/picard/commit/9abf6874d32e6897f7df5a2571d246c8748afac1


Oliver Charles added a comment - 03/Feb/12 03:12 PM

Ok, thanks for looking into this! That makes this MBS ticket a wontfix, right?


voiceinsideyou added a comment - 03/Feb/12 04:46 PM

Yeah. I'll try and open something else to suggest ways of improving tag-based release/cluster search at some point.