[mythtv] [mythtv-commits] Ticket #6943: Mythvideo filename parsing

[Michael T. Dean]

On 08/30/2009 03:24 AM, MythTV wrote:
> #6943: Mythvideo filename parsing
>
>  Refers to ticket 6942 (closed).
>  I really hate creating a duplicate ticket, but I wasn't really given a
>  choice in the matter.
>
>  The suggested fix (which I've now attached) doesn't break existing
>  functionality as the redundent space in the episode marker pattern is
>  handled equally well by the generic seperator pattern.

FWIW, the suggested fix /does/ break pattern matching for some of the 
previously-allowed formats.  The space was allowing all of space, 
period, dash, and underscore (as the others are replaced with spaces 
before the pattern matching).

So, before the change, the following were valid formats:

Lost-1-1-Pilot.mpg
Lost_1_1_Pilot.mpg
Lost_1-1_Pilot.mpg
Lost.1.1.Pilot.mpg
Lost 1 1 Pilot.mpg

The reason the fix is changing the behavior--even though the optional 
separator allows a space--is because the optional separator is, er, 
optional, but the episode separator is mandatory.  Therefore, removing 
space from the list of valid episode separators, means that /only/ e, x, 
and episode are valid episode separators.

However, Robert decided that it would likely be easier for users to 
understand why some formats fail if he just limited the list of valid 
episode separators to e, x, and episode, and no longer allowed space, 
period, dash, and underscore.

Mike