Skip to content

fix(adminer): php 8 warning of undefined variable #3590

fix(adminer): php 8 warning of undefined variable

fix(adminer): php 8 warning of undefined variable #3590

Triggered via pull request August 5, 2024 16:10
Status Success
Total duration 30s
Artifacts

phpcodesniffer.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
phpcs: webapp/resources/adminer.php#L9
Expected "function abc(...)"; found "function\nabc(...)"
phpcs: webapp/resources/adminer.php#L9
Expected 1 space after FUNCTION keyword; newline found
phpcs: webapp/resources/adminer.php#L10
Expected 1 space between comma and argument "$ec"; 0 found
phpcs: webapp/resources/adminer.php#L10
There must be a single space between the closing parenthesis and the opening brace of a multi-line function declaration; found 0 spaces
phpcs: webapp/resources/adminer.php#L10
No space found after comma in argument list
phpcs: webapp/resources/adminer.php#L10
Each PHP statement must be on a line by itself
phpcs: webapp/resources/adminer.php#L10
No space found after comma in argument list
phpcs: webapp/resources/adminer.php#L10
Each PHP statement must be on a line by itself
phpcs: webapp/resources/adminer.php#L10
No space found after comma in argument list
phpcs: webapp/resources/adminer.php#L10
Each PHP statement must be on a line by itself
phpcs
The following actions uses node12 which is deprecated and will be forced to run on node16: domjudge/action-php-codesniffer@master. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
phpcs
The following actions uses Node.js version which is deprecated and will be forced to run on node20: domjudge/action-php-codesniffer@master. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
phpcs: webapp/resources/adminer.php#L1
A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 9 and the first side effect is on line 10.
phpcs: webapp/resources/adminer.php#L180
Property name "$_result" should not be prefixed with an underscore to indicate visibility
phpcs: webapp/resources/adminer.php#L197
Property name "$_offset" should not be prefixed with an underscore to indicate visibility
phpcs: webapp/resources/adminer.php#L202
Property name "$_conn" should not be prefixed with an underscore to indicate visibility
phpcs: webapp/resources/adminer.php#L356
Property name "$_link" should not be prefixed with an underscore to indicate visibility
phpcs: webapp/resources/adminer.php#L356
Property name "$_result" should not be prefixed with an underscore to indicate visibility
phpcs: webapp/resources/adminer.php#L376
Property name "$_result" should not be prefixed with an underscore to indicate visibility
phpcs: webapp/resources/adminer.php#L376
Property name "$_offset" should not be prefixed with an underscore to indicate visibility