Skip to content

Conversation

@mikadamczyk
Copy link
Contributor

🎫 Issue N/A

Description:

  Line   src/bundle/Command/MigrateLegacyMatrixCommand.php                      
 ------ ----------------------------------------------------------------------- 
  105    Argument of an invalid type array<SimpleXMLElement>|null supplied for  
          foreach, only iterables are supported.                                
         🪪  foreach.nonIterable                                                
  181    Parameter #1 $cells of method                                          
         Ibexa\Bundle\FieldTypeMatrix\Command\MigrateLegacyMatrixCommand::conv  
         ertCellsToRows() expects array<int|string, mixed>, array<SimpleXMLEle  
         ment>|null given.                                                      
         🪪  argument.type                                                      
 ------ ----------------------------------------------------------------------- 

 ------ ----------------------------------------------------------------------- 
  Line   src/lib/FieldType/Mapper/MatrixFormMapper.php                          
 ------ ----------------------------------------------------------------------- 
  64     Access to protected property                                           
         Ibexa\Contracts\ContentForms\Data\Content\FieldData::$fieldDefinition  
         .                                                                      
         🪪  property.protected                                                 
 ------ ----------------------------------------------------------------------- 

For QA:

Documentation:

@mikadamczyk mikadamczyk self-assigned this Aug 25, 2025
@sonarqubecloud
Copy link

@mikadamczyk mikadamczyk requested a review from a team August 25, 2025 07:35
@ezrobot ezrobot requested review from Steveb-p, ViniTou, adamwojs, alongosz, barw4, ciastektk, konradoboza, tbialcz and wiewiurdp and removed request for a team August 25, 2025 07:35
@adamwojs adamwojs merged commit 580f3a6 into main Aug 25, 2025
22 checks passed
@adamwojs adamwojs deleted the fixed-phpstan branch August 25, 2025 08:32
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.

4 participants