Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix earlyTerminatingMethodCalls syntax #173

Merged

Conversation

mundschenk-at
Copy link
Contributor

Not sure if that was a change with recent PHPStan versions or always incorrect, but these only work when I follow the syntax example from https://phpstan.org/writing-php-code/solving-undefined-variables

@szepeviktor
Copy link
Owner

A surprise PR! Thank you @mundschenk-at!

Could you add a test that fails without this PR?

@szepeviktor
Copy link
Owner

szepeviktor commented Apr 7, 2023

@dingo-d started it 3 years ago.

@dingo-d
Copy link
Contributor

dingo-d commented Apr 10, 2023

@dingo-d started it 3 years ago.

I did? 😅

@szepeviktor
Copy link
Owner

Thank you @mundschenk-at

@szepeviktor szepeviktor merged commit 5774598 into szepeviktor:master Apr 21, 2023
IanDelMar pushed a commit to IanDelMar/phpstan-wordpress that referenced this pull request Jul 27, 2023
szepeviktor added a commit that referenced this pull request Jul 27, 2023
* Remove has_filter extension

* Remove current_time extension

* Remove mysql2date extension

* Remove get_object_taxonomies extension

* Remove get_taxonomies extension

* Adapt get_post extension to new stub file

* Adapt get_comment extension to new stub file

* Fix CS

* Update composer.json

* Remove WP_Theme::get()

* Remove get_permalink extension

* Update .travis.yml

* Remove term_exists extension

* Update wp_error parameter extension

* Update GetPostDynamicFunctionReturnTypeExtension.php

* Fully remove wp_error parameter extension

* Merge get_comment extension into get_post extension

* Remove type specifiying extension and rule for `is_wp_error()`

* Update README.md

Co-authored-by: Viktor Szépe <[email protected]>

* Remove echo parameter extension

* Remove _get_list_table extension

* Revert "Remove _get_list_table extension"

This reverts commit 0191253.

* Update get_post.php

* Fix earlyTerminatingMethodCalls syntax (#173)

* Remove deprecated instanceof (#183)

* Remove deprecated `instanceof` (#184)

* Remove deprecated `instanceof`

* Remove space

Co-authored-by: Viktor Szépe <[email protected]>

* Return early

* Fix handling of fields unions

* Add tests

---------

Co-authored-by: Viktor Szépe <[email protected]>

* Remove deprecated `instanceof` (#185)

* Fix CS (#186)

* Fix CS

* Fix PHP 7.2 compat.

* Fix _get_list_table extension (#190)

* Revert "Update composer.json"

This reverts commit d15b7e1.

---------

Co-authored-by: Viktor Szépe <[email protected]>
Co-authored-by: Der Mundschenk & Compagnie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants