coala-bears icon indicating copy to clipboard operation
coala-bears copied to clipboard

Bear Proposal: PHPLint

Open ishanSrt opened this issue 6 years ago • 5 comments

According to http://www.icosaedro.it/phplint/phplint2/doc/tutorial.htm This is wrong code:

if( ! $f = fopen("myFile.txt", "r") ){
    die("error opening the file");
}

Can't get PHPLintBear to detect it.

Infact can't get PHPLintBear to detect any error in my code

ishanSrt avatar Mar 29 '18 18:03 ishanSrt

GitMate.io thinks possibly related issues are https://github.com/coala/coala-bears/issues/108 (JSHintBearTest not working properly.), https://github.com/coala/coala-bears/issues/1542 (Markdownbear isn't working as intended), https://github.com/coala/coala-bears/issues/593 ([ESLint] Empty files does not work as expected), https://github.com/coala/coala-bears/issues/572 (ESLintBear doesn't work), and https://github.com/coala/coala-bears/issues/2065 (CheckStyleBear is not Working).

gitmate-bot avatar Mar 29 '18 18:03 gitmate-bot

@ishanSrt I like to work on this issue.

The above code you mentioned is in php and I can't find any php file in coala-bears with that specific part of code in it. So, is there any seperate source code to it?

rammanoj avatar Apr 03 '18 01:04 rammanoj

@rammanoj this is just a sample. What's important is that the bear doesn't seem to work according to @ishanSrt , so that needs investigation and confirmation.

Makman2 avatar Apr 03 '18 19:04 Makman2

PHPLintBear is php -l, not related to http://www.icosaedro.it/phplint/

Adding phplint is a great idea.

jayvdb avatar Jul 01 '19 03:07 jayvdb

Potentially the current PHPLintBear could be renamed to something else, so that PHPLint can use the name PHPLintBear.

PHPLint is also called phpl so that also needs to be avoided if the current bear is being renamed. PHPCLILBear ? PHPInternalLintBear?

jayvdb avatar Jul 01 '19 03:07 jayvdb