AM/PM issue in Change Attributes dialog

Bug reports will be moved here when the described bug has been fixed

Moderators: white, sheep, Hacker, Stefan2

Post Reply
User avatar
MarcinW
Power Member
Power Member
Posts: 852
Joined: 2012-01-23, 15:58 UTC
Location: Poland

AM/PM issue in Change Attributes dialog

Post by *MarcinW » 2018-06-14, 11:26 UTC

In Change Attributes dialog, when filling the Time field with:

- "1 pm" => file time is set to 13:00:00
- "1 PM" => file time is set to 01:00:00
- "1 pM" => file time is set to 01:00:00
- "1 Pm" => file time is set to 01:00:00

I suppose this is not intentional.

Regards

User avatar
petermad
Power Member
Power Member
Posts: 8066
Joined: 2003-02-05, 20:24 UTC
Location: Valsted, Denmark
Contact:

Post by *petermad » 2018-06-14, 11:47 UTC

Confirmed.

The same goes for:

- "12 am" => file time is set to 00:00:00
- "12 AM" => file time is set to 12:00:00
- "12 aM" => file time is set to 12:00:00
- "12 Am" => file time is set to 12:00:00
License #524
Danish Total Commander Translator
TC 9.21a 32+64bit on Win XP 32bit, Win 7, 8.1 & 10 64bit and TC 2.92b1 on Android 6.0
Get: Extended Total Commander Menus | PHSM-Calendar

User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 36511
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) » 2018-06-15, 10:03 UTC

Currently only lowercase am/pm and a/p are supported. I will add support for uppercase and mixed case am/pm.
Author of Total Commander
http://www.ghisler.com

User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 36511
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) » 2018-06-20, 18:31 UTC

This should work now in RC2, please test!
Author of Total Commander
http://www.ghisler.com

User avatar
petermad
Power Member
Power Member
Posts: 8066
Joined: 2003-02-05, 20:24 UTC
Location: Valsted, Denmark
Contact:

Post by *petermad » 2018-06-20, 20:53 UTC

Confirmed fixed for both 1 PM/pM/Pm and 12 AM/aM/Am in TC 9.20rc2 :-)
License #524
Danish Total Commander Translator
TC 9.21a 32+64bit on Win XP 32bit, Win 7, 8.1 & 10 64bit and TC 2.92b1 on Android 6.0
Get: Extended Total Commander Menus | PHSM-Calendar

User avatar
MarcinW
Power Member
Power Member
Posts: 852
Joined: 2012-01-23, 15:58 UTC
Location: Poland

Post by *MarcinW » 2018-06-22, 14:05 UTC

Fully fixed.

Post Reply