search

blmage / mage-enhanced-admin-grids

[ARCHIVED] Enhanced Admin Grids extension for Magento 1. WIP version available for testing.
248 stars 115 forks source link

Paymenth method #199

Open simulatorhck opened 9 years ago

simulatorhck commented 9 years ago

Hi I selected this column from optionals but nothing displayed in order grid. Someone make it run? It gave a block error

mage-eag commented 9 years ago

Sorry for the really late answer !

If you get this error : "The block is not valid", it means that the grid block on which you applied the column is currently rewrited by another extension, and that its compatibility with the payment method column (eg) can not be assured.

Starting with version 1.0.0 (wip), this error will be by default turned into a warning, that will not prevent anymore the column from being applied to the block. In the waiting, you can edit the app/code/community/BL/CustomGrid/Helper/Grid.php file and make the verifyGridBlock() method always return true.

The same goes for this message : "The block collection is not valid", in which case you would have to modify the verifyGridCollection() method in the same file as above.

simulatorhck commented 9 years ago

Hi. Not problem for the delay in replying i know that you have many things to do and not so much time for. I read the to-do list for release 1.0.0. Many points so i think it will pass some time to be on magrnto connect. I ll try to use the ancient version and to make changes but i m not a programmer so...good luck to me :P... Waiting for next version Bye Danilo Pizzini Italy Il 05/ago/2015 01:43 PM, "Benoît" notifications@github.com ha scritto:

Sorry for the really late answer !

If you get this error : "The block is not valid", it means that the grid block on which you applied the column is currently rewrited by another extension, and that its compatibility with the payment method column (eg) can not be assured.

Starting with version 1.0.0 (wip), this error will be by default turned into a warning, that will not prevent anymore the column from being applied to the block. In the waiting, you can edit the app/code/community/BL/CustomGrid/Helper/Grid.php https://github.com/mage-eag/mage-enhanced-admin-grids/blob/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php file and make the verifyGridBlock() https://github.com/mage-eag/mage-enhanced-admin-grids/blob/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php#L91 method always return true.

The same goes for this message : "The block collection is not valid", in which case you would have to modify the verifyGridCollection() https://github.com/mage-eag/mage-enhanced-admin-grids/blob/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php#L100 method in the same file as above.

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-127968398 .

simulatorhck commented 9 years ago

I tryed in this eay but I recrive blank page.. Thank you Il 05/ago/2015 04:02 PM, "Danilo Pizzini" danilo.pizzini@gmail.com ha scritto:

Hi. Not problem for the delay in replying i know that you have many things to do and not so much time for. I read the to-do list for release 1.0.0. Many points so i think it will pass some time to be on magrnto connect. I ll try to use the ancient version and to make changes but i m not a programmer so...good luck to me :P... Waiting for next version Bye Danilo Pizzini Italy Il 05/ago/2015 01:43 PM, "Benoît" notifications@github.com ha scritto:

Sorry for the really late answer !

If you get this error : "The block is not valid", it means that the grid block on which you applied the column is currently rewrited by another extension, and that its compatibility with the payment method column (eg) can not be assured.

Starting with version 1.0.0 (wip), this error will be by default turned into a warning, that will not prevent anymore the column from being applied to the block. In the waiting, you can edit the app/code/community/BL/CustomGrid/Helper/Grid.php https://github.com/mage-eag/mage-enhanced-admin-grids/blob/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php file and make the verifyGridBlock() https://github.com/mage-eag/mage-enhanced-admin-grids/blob/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php#L91 method always return true.

The same goes for this message : "The block collection is not valid", in which case you would have to modify the verifyGridCollection() https://github.com/mage-eag/mage-enhanced-admin-grids/blob/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php#L100 method in the same file as above.

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-127968398 .

mage-eag commented 9 years ago

Could you fix it with a backup file ? Otherwise, you can download it again from here : https://raw.githubusercontent.com/mage-eag/mage-enhanced-admin-grids/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php.

Also, I've created a gist with the updated helper code, that unlike the original does not actually verify grid blocks and collections : https://gist.github.com/mage-eag/01569bf6af84f695b798. Please use it with caution, especially if you do not have a technical provider.

simulatorhck commented 9 years ago

Yes i can restore. But alsi with your grid is the same. Blqnk page grid. It ciuld be a fode from somewherre thatcauses it? Thanks Il 06/ago/2015 03:10 PM, "Benoît" notifications@github.com ha scritto:

Could you fix it with a backup file ? Otherwise, you can download it again from here : https://raw.githubusercontent.com/mage-eag/mage-enhanced-admin-grids/0.9.0/app/code/community/BL/CustomGrid/Helper/Grid.php .

Also, I've created a gist with the updated helper code, that unlike the original does not actually verify grid blocks and collections : https://gist.github.com/mage-eag/01569bf6af84f695b798. Please use it with caution, especially if you do not have a technical provider.

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-128357953 .

mage-eag commented 9 years ago

Which version of the extension are you using ? (I maybe have wrongly assumed that it was 0.9.0)

simulatorhck commented 9 years ago

Yes you are tight. It can depends from other modules or settings? It s frusttrating Il 10/ago/2015 01:35 PM, "Benoît" notifications@github.com ha scritto:

Which version of the extension are you using ? (I maybe have wrongly assumed that it was 0.9.0)

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-129414807 .

mage-eag commented 9 years ago

Do you have access to your server error logs ? The reason for the blank pages has certainly been logged there, so it could be really helpful if you could paste it here. Otherwise, do you have a test environment aside from your production website ?

simulatorhck commented 9 years ago

I think i can activate error log drom magento Isntit in system conf development? And put your grid .php So we can see if thete is some way to solve Il 10/ago/2015 01:55 PM, "Benoît" notifications@github.com ha scritto:

Do you have access to your server error logs ? The reason for the blank pages has certainly been logged there, so it could be really helpful if you could paste it here. Otherwise, do you have a test environment aside from your production website ?

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-129420709 .

simulatorhck commented 9 years ago

Here you are Il 10/ago/2015 01:59 PM, "Danilo Pizzini" danilo.pizzini@gmail.com ha scritto:

I think i can activate error log drom magento Isntit in system conf development? And put your grid .php So we can see if thete is some way to solve Il 10/ago/2015 01:55 PM, "Benoît" notifications@github.com ha scritto:

Do you have access to your server error logs ? The reason for the blank pages has certainly been logged there, so it could be really helpful if you could paste it here. Otherwise, do you have a test environment aside from your production website ?

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-129420709 .

mage-eag commented 9 years ago

A blank page is most often synonymous of a fatal error, which is directly written to your server error log. Some common ways to fix it (related to the most common causes) include refreshing the cache or disabling the compiler, but because none is without risk, I think the best solution would currently still be for you to restore the previous file to its original state (at least, until you can get access to your server logs, so that we know the exact error before trying to randomly fix it :)

simulatorhck commented 9 years ago

I think tath attached logfile tells domething Il 10/ago/2015 02:26 PM, "Benoît" notifications@github.com ha scritto:

A blank page is most often synonymous of a fatal error, which is directly written to your server error log. Some common ways to fix it (related to the most common causes) include refreshing the cache or disabling the compiler, but because none is without risk, I think the best solution would currently still be for you to restore the previous file to its original state (at least, until you can get access to your server logs, so that we know the exact error before trying to randomly fix it :)

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-129426488 .

mage-eag commented 9 years ago

Can you attach the log file by commenting directly on the issue's page ? (seems like attachments are ignored when replying by email) Thanks !

simulatorhck commented 9 years ago

This is the error part. Can be something with path or simlinks?

