Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

is_executable warning exception#26473

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
m3esma wants to merge1 commit intosymfony:masterfromm3esma:master
Closed

is_executable warning exception#26473

m3esma wants to merge1 commit intosymfony:masterfromm3esma:master

Conversation

@m3esma
Copy link

@m3esmam3esma commentedMar 9, 2018•
edited
Loading

QA
Branch?master
Bug fix?yes
New feature?no
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed ticketsn/a
LicenseMIT
Doc PRn/a

is_executable warning exception
The @ gets rid of the warning when this command fails.is_executable(): open_basedir restriction in effect

e_warning

The @ gets rid of the warning when this command fails. `is_executable(): open_basedir restriction in effect`
@fabpot
Copy link
Member

I'm 👎 on this one. We try to avoid adding@ on function calls whenever possible. Here, it looks like you are usingopen_basedir, which means you need to take care of that on your side.

@fabpotfabpot closed thisMar 10, 2018
@m3esma
Copy link
Author

if (basename($path) ==$name && @is_executable($path)) {

if (@is_file($file =$dir.DIRECTORY_SEPARATOR.$name.$suffix) && ('\\' ===DIRECTORY_SEPARATOR ||is_executable($file))) {

@fabpot
@ Very used in symfony!

@fabpot
Copy link
Member

I'm not saying we don't use@, but that we try to avoid using it whenever possible.

@m3esma
Copy link
Author

@fabpot
ok. thanks ♥, I'm trying to find a better way

@m3esmam3esma changed the titleFix is_executable warning exception[s] Fix is_executable warning exceptionMar 13, 2018
@m3esmam3esma changed the title[s] Fix is_executable warning exceptionis_executable warning exceptionMar 13, 2018
@pwronline
Copy link

@m3esma
Did you fix it? I have the same problem..

@m3esma
Copy link
Author

@pwronline
I personally solved it with this commit
6972e1e
And now everything is fine for me 😄

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@m3esma@fabpot@pwronline@xabbuh@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp