[Gravity/Sphere/Vortex] Fixes for PHP 7.2.

Review Request #537 — Created Dec. 12, 2017 and submitted

Information
pprkut
Lunr
2cbb149...
Reviewers
lunr 
Apparently, PHP 5.6 not having TypeError is not a problem, since the respective code is never executed.
No clue what the deal with empty() not working for $this->configuration['db']['ro_host'] is. For some reason empty() makes it return NULL.
Unit tests on 7.2 and 5.6
  • 0
  • 0
  • 0
  • 2
  • 2
Description From Last Updated
There are no open issues
smillernl
smillernl 
  1. 
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 

   
    
    
    



    
 
    
  
    
   
   
    The issue has been dropped.
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    can't this be fixed with a mock to offsetExists? Not sure but I remember issues with offsetGet vs offsetExists before.
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      that's already there
      
 
      

      2. 


 
    


    

 
    

    3. 






  3. 
 
    
  
  
    
  
 

   
    
    
    



    
 
    
  
    
   
   
    The issue has been dropped.
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    Doesn't this test something different?
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      Yeah, the test was broken. This tests a function called in the constructor, but if you call it again without the same pre-conditions it returns a different result.
      
The function is supposed to be void in a success case, it was doing a count(FALSE) before...
      
 
      

      2. 


 
    


    

 
    

    4. 




  4. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    5. 




  

 
















 





 

  

   




smillernl




   

   







  

 



 

  


   

   

    
 
smillernl


    
   


  

  

  



    




  1. 
 
    
  
    
   
    Ship It!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  

 
















 



 

  

   




pprkut




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 




 
Change Summary:

 
Merged into master





 


 










  

 









     

    

   

  

  

   
   Loading...