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.

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

Try to create config directories if non existant #3115

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Try to create config directories if non existant #3115

wants to merge 1 commit into from

Commits on Jan 23, 2023

  1. Try to create config directories if non existant 

    The patch will allow dompdf to try creating tmpDir, fontDir and fontCache if they do not exist on the filesystem - and will trigger a warning if they can't be created.

Even if the Options.php file mention that pathes should exist, a number of people will set the options and won't even see that the folders weren't created.

Trying to create them by default will : 
- Avoid bugs / lack of performances when directories are created
- Will warn the user that his configurations are not suitable if necessary.

Later, it might also be possible to change the trigger_error to an exception, but right now, it would be a BC break.
    @Blaklis
    Blaklis committed Jan 23, 2023
    Configuration menu
    Copy the full SHA
    72b3b07 View commit details
    Browse the repository at this point in the history