View GitHub Repository Bookdown.Bookdown

Schedule Inspection
Subscribe

Service::__invoke()   A
last analyzed

↳ Parent: Service

Complexity

Conditions 1
Paths 1

Size

Total Lines 8
Code Lines 6

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 7
CRAP Score 1

Importance

Changes 4
Bugs 1 Features 2
Metric Value
c 4
b 1
f 2
dl 0
loc 8
ccs 7
cts 7
cp 1
rs 9.4285
cc 1
eloc 6
nc 1
nop 2
crap 1
1 
<?php
2 
namespace Bookdown\Bookdown\Service;
3 

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    4
                

                                    
                                                    
                
                
                
class Service

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    5
                

                                    
                                                    
                
                
                
{

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    6
                

                                    
                                                    
                
                
                
    protected $collector;

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    7
                

                                    
                                                    
                
                
                
    protected $processorBuilder;

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    8
                

                                    
                                                    
                
                
                
    protected $timer;

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    9
                

                                    
                                                    
                
                
                

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    10
                

                                    
                             3                         
                
                
                
    public function __construct(

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    11
                

                                    
                                                    
                
                
                
        Collector $collector,

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    12
                

                                    
                                                    
                
                
                
        ProcessorBuilder $processorBuilder,

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    13
                

                                    
                                                    
                
                
                
        Timer $timer

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    14
                

                                    
                                                    
                
                
                
    ) {

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    15
                

                                    
                             3                         
                
                
                
        $this->collector = $collector;

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    16
                

                                    
                             3                         
                
                
                
        $this->processorBuilder = $processorBuilder;

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    17
                

                                    
                             3                         
                
                
                
        $this->timer = $timer;

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    18
                

                                    
                             3                         
                
                
                
    }

            


            
                                                                                                            
                                                                
            
                                    
            
            

                
                    19
                

                                    
                                                    
                
                
                

            


            
                                                        
            
                                    
            
            

                
                    20
                

                                    
                             1                         
                
                
                
    public function __invoke($rootConfigFile, array $rootConfigOverrides)

            


            
                                                        
            
                                    
            
            

                
                    21
                

                                    
                                                    
                
                
                
    {

            


            
                                                        
            
                                    
            
            

                
                    22
                

                                    
                             1                         
                
                
                
        $this->collector->setRootConfigOverrides($rootConfigOverrides);

            


            
                                                        
            
                                    
            
            

                
                    23
                

                                    
                             1                         
                
                
                
        $rootPage = $this->collector->__invoke($rootConfigFile);

            


            
                                                        
            
                                    
            
            

                
                    24
                

                                    
                             1                         
                
                
                
        $processor = $this->processorBuilder->newProcessor($rootPage->getConfig());

            


            
                                                        
            
                                    
            
            

                
                    25
                

                                    
                             1                         
                
                
                
        $processor->__invoke($rootPage);

            


                            

                    
                        
                    
                    
                    
                    
                                                                                                    
                        

                            
                            
0 ignored issues

                            –
                            show
                        


                        

                                                                                        




    

        
            Compatibility
    
    
    
        introduced 
                            by 
            

        

        

            
        

        

            

                

                    
                
                
            


        

    

    

        
                        

                            
                                
                            
                        

                                        
$rootPage of type object<Bookdown\Bookdown\Content\Page> is not a sub-type of object<Bookdown\Bookdown\Content\RootPage>. It seems like you assume a child class of the class Bookdown\Bookdown\Content\Page to be always present.
                                                                                                                                                                                                                                                                                                                                                                    

                    
                                            

                            

                            

                                
                                    
                                
                            


                            
This check looks for parameters that are defined as one type in their type
hint or doc comment but seem to be used as a narrower type, i.e an
implementation of an interface or a subclass.


Consider changing the type of the parameter or doing an instanceof check before assuming
your parameter is of the expected type.


                        

                                                    

                     Loading history...
                

                    


    




                                                                                    


                    		
                

            
                                                        
            
                                    
            
            

                
                    26
                

                                    
                             1                         
                
                
                
        $this->timer->report();

            


            
                                                        
            
                                    
            
            

                
                    27
                

                                    
                             1                         
                
                
                
    }

            


            
                                                        
            
                                    
            
            

                
                    28
                

                                    
                                                    
                
                
                
}

            


            
                                                        
            
                                    
            
            

                
                    29