After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 308208 - Play Album window does not show all albums
Play Album window does not show all albums
Status: RESOLVED FIXED
Product: muine
Classification: Other
Component: interface
trunk
Other All
: Normal major
: ---
Assigned To: Muine Maintainers
Muine Maintainers
: 315327 329011 491087 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-06-18 18:00 UTC by geraldo
Modified: 2008-09-28 13:54 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Re-implements "only_complete_albums" gconf key. (4.10 KB, patch)
2006-11-14 03:50 UTC, Diego Escalante Urrelo (not reading bugmail)
none Details | Review
[PATCH] Bug 308208 – Play Album window does not show all albums (3.88 KB, patch)
2008-09-28 13:54 UTC, Diego Escalante Urrelo (not reading bugmail)
committed Details | Review

Description geraldo 2005-06-18 18:00:32 UTC
Please describe the problem:
Various albums dont apear in the "play album". I though it was because these
albums were incomplete. But in full albums the problems apears too. 

The problem was the lack of "track number" information. After adding that info
with easytag, the album apeared.

Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Pat Suwalski 2005-10-15 02:11:12 UTC
*** Bug 315327 has been marked as a duplicate of this bug. ***
Comment 2 Pat Suwalski 2005-10-15 02:18:00 UTC
From 315327:
"10 songs with the same album in their ID3's will not be counted as an album. 10
songs with the same album in their ID3's AND in a directory with the name of
the album will be counted as an album."

While mostly true, the directory name does not seem to be an issue. However, the
tracks not only need numbering, but the numbering must be sequential.

For example, I have a CD where I deleted some tracks because I did not like them
at all, so I was left with 1, 3, 4, 6, 7, and 9. This was not considered an
album. However, renumbering the files to 1, 2, 3, 4, 5, 6 made Muine recognize
it as an album.

This behaviour is highly broken, though. I have some single tracks that are just
a song, for example track 3 out of 19. *This* is considered an album because of
the available tracks (1), all of the files are in sequence.

This behaviour is very confusing to users and took a good deal of time to discover.

Marking critical because it needs to be considered at the very least.
Comment 3 Rubin 2005-11-26 00:51:17 UTC
Hi, This happened to me also: If I have an album which has tracknumbers like 01
instead of 01/21, it won't show up in Muine. If you add the /21 (if the
particular album has 21 tracks) it will pop up in muine after a restart.

greets,

rubin at xs4all dot nl
Comment 4 Diego Escalante Urrelo (not reading bugmail) 2006-05-08 04:58:43 UTC
Is this bug still present?
Comment 5 Pat Suwalski 2006-05-08 13:38:22 UTC
In the 0.8.4 version in Debian unstable it still appears to be. But it's not clear if it's desired behaviour or not. It might also be somewhere in-between, desired but buggy.
Comment 6 Diego Escalante Urrelo (not reading bugmail) 2006-05-08 22:33:35 UTC
In that case, this should be configurable, like 'show incomplete albums'.
Comment 7 Diego Escalante Urrelo (not reading bugmail) 2006-06-20 16:31:42 UTC
Where is this handled inside the code?
I would like to add the option for showing incomplete albums.
Comment 8 Wouter Bolsterlee (uws) 2006-09-29 15:57:03 UTC
Can one of our benevolent dictators share his/her opinion about this issue? Is this WONTFIX or should an option be added?
Comment 9 Wouter Bolsterlee (uws) 2006-09-29 15:59:42 UTC
*** Bug 329011 has been marked as a duplicate of this bug. ***
Comment 10 Diego Escalante Urrelo (not reading bugmail) 2006-11-14 03:50:22 UTC
Created attachment 76542 [details] [review]
Re-implements "only_complete_albums" gconf key.

Muine's cvs logs say that long ago there was a GConf key to make everyone happy around this bug/feature of incomplete albums. This patch adds that key and its corresponding magic.

:)
Comment 11 Wouter Bolsterlee (uws) 2006-11-14 13:39:15 UTC
patch looks good, thanks.

Latexer, was this issue decided yet or is this patch ok to commit?
Comment 12 Wouter Bolsterlee (uws) 2006-11-14 13:46:34 UTC
Changing component. Filter "uws doing muine component triaging" to get rid of the spam. Thanks!
Comment 13 Pat Suwalski 2006-11-14 14:31:12 UTC
I don't see this patch solving the actual problem. It works around it by making the current behaviour official.

From what I can see in the patch, unchecking the option for complete albums doesn't actually show me *all* of my albums.

The old behaviour was that partial albums were listed in the album list too.
Comment 14 Diego Escalante Urrelo (not reading bugmail) 2006-11-14 18:07:08 UTC
Pat, the patch creates/adds a gconf key that when FALSE will make Muine have the old behavior of incomplete albums (that means ALL).
When TRUE it will have the current behavior of showing only complete albums.
Comment 15 Diego Escalante Urrelo (not reading bugmail) 2006-12-25 09:34:52 UTC
Hmmm... alo?
Comment 16 Wouter Bolsterlee (uws) 2006-12-25 13:30:53 UTC
It's up to latexer to decide.
Comment 17 Diego Escalante Urrelo (not reading bugmail) 2007-08-06 11:00:51 UTC
zing
Comment 18 Luis Medinas 2007-09-04 02:10:27 UTC
One vote more to add the patch.
Comment 19 Michelle 2007-10-28 14:50:13 UTC
*** Bug 491087 has been marked as a duplicate of this bug. ***
Comment 20 Wouter Bolsterlee (uws) 2008-09-28 13:32:54 UTC
Whatever. If Diego and Luis agree, go ahead and commit.
Comment 21 Diego Escalante Urrelo (not reading bugmail) 2008-09-28 13:54:01 UTC
Created attachment 119530 [details] [review]
[PATCH] Bug 308208 – Play Album window does not show all albums


Revert an old commit removing only_complete_albums functionality.

You can now choose wheter muine should or should not show incomplete
albums using a gconf key: /apps/muine/only_complete_albums.
This is the	old-old-really-old behaviour of muine of including every
album in the list no matter it is not complete.
---
 ChangeLog             |   15 +++++++++++++++
 data/muine.schemas.in |   11 +++++++++++
 src/Album.cs          |    7 ++++++-
 src/SongDatabase.cs   |   20 ++++++++++++++++++--
 4 files changed, 50 insertions(+), 3 deletions(-)
Comment 22 Diego Escalante Urrelo (not reading bugmail) 2008-09-28 13:54:53 UTC
Committed the updated version.