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

Jump to bottom

Notice: tempnam() on PHP 7.1 #146

Closed
firstred opened this issue Mar 17, 2017 · 4 comments
Closed

Notice: tempnam() on PHP 7.1 #146

firstred opened this issue Mar 17, 2017 · 4 comments
Labels
Bug
Milestone
1.0.1

Comments

@firstred
Copy link
Contributor

Notice: tempnam(): file created in the system's temporary directory in /var/www/default/classes/PrestaShopAutoload.php on line 125
@firstred firstred added this to the 1.0.1 milestone Mar 17, 2017
@devnix
Copy link

devnix commented Mar 17, 2017

Shouldn't be fixed by configuring php.ini correctly?

@firstred
Copy link
Contributor Author

Not everyone is capable of configuring php.ini correctly. There might be a better way.

@devnix
Copy link

devnix commented Mar 18, 2017

Maybe, but if you are using a shared hosting account of a provider, it should be properly configured (even if you don't have enough capabilities to do it).

As I see it, the best approach would be to use an alternative function/system of tempnam. Using a @ to mute the warning is considered a very bad practice.

@firstred
Copy link
Contributor Author

Yeah it should be fixed via the configuration indeed. Thanks for the update!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
1.0.1
Development

No branches or pull requests
2 participants
@devnix @firstred