-
Notifications
You must be signed in to change notification settings - Fork 40
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
fputcsv() flagged when writing to stream #404
Comments
Does this still happen with VIPCS 0.4.0 (latest published version)? How about |
|
How about |
@GaryJones Is there a specific commit or PR you think addresses this or a related problem? Or are you just enjoying having me randomly regress things? Yes, the problem still occurs with |
Thank you. Background notes:
|
Due to the nature of static analysis, I think that that example would still flagged it because of @jrfnl What are your thoughts for this one? |
@rebeccahum I agree. Just based on the call to With the code snippet above, the sniff could walk back to find how We know it will be a resource handle, but that's it. Presuming it is a resource handle to |
Bug Description
The below code creates a CSV for download. PHPCS flags this as an error. I'm writing to a stream, not the filesystem.
Minimal Code Snippet
Error Code
Environment
The text was updated successfully, but these errors were encountered: