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.

Sign up for GitHub

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

Jump to bottom

Test ini-parameter for truthy instead of string #216

Merged

sprain merged 1 commit into sprain:master from kohlerdominik:fix-issue#214

Feb 21, 2023

Contributor

kohlerdominik commented Feb 20, 2023 •

edited

Loading

Acording to the docs, ini_get should return "", "0" or "1" for boolean config

Note: When querying boolean values A boolean ini value of off will be returned as an empty string or "0" while a boolean ini value of on will be returned as "1". The function can also return the literal string of INI value.

However, according to #214 there seem to be environments, that do not follow this. So it's probably best to test for truthy.


          Test ini-parameter for truthy instead of string

00f10a2

Owner

sprain commented Feb 21, 2023

🙏

sprain merged commit 75b45a0 into sprain:master

sprain mentioned this pull request

allow_url_fopen check in fpdf #214

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet