New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed issue #18401: Unable to use checkdate function if debug set #3214
Conversation
checkdate wrap added.
checkdate function result test added.
… set" This reverts commit 4c677ff.
checkdate wrap added.
Pnding of hooking new function to expression manager |
SonarCloud Quality Gate failed. |
Now should be Ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
Maybe adding some invalid date checkdate(2,31,2021) checkdate(13,13,2021)
public function testYearCheck() | ||
{ | ||
$check_string = exprmgr_checkdate(06, 07, 'Twenty Twenty-three'); | ||
$this->assertFalse($check_string, 'Unexpected evaluation result, the year parameter should not be a string.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:)
$this->assertFalse($check_string, 'Unexpected evaluation result, the year parameter should not be a string.'); | ||
|
||
$check_numeric = exprmgr_checkdate(06, 07, '2023'); | ||
$this->assertTrue($check_numeric, 'Unexpected evaluation result, the year parameter can be a numeric value.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😄
$this->assertTrue($check_numeric, 'Unexpected evaluation result, the year parameter can be a numeric value.'); | ||
|
||
$check_int = exprmgr_checkdate(06, 07, 2023); | ||
$this->assertTrue($check_int, 'Unexpected evaluation result, the year parameter can be an integer.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
No description provided.