cancel
Showing results for 
Search instead for 
Did you mean: 

ECQ - Advice Required from the exeprts

ECQ - Advice Required from the exeprts

Hello,

 

I have finally completed the development of an extension that will boost the migration process to Magento. It's related to ranked URLs etc...

 

Now the issue I'm facing is with the ECQ it found few errors and lots of warnings, so I would appreciate any advice in this regard of the best approach.

 

I'll attach the report here:

 

[root@host public_html]# php phpcs.phar --standard=EcgM2 /home/**********/public_html/**********/

FILE: /home/**********/public_html/**********/Console/Command/RegenerateUrlCommand.php
---------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
---------------------------------------------------------------------------------------
 34 | WARNING | The method parameter $state is never used
 55 | WARNING | The method parameter $inp is never used
 73 | WARNING | Model LSD method save() detected in loop
---------------------------------------------------------------------------------------


FILE: /home/**********/public_html/**********/Console/Command/RegenerateCategoryUrlCommand.php
-----------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 4 WARNINGS AFFECTING 4 LINES
-----------------------------------------------------------------------------------------------
  63 | WARNING | The method parameter $inp is never used
  63 | WARNING | The method parameter $out is never used
  81 | WARNING | Possible slow SQL method group detected
  83 | WARNING | fetchAll() can be memory inefficient for large data sets.
 101 | ERROR   | Empty CATCH statement detected
-----------------------------------------------------------------------------------------------


FILE: /home/**********/public_html/**********/Console/Command/RegenerateProductUrlCommand.php
----------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 4 WARNINGS AFFECTING 4 LINES
----------------------------------------------------------------------------------------------
 58 | WARNING | The method parameter $inp is never used
 58 | WARNING | The method parameter $out is never used
 79 | WARNING | Possible slow SQL method group detected
 81 | WARNING | fetchAll() can be memory inefficient for large data sets.
 98 | ERROR   | Empty CATCH statement detected
----------------------------------------------------------------------------------------------


FILE: /home/**********/public_html/**********/Model/Category/Plugin/Storage.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 17 | WARNING | The method parameter $object is never used
--------------------------------------------------------------------------------


FILE: /home/**********/public_html/**********/Model/ProductUrlRewriteGenerator.php
-----------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------
 33 | WARNING | The method parameter $productCategories is never used
-----------------------------------------------------------------------------------


FILE: /home/**********/public_html/**********/Model/Storage/DbStorage.php
--------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------------
 73 | WARNING | fetchAll() can be memory inefficient for large data sets.
 98 | ERROR   | Comment refers to a FIXME task "Get rid of rewrite for root Magento category to unduplicate things"
--------------------------------------------------------------------------------------------------------------------


FILE: /home/**********/public_html/**********/Setup/InstallData.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
 30 | WARNING | The method parameter $context is never used
----------------------------------------------------------------------

Looking forward to hearing from you all.

 

Thanks in advance.