Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fatal Error: Unable to save any form in Magento admin #27

Open
33hops opened this issue Dec 13, 2012 · 2 comments
Open

Fatal Error: Unable to save any form in Magento admin #27

33hops opened this issue Dec 13, 2012 · 2 comments

Comments

@33hops
Copy link

33hops commented Dec 13, 2012

I installed the extension in Magento 1.7.0.2, the Varnish part works well but I get this error when trying to update any form in the Magento admin:

Fatal error: Call to a member function isActive() on a non-object in /var/www/html/magento/app/code/community/Magneto/Varnish/Helper/Data.php on line 113

I am using Varnish in a separate server, in any case that does not seem to be the problem

The part of code throwing the error is:

(line 113) if (!Mage::getSingleton('enterprise_logging/config')->isActive($eventCode, true)) {
return;
}

I have searched the web with no luck, there must be more 1.7.0.2 users with the same problem

@33hops
Copy link
Author

33hops commented Dec 13, 2012

Provisional solution:

I am posting this auto-reply here just in case somebody gets the same error, in any case if somebody knows how to fix it please post code.

The function where the offender line is at

/var/www/html/magento/app/code/community/Magneto/Varnish/Helper/Data.php on line 113

is supposed to log magneto-varnish purges in Magento admin log. You can safely skip it by just making the function return false. In any case this issue has to be related to any recent Magento structure or syntax change previous to version 1.7.0.2. I don't have much time to investigate it, for me it is O.K. to simply turn off logging. I guess a simple update inside getSingleton('-- HERE --') providing the route to the loggin facility should be enough.

@0d1
Copy link

0d1 commented May 14, 2013

hello,

could you please paste the portion of code you modified ?

thank you.

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

No branches or pull requests

2 participants