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

phpcs can't recognize tmp file on Windows 10 #1867

Open
lifecrisis opened this issue Aug 28, 2018 · 5 comments
Open

phpcs can't recognize tmp file on Windows 10 #1867

lifecrisis opened this issue Aug 28, 2018 · 5 comments
Labels

Comments

@lifecrisis
Copy link
Contributor

Information

VIM version

VIM - Vi IMproved 8.1 (2018 May 18, compiled May 18 2018 18:36:07)
MS-Windows 32-bit GUI version with OLE support

Operating System: Windows 10

:ALEInfo

Current Filetype: php
Available Linters: ['langserver', 'phan', 'php', 'phpcs', 'phpmd', 'phpstan']
Enabled Linters: ['php', 'phpcs']
Suggested Fixers:
'php_cs_fixer' - Fix PHP files with php-cs-fixer.
'phpcbf' - Fix PHP files with phpcbf.
'remove_trailing_lines' - Remove all blank lines at the end of a file.
'trim_whitespace' - Remove all trailing whitespace characters at the end of every line.
Linter Variables:

let g:ale_php_phpcs_executable = 'phpcs'
let g:ale_php_phpcs_options = ''
let g:ale_php_phpcs_standard = ''
let g:ale_php_phpcs_use_global = 0
Global Variables:

let g:ale_cache_executable_check_failures = v:null
let g:ale_change_sign_column_color = 0
let g:ale_command_wrapper = ''
let g:ale_completion_delay = v:null
let g:ale_completion_enabled = 0
let g:ale_completion_max_suggestions = v:null
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%code: %%s'
let g:ale_echo_msg_info_str = 'Info'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_history_enabled = 1
let g:ale_history_log_output = 1
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_filetype_changed = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_lint_on_insert_leave = 0
let g:ale_linter_aliases = {}
let g:ale_linters = {'php': ['phpcs', 'php']}
let g:ale_linters_explicit = 0
let g:ale_list_window_size = 10
let g:ale_list_vertical = 0
let g:ale_loclist_msg_format = '%code: %%s'
let g:ale_max_buffer_history_size = 20
let g:ale_max_signs = -1
let g:ale_maximum_file_size = v:null
let g:ale_open_list = 0
let g:ale_pattern_options = v:null
let g:ale_pattern_options_enabled = v:null
let g:ale_set_balloons = 1
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 0
let g:ale_sign_error = '>>'
let g:ale_sign_info = '--'
let g:ale_sign_offset = 1000000
let g:ale_sign_style_error = '>>'
let g:ale_sign_style_warning = '--'
let g:ale_sign_warning = '--'
let g:ale_statusline_format = v:null
let g:ale_type_map = {}
let g:ale_use_global_executables = v:null
let g:ale_warn_about_trailing_blank_lines = 1
let g:ale_warn_about_trailing_whitespace = 1
Command History:

(executable check - success) php
(finished - exit code 0) 'cmd /s/c "php -l -d error_reporting=E_ALL -d display_errors=1 -d log_errors=0 -- < C:\Users\JRFRAN~1\AppData\Local\Temp\VIA4F12.tmp\test.php"'

<<>>
No syntax errors detected in -
<<>>

(executable check - success) phpcs
(finished - exit code 3) 'cmd /s/c "phpcs -s --report=emacs --stdin-path=C:\Users\jrfranklin\Downloads\test.php < C:\Users\JRFRAN~1\AppData\Local\Temp\VIC4F24.tmp\test.php"'

<<>>
ERROR: You must supply at least one file or directory to process.

Run "phpcs --help" for usage information

<<>>

What went wrong

The phpcs command doesn't seem to be able to read the tmp file passed to it by ALE.

Reproducing the bug

  1. Use only phpcs and php (or just phpcs) to lint any php file on Windows 10.

  2. The tmp file is not recognized.

@w0rp
Copy link
Member

w0rp commented Aug 28, 2018

Which version of phpcs are you using?

@w0rp w0rp added the triage label Aug 28, 2018
@lifecrisis
Copy link
Contributor Author

PHP_CodeSniffer version 3.3.1 (stable) by Squiz (http://www.squiz.net)

@w0rp
Copy link
Member

w0rp commented Aug 28, 2018

I don't see why I wouldn't work. Maybe someone can have a look at this when they have the time.

@w0rp
Copy link
Member

w0rp commented Jun 10, 2019

This and #2398 are related.

@char101
Copy link

char101 commented Apr 15, 2020

Without specifying - as file input, phpcs won't use stdin (PHP 7.4, phpcs 3.5.4, Windows).

Temporary fix:

let g:ale_php_phpcs_options = '-'

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

No branches or pull requests

3 participants