Phpinspectionsea: Reports the usage of 'at' operator

[rentalhost]

It should be avoided as much as possible, because you can create some unfindable issues on your code. For instance: if used on a method that throws an exception, it will can't be treated.

// Warning: avoid at operador.
@my_error();

But there are some fews exceptions to at operator, when it handles some race conditions like by using unlink(). Then in this case, it should be allowed (I guess that it can happen with all file operations functions).

[funivan]

I can try to implement this feature =)

[kalessil]

Go ahead @funivan =)

[funivan]

There is standard inspection "Usage of a silence operator" =) But this inspection does not have any options.
So there are 2 cases.

1. Add configurable inspection to the plugin (allow to modify functions list, or hardcode functions that can be used with @ operator)
2. Create issue https://youtrack.jetbrains.com/issues/WI

Functions that can be used with @ operator:

@mkdir()
@trigger_error()
@unlink()

This one is from symfony

@ldap_add()
@ldap_bind()
@ldap_connect()
@ldap_delete()
@ldap_get_entries()
@ldap_get_option()

@posix_isatty
@posix_isatty()
@posix_kill()
// ... 

[kalessil]

Let's add our own inspection - less headache for everyone =)

[funivan]

Ok)

[kalessil]

@funivan : FYI, I added content-aware behaviour and corresponding documentation.

If the content awareness settings switched off, the inspection behaves as in original version.

[funivan]

@kalessil nice documentation. Simple inspection become smarter. Nice job ;)

[kalessil]

Thank you @funivan, it was a good basis there ;)