We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
adult = "A String False"
For example, if in here, user sent API call that adult = "A String False"
shoptimizer/shoptimizer_api/optimizers_builtin/adult_optimizer.py
Line 89 in a9fff3a
Then in here the UT will fail, as shoptimizer will send string "False" directly outside.
shoptimizer/shoptimizer_api/optimizers_builtin/adult_optimizer_test.py
Line 273 in a9fff3a
Because in Python3, if not "any string" its value will be False in boolean. Then this will cause all these two if being passed
if not "any string"
False
boolean
if
Line 92 in a9fff3a
Line 101 in a9fff3a
Then shoptimizer don't think its adult field need to change or santinize.
Just ask, is this phenomenon within the expectation?
The text was updated successfully, but these errors were encountered:
No branches or pull requests
For example, if in here, user sent API call that
adult = "A String False"
shoptimizer/shoptimizer_api/optimizers_builtin/adult_optimizer.py
Line 89 in a9fff3a
Then in here the UT will fail, as shoptimizer will send string "False" directly outside.
shoptimizer/shoptimizer_api/optimizers_builtin/adult_optimizer_test.py
Line 273 in a9fff3a
Because in Python3,
if not "any string"
its value will beFalse
inboolean
. Then this will cause all these twoif
being passedshoptimizer/shoptimizer_api/optimizers_builtin/adult_optimizer.py
Line 92 in a9fff3a
shoptimizer/shoptimizer_api/optimizers_builtin/adult_optimizer.py
Line 101 in a9fff3a
Then shoptimizer don't think its adult field need to change or santinize.
Just ask, is this phenomenon within the expectation?
The text was updated successfully, but these errors were encountered: