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.

Sign up for GitHub

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

Jump to bottom

fix: Do not shorten FQN for class resolution if imported symbol is not a class #7705

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

fix: Do not shorten FQN for class resolution if imported symbol is not a class #7705

wants to merge 4 commits into from
+149 −76

Commits on May 30, 2024

  1. Add failing test case

    @Wirone
    Wirone committed May 30, 2024
    Configuration menu
    Copy the full SHA
    3c352a6 View commit details
    Browse the repository at this point in the history

  2. Prepare FullyQualifiedStrictTypesFixer for supporting all import kinds 

    Previously all imports in the processed namespace were flattened to one list and use later for shortening. We need to have imports list sliced by import kind, so we can use proper imports in the context of currently processed part of code (e.g. use only class imports when processing function arguments).
    @Wirone
    Wirone committed May 30, 2024
    Configuration menu
    Copy the full SHA
    914995f View commit details
    Browse the repository at this point in the history

  3. Align the code to the current PHPStan rules

    @Wirone
    Wirone committed May 30, 2024
    Configuration menu
    Copy the full SHA
    2811185 View commit details
    Browse the repository at this point in the history

  4. Fix "PHP 8.3 Fixer with migration rules" job

    @Wirone
    Wirone committed May 30, 2024
    Configuration menu
    Copy the full SHA
    375464b View commit details
    Browse the repository at this point in the history