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

TASK: Migrate Neos.Fusion/Core to phpstan level 8 (Neos 8.3) #4844

Merged
merged 1 commit into from
Feb 2, 2024

Commits on Jan 24, 2024

  1. TASK: Migrate Neos.Fusion/Core to phpstan level 8 (Neos 8.3)

    Only files in `Neos.Fusion/Classes/Core` have been migrated, as they contain relatively new code like the parser.
    
    Two kind of errors have been partially ignored from the level 8
    
    For one: "no value type specified in iterable type array" as adding `array<int|string, mixed>` would just bloat this commit up. In these files we are mostly dealing with the fusion configuration which is highly dynamic:
    
    - Neos.Fusion/Classes/Core/Runtime.php
    - Neos.Fusion/Classes/Core/Cache/RuntimeContentCache.php
    - Neos.Fusion/Classes/Core/Cache/ContentCache.php
    - Neos.Fusion/Classes/Core/Cache/CacheSegmentParser.php
    - Neos.Fusion/Classes/Core/RuntimeConfiguration.php
    - Neos.Fusion/Classes/Core/ObjectTreeParser/MergedArrayTreeVisitor.php
    - Neos.Fusion/Classes/Core/ObjectTreeParser/MergedArrayTree.php
    
    And "has no return type specified" was noticed a lot in the ast visiting code. But i plan to refactor this instead to something better type able:
    
    - Neos.Fusion/Classes/Core/ObjectTreeParser/MergedArrayTreeVisitor.php
    - Neos.Fusion/Classes/Core/ObjectTreeParser/AstNodeVisitorInterface.php
    - Neos.Fusion/Classes/Core/ObjectTreeParser/Ast
    mhsdesign committed Jan 24, 2024
    Configuration menu
    Copy the full SHA
    d0377b9 View commit details
    Browse the repository at this point in the history