Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Remove permission denied warning if unable write config to cache #48

Conversation

freax
Copy link

@freax freax commented Aug 17, 2016

Remove warning "Warning: file_put_contents(data/cache/module-config-cache.application.config.cache.php): failed to open stream: Permission denied in ...".

@samsonasik
Copy link
Contributor

I think it is hiding issue. I think we may have throw an Exception for this.

@weierophinney
Copy link
Member

This is fixed with #39 and #53, which add the LOCK_EX flag as a third argument to file_put_contents(), ensuring the issue should not arise. (The issue typically occurs during race conditions from initial load.)

If you are seeing this for some other reason, please re-submit the pull request. However, as @samsonasik notes, you should raise an exception instead.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants