Actions
Bug #3948
closedWTimeEdit regexp for "mm" is incorrect
Start date:
04/03/2015
Due date:
% Done:
0%
Estimated time:
Description
The leading zero for minutes should not be optional.
The following patch fixes the bug.
diff --git a/src/Wt/WTime.C b/src/Wt/WTime.C
index 9b0fdd8..b8844a8 100644
--- a/src/Wt/WTime.C
+++ b/src/Wt/WTime.C
@@ -536,7 +536,7 @@ WTime::RegExpInfo WTime::formatMinuteToRegExp(WTime::RegExpInfo& result,
if (sf == "m") /* Minutes without leading 0 */
result.regexp += "(0|[1-5]?[0-9])";
else /* Minutes with leading 0 */
- result.regexp += "([0-5]?[0-9])";
+ result.regexp += "([0-5][0-9])";
return result;
}
Updated by Koen Deforche over 9 years ago
- Status changed from New to InProgress
- Assignee set to Benoit Daccache
Updated by Benoit Daccache over 9 years ago
- Status changed from InProgress to Resolved
Updated by Benoit Daccache over 9 years ago
Fixed also for seconds + Fixed in JWT
Updated by Patrick Oppenlander over 9 years ago
I've just updated my git tree and this patch doesn't appear to be in there yet?
Updated by Benoit Daccache over 9 years ago
It has been pushed in the private git, It will be normally pushed to the public git in the following hours.
Benoit
Updated by Koen Deforche about 9 years ago
- Status changed from Resolved to Closed
Actions