[PATCH] Re: atomicparsley tagging issue

[dinkypumpkin]

On 1 Apr 2011, at 23:04, Jon Davies wrote:

> On 9 February 2011 22:55,  <grz88 at wolke7.net> wrote:
>> However, I wonder why does get_iplayer use the short description (descshort) field, rather that the long description (desc) for both the "description" and "comment" tag. Wouldn't it make more sense to tag the file using the full programme description? Maybe there is good reason why things are the way they are. If not, then I suggest changing the invocation of atomicparsley to use the "desc" field for the "description" tag (and keep shortdesc for the comment). Below is a patch that does this.

At least in iTunes, the "description" and "comment" fields are both limited to 255 chars, so the full "desc" value (which can be quite long for some programmes) will be truncated.  I guess it boils down to whether truncation matters and whether iTunes should be the yardstick, though I don't think it's unique in this respect.  Alternatively, the full "desc" value can be shoved into the "lyrics" field used by iTunes and other apps.  I've worked out the full mapping from get_iplayer fields to all iTunes fields for both ID3 and MP4, which I use in a command script launched by get_iplayer to automatically tag files and put them in iTunes.  I'll work out the equivalent id3v2 and atomicparsley command lines and perhaps start a new thread on this issue.  I think the tagging in get_iplayer could be extended a bit, and it would be useful to get feedback from people using other media managers and maybe put together a more comprehensive tagging patch.