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

Do not pass null value to strtolower() #127

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mplattu
Copy link

@mplattu mplattu commented Apr 17, 2024

Passing a null value to strtolower() is deprecated in PHP 8.1. However, without output shell_exec() returns null instead of an empty string, which causes a deprecation warning.

I wrote the if clause open instead of using the ternary operator to improve the code readability.

This fixes issue #124

Passing a null value to strtolower() is deprecated in PHP 8.1. However,
without output shell_exec() returns null instead of an empty string,
which causes a deprecation warning.

I wrote the if clause open instead of using the ternary operator to
improve the code readability.
* All fonts are licensed with OFL and the license texts are now
  attached to the fonts (Gregwar#102)
* Replaced font 2 with a one with both upper and lower case
  fonts (Gregwar#83)
* Font 5 (Sling) did not contain fonts to some 8 bit characters
  (ÅÄÖ)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant