weird format time bug in moment about momentcfc

[atuttle]

your format string should be hh:nn tt not hh:mm tt ~ the "mm" is the month, which is why it's 02. :)

[artknight]

ah, damn it! you are correct!

[artknight]

Hi, just wondering.. I checked moment.js and they are using "mm" for minutes and "MM" for months. Do you think it would be possible to adjust this component to be fully compatible with the moment syntax?

[atuttle]

I don't see an overwhelming case for it. We're not aiming for complete parity with moment.js, and the current implementation is very much in line with dateTimeFormat() masks (in most cases, simply delegating to that method), which makes it familiar for existing CFML developers.

I'm not saying no, I'm saying convince me. ;)

[artknight]

Yeah, you are absolutely correct that the format does comply with the CF date/time format. Honestly I am not sure which way is better, but I do tend to put "mm" for months as a force of habbit and can imagine others, having worked with moment.js, will too. So, this would be a pure convenience change.