View Issue Details

IDProjectCategoryView StatusLast Update
0006666MMW v4Main Panel/Toolbars/Menuspublic2011-06-14 11:11
ReporterLudek Assigned To 
PriorityhighSeverityminorReproducibilityalways
Status assignedResolutionreopened 
Product Version4.0 
Fixed in Version4.0 
Summary0006666: Find more from same -> Album is no longer available (regression)
DescriptionThis is a feature I use a lot, but it is no longer available in recent builds.
TagsNo tags attached.
Fixed in build1335

Relationships

related to 0006936 closedLudek Find more from same should be adapted to Video 
related to 0007796 closedLudek Find More from Same shouldn't show values not in DB when in My Computer 
parent of 0006865 closedpetr Find more from same switches Collections 

Activities

Ludek

2010-11-09 16:33

developer   ~0021274

Last edited: 2010-11-09 16:35

This is caused by this condition:

if MainTree.IsVisible[ TB.AlbumsNode[FLT_NONE]] and (SD.Album<>'') then

and it means that the bug is reproducible only if 'Entire Library' is not visible or its 'Albums' subnode is not visible.

Shouldn't we make 'Entire Library' always visible?

jiri

2010-11-09 17:20

administrator   ~0021275

Last edited: 2010-11-09 17:21

As discussed over IM with Petr, it should be fixed like this:
1. If it's clear which collection we are at, this collection should be used (e.g. if we are in a node, which is a sub-node of a collection).
2. If it isn't clear which collection to use, it should be used based on Type of the focused track (e.g. if we are in Now Playing or a Playlist). The first visible collection that contains given Type should be used.
3. Note that we should also modify the visual aspect of this feature - e.g. if working with Video, instead of Artist we should use Director in the pop-up menu.

petr

2010-11-09 19:35

developer   ~0021278

Fixed in 1324

stephen_platt

2010-11-14 04:26

developer   ~0021344

verified 1325

Ludek

2010-12-10 19:05

developer   ~0021723

Re-opening,
while fixing 0006936 I found one instance when it fails:

1. Load some Music tracks to 'Now Playing' list
2. Select any director subnode like
Video -> Director -> Director1
3. Right-click a music track on the right (in 'Now Playing') and select
Find More From Same -> Album

=> MM fails to find and focus the album node

petr

2010-12-11 20:50

developer   ~0021741

Based on current internal logic it's correct (MM is trying to locate album in current collection). Assigning to Jiri whether we should change this logic to something more sophisticated (if wasn't found on current collection, try in another ?).

jiri

2010-12-12 11:56

administrator   ~0021769

The problem seems to be that step 2. as I described above (http://www.ventismedia.com/mantis/view.php?id=6666#c21275) isn't implemented. As described there, if the search is made from outside of a Collection (which includes a NP window, as there isn't any Collection specified), then we should first find a good Collection for searching, based on Type of track. For example.
a. User executes Find more from Album on a Music track in NP.
b. In this case it _isn't_ important which Collection is selected in Tree.
c. We should go through visible Collections and find the first one that contains Music type.
d. Alternatively, we could also make sure that the track actually belongs to this Collection. This step probably isn't necessary, but might be useful in case user configures more collections/filters.
e. Find more from ... is performed on the Collection found in steps c&d.

petr

2010-12-12 12:34

developer   ~0021770

Fixed in 1335

peke

2011-01-12 23:55

developer   ~0022331

verified 1343

lowlander

2011-06-13 17:55

developer   ~0026102

Last edited: 2011-06-13 17:56

The current problem in 1388 is that if FMFS is used it will fail if just Entire Library and a Custom Collection (that doesn't contain FMFS results) is enabled. The problem with trying to use a Collection that matches when using FMFS out of Collection is that Collections can have filters excluding results. I suspect that using the following order would work better for out of Collection FMFS:
1) Use Entire Library if available
2) Use default Collection based on type if available
3) Use custom Collection

Both 2 (user can modify default Collection) and 3 run across the problem of results of FMFS being filtered out by rules set by user. Ideally FMFS would evaluate if rules do filter out results.

Alternatively out of Collection always uses Entire Library and if disabled shows the Entire Collection temporarily like the Search node. Alternatively to that FMFS runs a specific search instead of opening Collection (FMFS on Artist would run Artist:query in the search node).

petr

2011-06-13 19:12

developer   ~0026104

Assigning to Rusty for a feedback.

lowlander

2011-06-13 19:40

developer   ~0026107

Additionally there could be a user setting that would allow user to choose:
1) Always create search on FMFS use
2) Always open FMFS in Entire Library (even if hidden)
3) Use current method (not sure how to describe it)

I have seen some forum posts suggesting some user would prefer FMFS to perform a search instead of opening corresponding node in Library (now Collections).

rusty

2011-06-13 20:34

administrator   ~0026110

From my perspective:
1) If FMS is operating within a Collection, by default FMFS should search the current collection by opening a node.
2) If FMS is operating on a track that isn't within a Collection (e.g. NP, My Computer), it might be simplest to allow the user to return results within a Search node instead. Ideally, the Search node would show 'Entire Library: <field> contains <search term>' (but this would imply additional changes--currently Searches don't persist the library to which they apply.
3) Optional: For case 1), a simple means of allowing the user to choose to return FMFS results as search results instead of as a node (without adding new strings) could be via a checkbox below a horizontal bar:
[ ] <Current Collection>
[ ] Entire Library

Just a thought...

lowlander

2011-06-13 20:38

developer   ~0026111

I think something like that would work.

jiri

2011-06-13 22:21

administrator   ~0026118

I'm afraid that the proposed solution introduces a small inconsistency, which isn't nice. As discussed over IM with Petr, we are able to fix this for almost all cases that can realistically happen.

The only problem is in case when user tries to find a track that 1. Isn't in DB, and 2. Entire Library isn't visible, and 3. other specific condition re. visible collections. We will be able to fix this remaining (almost negligible) issue later after 4.0, but since it isn't a trivial fix, there's no rush.

petr

2011-06-14 11:11

developer   ~0026125

Last edited: 2011-06-14 16:40

Implemented in 1390. Reducing priority for further fix.
As discussed with Jiri by IM we've updated to work this way:
1. use current selected collection
2. if 1 fail, try to find track in visible collections (it's new step)
3. if 2 fail, try to find collection (from among visible collections) based on Track type
4. if 3 fails, try to find in 'Entire Library' (if it's visible)

Also in 1390 added a detection to show track related items in FMFS popup only when track can be located in visible collections.