Skip to content

Restore httpStatusCodetoString #35

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

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Restore httpStatusCodetoString #35

merged 1 commit into from
Dec 9, 2024

Conversation

martialblog
Copy link
Member

@martialblog martialblog commented Dec 9, 2024

Is still in use. Must have missed it.

@martialblog martialblog merged commit f204a8d into main Dec 9, 2024
2 checks passed
@martialblog martialblog deleted the restore-http branch December 9, 2024 10:06
@martialblog
Copy link
Member Author

ok ok ok, had ANOTHER look at this. Now I know what's up.

I never saw the httpStatusCodetoString function in use, because it is used as httpStatusCodeToString:

    public static function httpStatusCodetoString($code = 0)
    {
        $statuscodes = array(
            $imageHtml .= $this->translate('Cannot fetch Grafana graph')
                . ": "
                . Util::httpStatusCodeToString($statusCode)
                . " "

🤦

@martialblog
Copy link
Member Author

Strange, I feel like this must have never worked in the first place? Mikesch-mp/icingaweb2-module-grafana@07afe21

@martialblog
Copy link
Member Author

Today I learned: n PHP, classes, functions and methods are case-insensitive

@RincewindsHat
Copy link
Member

Say what? Who thought that would be a good idea?

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.

2 participants