Jacob Dreesen comments

Results 183 comments of


                                            Jacob Dreesen

PHPStorm Plugin: "IllegalStateException: You must not register toolwindow programmatically so early"

Did you even look if someone already reported this? Duplicate of: #1805, #1820, #1824, #1828.

[False positive] JsonEncodingApiUsageInspection

And of #1639 and #1807

Incorrect "Operation on an array doesn't make much sense here"

Stumbled upon this, too. This is clearly a false positive, as the spaceship operator allows comparing arrays (as can be seen in [the RFC](https://wiki.php.net/rfc/combined-comparison-operator)).

UnsupportedStringOffsetOperationsInspection on array{...}[]

Not sure if `@var array{a: array, s: string}[]` is a valid syntax... if I change it to `@var array` I don't get the warning anymore.

Self class referencing inspection doesn't check return or parameter type

What exactly is the problem here? I don't get it...

False Positive: statement seems to be disconnected from its parent foreach

There's no warning if you do it like this, though: ```php function foo(array $param) { foreach ($param as ['field' => $field]) { echo $field; } } ```

IMPORTANT! "Return type hint can be used" suggests for libraries and php core classes

You can just add the return type. PHP allows this for sub-classes.

[RFC] TODOs for version 2.0

> ❗ Do we want to continue passing value, index, collection? > E.g. map suffers from the same problems JavaScript’s map suffers from > ```php >> ["0", "1", "2", "3",...

Hash size

Do you have some patience, please?

Too many collisions?

In PHP you could do `decbin(hexdec($hash))` or `base_convert($hash, 16, 2)` to convert the hex value into a binary value.