Skip to content
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

Bug at Line 588? #86

Open
wolf-in-sheeps-clothing opened this issue Aug 20, 2019 · 1 comment
Open

Bug at Line 588? #86

wolf-in-sheeps-clothing opened this issue Aug 20, 2019 · 1 comment
Assignees
Labels

Comments

@wolf-in-sheeps-clothing

Hi,

the script found 2 php.ini. All of them have a correct date.timezone. But in the output I see this:

  • At least one php.ini file has no valid timezone setting

At Line 27 you set PHPINITIMEZONEMISSING=false

And at Line 294 to 302 you set this to "true" if you find a php.ini without a correct date.timezone.
Right?

So, then in my opinion Line 588 should be:

if [ ${PHPINITIMEZONEMISSING} = true ]

Otherwise the If Statement is everytime true.

Greetings

My System: Ubuntu Server 16.04.6 x64

@widhalmt
Copy link
Member

Good catch! Thank you.

@widhalmt widhalmt self-assigned this Aug 22, 2019
@widhalmt widhalmt added the bug label Aug 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants