View Issue Details

IDProjectCategoryView StatusLast Update
0004450MMW v4Otherpublic2008-03-21 20:37
ReporterLudek Assigned To 
PriorityurgentSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version3.0 
Fixed in Version3.0 
Summary0004450: <date> tag is missing from masks
DescriptionAs reported here
http://www.mediamonkey.com/forum/viewtopic.php?t=24632&highlight=&sid=98033c17c6eba29d559b37dcbc43cd49

the <date> tag should be added to masks.
Additional InformationTicket: http://www.mediamonkey.com/support/staff/index.php?_m=tickets&_a=viewticket&ticketid=263&highlight=--+Mixed+--&ticketsearchid=350#
TagsNo tags attached.
Fixed in build1147

Relationships

related to 0004462 closedLudek Original Date field is missing in Auto-Playlist/Search criteria 
parent of 0005154 newLudek Auto-tag from Filename can't interpret dates 
Not all the children of this issue are yet resolved or closed.

Activities

jiri

2008-03-12 10:12

administrator   ~0013294

Since there shouldn't be any problem with localization, I'd suggest to implement it right away.

Another user also mentioned that Original Date would be nice to have.

Question is, how to format the date. I'd suggest to do something like:

<Date> shows date in 'technical' format, e.g. 19981123 - particularly useful for sorting. However, maybe that 1998-11-23 would be slightly better for readability reasons. In case some part of date (like day) is missing, it would be missing here to - e.g. 1998-11.

<Local Date> shows date as specified in user's locale settings, like 23.11.1998 or 11/23/98 or so.

jiri

2008-03-12 10:13

administrator   ~0013295

Reminder sent to: petr, rusty

Rusty, Petr, feel free to comment this proposal.

rusty

2008-03-12 13:18

administrator   ~0013299

I think that <Date> should by default yield a readable format that is also sortable i.e. 1998-11-23 (as you suggested).

If other formats are desired, they should probably be a modified version of <Date>. e.g. <Date:dd.mm.yyyy> or <Date:mm/dd/yy> (i.e. configurable by the user).

petr

2008-03-12 13:31

developer   ~0013300

Last edited: 2008-03-12 13:32

I think we can leave user to set date/original date criteria in same format as he's using it in tracklist. We're already have functions to convert user input into our internal date format and it will be user friendly.

Also for scripters we can add new functions SDBApplication.DecodeDate and SDBApplication.EncodeDate for converting user typed date from/to our internal date format.

jiri

2008-03-12 15:02

administrator   ~0013301

Last edited: 2008-03-12 15:03

Ok, so let's implement:

<Date> as 1998-11-23 (or 1998-11 or 1998 in case some part is missing) since it's the most important to have this sortable date format.

As for other suggestions, let's defer them, because they would either involve localization or non-trivial changes in mask handling.

Ludek, please don't forget to include this in regression testing.

Ludek

2008-03-18 11:58

developer   ~0013358

Added in build 1143 and added also regression DUnit test for this.

rusty

2008-03-20 02:51

administrator   ~0013399

Minor issues:

1) <Date> mask should appear immediately after <Year> in the list
2) <Date> mask is missing from the mask list in the Set Destination Wizard

Ludek

2008-03-20 14:43

developer   ~0013404

Fixed in build 1145.

rusty

2008-03-21 16:12

administrator   ~0013447

Minor question/issue:
-It's missing from Auto-tag from Filename. Is that intentional (i.e. because MM can't interpret the date's format)?
-It's also missing from the mask config in the Sync dialog.

Ludek

2008-03-21 20:08

developer   ~0013455

> -It's missing from Auto-tag from Filename. Is that intentional (i.e. because > MM can't interpret the date's format)?

Yes, I think that this is intentional, i.e. <year> tag is much more useful there

> -It's also missing from the mask config in the Sync dialog.

Fixed in build 1147.

rusty

2008-03-21 20:37

administrator   ~0013457

Verified 1147.