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 almost 10 years ago
- Status changed from New to InProgress
- Assignee set to Benoit Daccache
Updated by Benoit Daccache almost 10 years ago
- Status changed from InProgress to Resolved
Updated by Benoit Daccache almost 10 years ago
Fixed also for seconds + Fixed in JWT
Updated by Patrick Oppenlander almost 10 years ago
I've just updated my git tree and this patch doesn't appear to be in there yet?
Updated by Benoit Daccache almost 10 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 over 9 years ago
- Status changed from Resolved to Closed
Actions