Respect enable_required_directories_diagnostic option in server info diagnostics #19137

Description:

I found another location where the accessibility of the global.ini.php is checked despite the enable_required_directories_diagnostic option is disabled in the config (the other case was fixed in #19031).
I just copied the changes over from #19031 to here.

Fixes #18967

Review

• [ ] Functional review done
• [ ] Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• [ ] Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• [ ] Security review done
• [ ] Wording review done
• [ ] Code review done
• [ ] Tests were added if useful/possible
• [ ] Reviewed for breaking changes
• [ ] Developer changelog updated if needed
• [ ] Documentation added if needed
• [ ] Existing documentation updated if needed

Hi @HCl-not-HCi, thanks for submitting this PR, much appreciated! :slightly_smiling_face:

The change looks good to me, the only minor thing would be adding a use statement for the GeneralConfig class:

use Piwik\Config\GeneralConfig;

If you get a chance to commit that then I'll get this merged into the development branch :+1:

Oh, I've overseen that. Thanks for letting me know @bx80
Now it should be ready to merge 👍