2015-08-10T12:08:50+00:00 ERR (3): Strict Notice: Declaration of LucidPath_ExtStatus_Helper_Data::isModuleEnabled() should be compatible with that of Mage_Core_Helper_Abstract::isModuleEnabled() in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Helper/Data.php on line 2 2015-08-10T12:09:01+00:00 ERR (3): Strict Notice: Declaration of LucidPath_ExtStatus_Helper_Data::isModuleEnabled() should be compatible with that of Mage_Core_Helper_Abstract::isModuleEnabled() in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Helper/Data.php on line 2 2015-08-10T12:09:01+00:00 ERR (3): Recoverable Error: Argument 1 passed to LucidPath_ExtStatus_Model_Adminhtml_Observer::addColumn() must be an instance of Mage_Adminhtml_Block_Sales_Order_Grid, instance of BL_CustomGrid_Block_Rewrite_Raveinfosys_Deleteorder_Block_Adminhtml_Sales_Order_Grid given, called in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 88 and defined in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 4 2015-08-10T12:09:02+00:00 ERR (3): Recoverable Error: Argument 1 passed to LucidPath_ExtStatus_Model_Adminhtml_Observer::addColumn() must be an instance of Mage_Adminhtml_Block_Sales_Order_Grid, instance of BL_CustomGrid_Block_Rewrite_Raveinfosys_Deleteorder_Block_Adminhtml_Sales_Order_Grid given, called in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 88 and defined in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 4 2015-08-10T12:13:43+00:00 ERR (3): Strict Notice: Declaration of LucidPath_ExtStatus_Helper_Data::isModuleEnabled() should be compatible with that of Mage_Core_Helper_Abstract::isModuleEnabled() in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Helper/Data.php on line 2 2015-08-10T12:13:43+00:00 ERR (3): Recoverable Error: Argument 1 passed to LucidPath_ExtStatus_Model_Adminhtml_Observer::addColumn() must be an instance of Mage_Adminhtml_Block_Sales_Order_Grid, instance of BL_CustomGrid_Block_Rewrite_Raveinfosys_Deleteorder_Block_Adminhtml_Sales_Order_Grid given, called in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 88 and defined in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 4 2015-08-10T12:13:44+00:00 ERR (3): Warning: include(BL/CustomGrid/Helper/Grid.php): failed to open stream: No such file or directory in /var/www/vhosts/tucompri.it/httpdocs/lib/Varien/Autoload.php on line 94 2015-08-10T12:13:44+00:00 ERR (3): Warning: include(BL/CustomGrid/Helper/Grid.php): failed to open stream: No such file or directory in /var/www/vhosts/tucompri.it/httpdocs/lib/Varien/Autoload.php on line 94 2015-08-10T12:13:44+00:00 ERR (3): Warning: include(): Failed opening 'BL/CustomGrid/Helper/Grid.php' for inclusion (include_path='/var/www/vhosts/ tucompri.it/httpdocs/app/code/local:/var/www/vhosts/tucompri.it/httpdocs/app/code/community:/var/www/vhosts/tucompri.it/httpdocs/app/code/core:/var/www/vhosts/tucompri.it/httpdocs/lib:.:') in /var/www/vhosts/tucompri.it/httpdocs/lib/Varien/Autoload.php on line 94 Il 10/ago/2015 05:41 PM, "Benoît" notifications@github.com ha scritto:

Can you attach the log file by commenting directly on the issue's page ? (seems like attachments are ignored when replying by email) Thanks !

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-129499618 .

simulatorhck commented 9 years ago

Done it seems to work now. No idea how i solved Great :) Il 10/ago/2015 05:43 PM, "Danilo Pizzini" danilo.pizzini@gmail.com ha scritto:

This is the error part. Can be something with path or simlinks?

