PHPStand level 6 fixes

Created on 18 July 2023, about 1 year ago
Updated 19 July 2023, about 1 year ago

Problem/Motivation

Attached is a report from PHPStan running on level 6 with code to be fixed.

📌 Task
Status

Fixed

Version

1.2

Component

Code

Created by

🇩🇪Germany jurgenhaas Gottmadingen

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @jurgenhaas
  • Open in Jenkins → Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.1 & MySQL 8
    last update about 1 year ago
    294 pass, 2 fail
  • @jurgenhaas opened merge request.
  • Open in Jenkins → Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.1 & MySQL 8
    last update about 1 year ago
    296 pass
  • Open in Jenkins → Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.1 & MySQL 8
    last update about 1 year ago
    296 pass
  • Issue was unassigned.
  • Status changed to Needs review about 1 year ago
  • 🇩🇪Germany jurgenhaas Gottmadingen

    This is ready for a review, although we still have some issues reported by PhpStan:

     ------ --------------------------------------------------------------------------------------------------------------------------------------
      Line   modules/form/eca_form.module
     ------ --------------------------------------------------------------------------------------------------------------------------------------
      40     Function eca_form_field_widget_single_element_inline_entity_form_complex_form_alter() has parameter $context with no type specified.
      40     Function eca_form_field_widget_single_element_inline_entity_form_complex_form_alter() has parameter $element with no type specified.
     ------ --------------------------------------------------------------------------------------------------------------------------------------
    
     ------ -----------------------------------------------------------------------------------------------------------------------
      Line   modules/render/src/Plugin/views/field/EcaRender.php
     ------ -----------------------------------------------------------------------------------------------------------------------
      63     Method Drupal\eca_render\Plugin\views\field\EcaRender::buildOptionsForm() has parameter $form with no type specified.
     ------ -----------------------------------------------------------------------------------------------------------------------
    
     ------ -------------------------------------------------------------------------------------------------------------------------
      Line   src/Plugin/DataType/DataTransferObject.php
     ------ -------------------------------------------------------------------------------------------------------------------------
      413    Method Drupal\eca\Plugin\DataType\DataTransferObject::writePropertyValue() has parameter $value with no type specified.
     ------ -------------------------------------------------------------------------------------------------------------------------
    
     ------ ---------------------------------------------------------------------------------------------------------------
      Line   src/PluginManager/Action.php
     ------ ---------------------------------------------------------------------------------------------------------------
      95     Method Drupal\eca\PluginManager\Action::setCacheBackend() has no return type specified.
      132    Method Drupal\eca\PluginManager\Action::clearCachedDefinitions() has no return type specified.
      140    Method Drupal\eca\PluginManager\Action::useCaches() has no return type specified.
      148    Method Drupal\eca\PluginManager\Action::processDefinition() has no return type specified.
      148    Method Drupal\eca\PluginManager\Action::processDefinition() has parameter $definition with no type specified.
      148    Method Drupal\eca\PluginManager\Action::processDefinition() has parameter $plugin_id with no type specified.
     ------ ---------------------------------------------------------------------------------------------------------------
    
     ------ --------------------------------------------------------------------------------------------------------
      Line   src/Token/ContribToken.php
     ------ --------------------------------------------------------------------------------------------------------
      34     Method Drupal\eca\Token\ContribToken::getTypeInfo() has parameter $token_type with no type specified.
      41     Method Drupal\eca\Token\ContribToken::getTokenInfo() has parameter $token with no type specified.
      41     Method Drupal\eca\Token\ContribToken::getTokenInfo() has parameter $token_type with no type specified.
     ------ --------------------------------------------------------------------------------------------------------
    

    Those can't be corrected since their interface's declarations are similar to the current usage and hence, we can't change them. Guess, this has to be reported to PhpStan as a bug.

  • Open in Jenkins → Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.1 & MySQL 8
    last update about 1 year ago
    296 pass
  • Open in Jenkins → Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.1 & MySQL 8
    last update about 1 year ago
    296 pass
  • Status changed to RTBC about 1 year ago
  • 🇩🇪Germany jurgenhaas Gottmadingen

    We've looked at this at #ddd2023 and since all tests are green, let's go for it. Can then also be the basis for the new 2.0 branch.

  • Status changed to Fixed about 1 year ago
  • 🇩🇪Germany jurgenhaas Gottmadingen
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024