Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search



Donate

Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Thank You for your continued support and contributions!

+  Rockbox Technical Forums
|-+  Rockbox Development
| |-+  Feature Ideas
| | |-+  Album Art finder should also look for a file named after Song Title in metadata
« previous next »
  • Print
Pages: [1]

Author Topic: Album Art finder should also look for a file named after Song Title in metadata  (Read 3669 times)

Offline Falco98

  • Member
  • *
  • Posts: 153
Album Art finder should also look for a file named after Song Title in metadata
« on: January 11, 2010, 07:30:57 PM »
There should be another option on the list of prioritized options for where to place Album Art images and the filenames. There are currently 7:
http://www.rockbox.org/wiki/AlbumArt

My suggested addition would become the new option 2, pushing all others down.
The first option is currently the only option to allow for song-specific artwork to be displayed, but this requires naming the image files *exactly* like the corresponding sound file, which screws up directory sorting.

Therefore the option should be added that if no image file has an identical name to the sound file playing, it then looks for an image file which matches the current song's Title metadata, similarly to how the following option looks for a match to the Album metadata.

This could cause conflicts when two songs have the same name, or when a song shares the album's name, or the song name contains unusable characters, but such conflicts are rare and minimal, and if anyone cares all that much about such things there is already the built-in flexibility to get around such issues. The additional code required should be minimal (and already blueprinted within the other options), and have essentially zero impact on any other functionality.

I may try to write this patch myself (as i did with "shuffle + track skip"), but if anyone beats me to it, it would be appreciated :)


Edited to Add:  I have created a working patch for this.  It works as intended in the ipod 5.5 simulator.  The code required is very small.
Could someone reopen FS#10905 so I can attach the patch there?
http://www.rockbox.org/tracker/task/10905

Thanks!



Edited to add (x2): Someone reopened the patch on FS for me, and the patch is posted.  If anyone can check it out and comment, please let me know!
* album art artist augmentation.patch.txt (1.35 kB - downloaded 159 times.)
« Last Edit: January 12, 2010, 01:44:03 PM by Falco98 »
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: Album Art finder should also look for a file named after Song Title in metadata
« Reply #1 on: January 13, 2010, 05:10:03 AM »
What do you mean by "screws up directory sorting" exactly?

If the song files don't have a problem being that name, why would the image files?
Logged

Offline Falco98

  • Member
  • *
  • Posts: 153
Re: Album Art finder should also look for a file named after Song Title in metadata
« Reply #2 on: January 18, 2010, 09:35:36 PM »
Quote from: Llorean on January 13, 2010, 05:10:03 AM
What do you mean by "screws up directory sorting" exactly?

If the song files don't have a problem being that name, why would the image files?

'screws up' in an aesthetic sense moreso than in any computer-sorting sense.

that is to say, when i browse into a directory, my music files (which i personally name like "02 - title of the song.ogg" etc) should all 'sort' together; having images with filenames exactly matching the corresponding audio files, suddenly a sorted listing of the directory has images and songs interspersed.  This is a very simple alternative option to that issue - the images can just take the name of the track title, nothing else.
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: Album Art finder should also look for a file named after Song Title in metadata
« Reply #3 on: January 18, 2010, 10:21:44 PM »
Wouldn't a simpler solution be to set the file view mode to "music" rather than "supported" so you don't see the image files at all?
Logged

Offline karashata

  • Rockbox Expert
  • Member
  • *
  • Posts: 603
  • Just a fluffy dragon with too much time on my paws
    • Alex Vanderpol (karashata) on Twitter
Re: Album Art finder should also look for a file named after Song Title in metadata
« Reply #4 on: January 19, 2010, 10:25:42 AM »
Alternatively, you could just set Rockbox to sort the files by type... (Go to Settings > General Settings > File View > Sort Files and select "By Type".)
Logged
Welp, my iriver H10 20GB is out of commission again.

Offline GodEater

  • Member
  • *
  • Posts: 2829
Re: Album Art finder should also look for a file named after Song Title in metadata
« Reply #5 on: January 19, 2010, 10:44:16 AM »
Which I think is an even better solution ;)
Logged

Read The Manual Please

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Album Art finder should also look for a file named after Song Title in metadata
« Reply #6 on: January 19, 2010, 03:54:50 PM »
I mentioned my objections in IRC (and the tracker I think), but I think my big issue can be cleanly fixed now.

My problem with this was that you still force your naming scheme on someone else, but thanks to the playlist viewer it shuold be really easy to now make the AA search look up any name based on the id3 info of a track.
The way it should be done is with a "custom" scheme being the highest priority check, this scheme comes from a setting in the .cfg "AA custom search" (or something) which would look like "%it.jpg" which would get translated to "currently playing track title.jpg" by the AA loader.

In fact! once I enable support for the directory structure you could setup a custom scheme like "/AA/%ia/%id/%it.jpg" which gives the user MUCH more flexibility than the current system.

now, if you implement that, it would go in without question.

edit: this might require a new tag added to get the size string from the wps, which is no big deal
« Last Edit: January 19, 2010, 03:56:54 PM by JdGordon »
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

  • Print
Pages: [1]
« previous next »
+  Rockbox Technical Forums
|-+  Rockbox Development
| |-+  Feature Ideas
| | |-+  Album Art finder should also look for a file named after Song Title in metadata
 

  • SMF 2.0.17 | SMF © 2019, Simple Machines
  • Rockbox Privacy Policy
  • XHTML
  • RSS
  • WAP2

Page created in 0.101 seconds with 15 queries.