AutoHotkey Community

I think there's a problem with the handling of AM and PM for 12 O'Clock.

It fails to take into account that 12 am in midnight and 12 pm is noon, not the reverse.

For example:
2:30 am = 200706300230
2:30 pm = 200706301430
Which is correct, but:
12:30 am = 200706301230
12:30 pm = 200706302430
Which is not. 24:30 doens't even exist.

This will be a very useful function if that can be fixed though. I'd try to fix it myself, but your RegEx is way beyond me.

Thanks, I believe it has been fixed in version 1.03.
It was due to a miscalculation in my expressions, not regex

_________________
GitHub • Scripts • IronAHK • Contact by email not private message.

I'm afraid I'm still getting wierd results. Now for either "12:30 AM" or "12:30 PM" I'm getting 20070701030 which aside from being wrong for pm, is one too few digits. Once again times other than 12 work correctly.

In 1.04:

12:30 am: 200707010030
12:30 pm: 200707011230
1:30 am: 200707010130
6:37 pm 15 October 2009: 200910151837
12:30: 200707011230 (assumed pm time on it's own)
1:30: 200707011330 (like above)

I hope everything works now.

_________________
GitHub • Scripts • IronAHK • Contact by email not private message.

Works great now!

Titan,

Perhaps I was just a bit fried, but I couldn't get it to work for a simple date structure like, 1/1/2007 or 10/21/07. Does this take care of those? It would be great to have an all-inclusive function.

_________________
SilverEdge78

1/1/2007 gives 20070101 which is correct. Your second date has the month and day mixed around, I realize it's the American format but it's not how I implemented my function.

_________________
GitHub • Scripts • IronAHK • Contact by email not private message.

So 1/1/07 works only because it's the same with the month/day in either position. I imagine then that 1/31/07 would not work. Would you be up for the task for allowing the American variant of date to work using this? Or is that too hard to add in there? I don't know RegEx at all, so when I look at your function, it only makes a little sense.

I ended up making up my own translation thing to use, but it definitely doesn't use RegEx. It uses StringSplit on / and then checks the length of the string to see if it needs a 0 before the day/month.

Thanks for the clarification.

_________________
SilverEdge78

If I did that, British/European formats won't work. Instead you could word the months or use ISO date, i.e. 1st April 2007 or 2007-04-1.

_________________
GitHub • Scripts • IronAHK • Contact by email not private message.

I would do that happily if I could. I am pulling the data from an SQL database that has the date stored as 10/31/2007 (american). I figured if you tried to account for our version, that it would break the european way. So no worries. But I will see if i can use your script in other things that I do. It is a good script for certain.

Ooo...could you make two versions of the script? One for the european and one for the american? Like, v1.4A and v1.4E, where the only difference is that specific date format change.

Just a thought!

_________________
SilverEdge78

All you need to do is add the highlighted line before the main function proc.:



Alternatively you can use the same regex before you make the function call.

_________________
GitHub • Scripts • IronAHK • Contact by email not private message.

How about an optional second parameter, Titan? Something like this:
Or, as suggested before, we simply coerce everyone into adopting ISO 8601...

Sounds like a case for The Daily WTF...

_________________
Improving my world, one script at a time.
Join the AutoHotkey IRC channel: irc.freenode.net #autohotkey

For such a trivial mod users can refer to the info on this thread. Incorporating it to my official release breaks the simplicity of a single parameter and introduces culture variation dependence which can be complicated given the extra parsing rules, word translations, etc.

_________________
GitHub • Scripts • IronAHK • Contact by email not private message.

Titan, that is easy for me to do. Thanks for throwing that up here. I don't think it necessary to pass a second parameter. If I'm the first one in a year to offer this up as a concern, I don't think it really is one. As for the formating, perhaps its cause its being exported to CSV? I don't know. I just have to process the file.

Thanks again Titan!

_________________
SilverEdge78

I needed to make the following change to make American Date Format mm/dd/yyyy hh:mm.... work: