Allow print_r(..., true) about phpstan-disallowed-calls HOT 4 CLOSED

Thinking out loud here (may server as a note to self or to anyone willing to implement it) but something this is what I'd like to see:

The allowedParams here, with type and/or value keys (both may be extraneous, so maybe type or value, or maybe neither, optionally, would need to think about it a bit more):

        arguments:
            forbiddenCalls:
                -
                    function: 'print_r()'
                    message: 'use logger instead'
                    allowedParams:
                      2:
                        type: boolean
                        value: true

from phpstan-disallowed-calls.

Works perfect for me. Thanks.

from phpstan-disallowed-calls.

Hi, thanks. I'd love to allow some "forbidden calls" with specified parameters but I'd like it to be more generic and configurable.

For example allowReturn only makes sense with print_r() and var_export() but not with var_dump() for example.

What I do now, until support for params is added, I just simply ignore the error:

ignoreErrors:
	-
		message: '#^Calling print_r\(\) is forbidden, use logger instead#'  # Used with $return = true
		paths:
			- file.php
			- foo.php

To not have to ignore the error in many files, a logger that would accept any type and print_r(..., true)-it before actually logging it could also be used.

from phpstan-disallowed-calls.

I've added allowParams support in #4, let me know if it works for you (it does for me), thanks!

from phpstan-disallowed-calls.