Phpinspectionsea: [false-positive] Can not iterate 'ø' (re‑check type hints)

Created on 6 Dec 2016 · 19Comments · Source: kalessil/phpinspectionsea

Code example

$array = array_filter([]);
foreach ($array as $item) {

}

bug / false-positive fixed

Source

nightlinus

All 19 comments

Confirm this bug.

funivan on 6 Dec 2016

Which PhpStrom/Idea version is it?

kalessil on 6 Dec 2016

PhpStorm 2016.3.1 EAP
Build #PS-163.9166.11, built on December 1, 2016
JRE: 1.8.0_112-release-408-b2 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o

Php Inspections (EA Extended)
Version: 2.2.0

nightlinus on 6 Dec 2016

I guess I have to hurry up with fixing this, until it not released...

kalessil on 7 Dec 2016

I guess this is sarcasm. I am sorry for disturbing, wont open issues for eap versions.
Checked on stable version — no false-positive.

nightlinus on 7 Dec 2016

Not at all, I was serious - feedback on EAPs is very valuable to me. Sorry for confusion.

kalessil on 7 Dec 2016

Thank you for reporting @nightlinus , I'm releasing a hotfix version of the plugin today.

kalessil on 8 Dec 2016

Ty for quick fix, but unfortunetly it is not resolved.

PhpStorm 2016.3.1
Build #PS-163.9735.1, built on December 6, 2016
JRE: 1.8.0_112-release-408-b2 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Php Inspections (EA Extended) Version: 2.2.1

Thank you for your work this is one of the greatest plugins ever, so much value.

nightlinus on 9 Dec 2016

@artspb : any hints about the mysterious 'ø' and where it coming from (type resolving)?

kalessil on 9 Dec 2016

@neuro159 should be aware of it, please ask him directly.

artspb on 13 Dec 2016

😄1

this is indeed my issue, in parameter-based type extension, will handle it for next week's EAP.

neuro159 on 13 Dec 2016

👍1

@artspb : sure, will address all similar topics to Alex ;)
@neuro159 : great, thank you.

kalessil on 13 Dec 2016

@neuro159 still present

PhpStorm 2017.1 EAP
Build #PS-171.2152.2, built on January 4, 2017
PhpStorm EAP User
Expiration date: February 3, 2017
JRE: 1.8.0_112-release-b621 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o

b1rdex on 9 Jan 2017

I know. In my short list...

On Mon, 9 Jan 2017, 04:09 Anatoly Pashin, notifications@github.com wrote:

@neuro159 https://github.com/neuro159 still present

PhpStorm 2017.1 EAP
Build #PS-171.2152.2, built on January 4, 2017
PhpStorm EAP User
Expiration date: February 3, 2017
JRE: 1.8.0_112-release-b621 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kalessil/phpinspectionsea/issues/65#issuecomment-271204227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFVDFmLO9gTNOStH8i6Cd3p81_xSmteks5rQaTvgaJpZM4LFG_p
.

neuro159 on 9 Jan 2017

@b1rdex @neuro159 : is this still present?

PS: I'd then drop my workaround finally )

kalessil on 9 Feb 2017

@kalessil got this bug today

PhpStorm 2017.1 EAP
Build #PS-171.3224.10, built on February 15, 2017
PhpStorm EAP User
Expiration date: March 17, 2017
JRE: 1.8.0_112-release-b702 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0