2015-08-10T12:08:50+00:00 ERR (3): Strict Notice: Declaration of LucidPath_ExtStatus_Helper_Data::isModuleEnabled() should be compatible with that of Mage_Core_Helper_Abstract::isModuleEnabled() in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Helper/Data.php on line 2 2015-08-10T12:09:01+00:00 ERR (3): Strict Notice: Declaration of LucidPath_ExtStatus_Helper_Data::isModuleEnabled() should be compatible with that of Mage_Core_Helper_Abstract::isModuleEnabled() in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Helper/Data.php on line 2 2015-08-10T12:09:01+00:00 ERR (3): Recoverable Error: Argument 1 passed to LucidPath_ExtStatus_Model_Adminhtml_Observer::addColumn() must be an instance of Mage_Adminhtml_Block_Sales_Order_Grid, instance of BL_CustomGrid_Block_Rewrite_Raveinfosys_Deleteorder_Block_Adminhtml_Sales_Order_Grid given, called in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 88 and defined in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 4 2015-08-10T12:09:02+00:00 ERR (3): Recoverable Error: Argument 1 passed to LucidPath_ExtStatus_Model_Adminhtml_Observer::addColumn() must be an instance of Mage_Adminhtml_Block_Sales_Order_Grid, instance of BL_CustomGrid_Block_Rewrite_Raveinfosys_Deleteorder_Block_Adminhtml_Sales_Order_Grid given, called in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 88 and defined in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 4 2015-08-10T12:13:43+00:00 ERR (3): Strict Notice: Declaration of LucidPath_ExtStatus_Helper_Data::isModuleEnabled() should be compatible with that of Mage_Core_Helper_Abstract::isModuleEnabled() in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Helper/Data.php on line 2 2015-08-10T12:13:43+00:00 ERR (3): Recoverable Error: Argument 1 passed to LucidPath_ExtStatus_Model_Adminhtml_Observer::addColumn() must be an instance of Mage_Adminhtml_Block_Sales_Order_Grid, instance of BL_CustomGrid_Block_Rewrite_Raveinfosys_Deleteorder_Block_Adminhtml_Sales_Order_Grid given, called in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 88 and defined in /var/www/vhosts/ tucompri.it/httpdocs/app/code/community/LucidPath/ExtStatus/Model/Adminhtml/Observer.php on line 4 2015-08-10T12:13:44+00:00 ERR (3): Warning: include(BL/CustomGrid/Helper/Grid.php): failed to open stream: No such file or directory in /var/www/vhosts/ tucompri.it/httpdocs/lib/Varien/Autoload.php on line 94 2015-08-10T12:13:44+00:00 ERR (3): Warning: include(BL/CustomGrid/Helper/Grid.php): failed to open stream: No such file or directory in /var/www/vhosts/ tucompri.it/httpdocs/lib/Varien/Autoload.php on line 94 2015-08-10T12:13:44+00:00 ERR (3): Warning: include(): Failed opening 'BL/CustomGrid/Helper/Grid.php' for inclusion (include_path='/var/www/vhosts/ tucompri.it/httpdocs/app/code/local:/var/www/vhosts/tucompri.it/httpdocs/app/code/community:/var/www/vhosts/tucompri.it/httpdocs/app/code/core:/var/www/vhosts/tucompri.it/httpdocs/lib:.:') in /var/www/vhosts/tucompri.it/httpdocs/lib/Varien/Autoload.php on line 94 Il 10/ago/2015 05:41 PM, "Benoît" notifications@github.com ha scritto:

Can you attach the log file by commenting directly on the issue's page ? (seems like attachments are ignored when replying by email) Thanks !

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-129499618 .

mage-eag commented 9 years ago

Seems like the file had not been uploaded correctly, something like this. Anyway, glad to hear that it is now fully working :) (I will keep the issue opened until I find some time to write down a FAQ section about custom columns and their compatibility with rewrited grid blocks / collections on version 0.9.0)

simulatorhck commented 9 years ago

Ook :) See you next relesse D Il 11/ago/2015 01:28 PM, "Benoît" notifications@github.com ha scritto:

Seems like the file had not been uploaded correctly, something like this. Anyway, glad to hear that it is now fully working :) (I will keep the issue opened until I find some time to write down a FAQ section about custom columns and their compatibility with rewrited grid blocks / collections on version 0.9.0)

— Reply to this email directly or view it on GitHub https://github.com/mage-eag/mage-enhanced-admin-grids/issues/199#issuecomment-129844324 .