Jump to content
thirty bees forum
  • 0

Consistency checks - Registered hooks are executable


Question

Posted (edited)

Hello,

I just updated to 1.4, after that I run Consistency check and I got this message:

 

These non-executable hooks do not (or at least they shouldn't) hurt your system much. But they contribute to performance degradation, because your system have to unnecessarily load and instantiate the module on every hook execution.

List of errors
Error description	Actions

Module blockcategories registered hook actionAdminLanguagesControllerStatusBefore but does not implement hook handler	
Module custompayments registered hook advancedPaymentOptions but does not implement hook handler	
Module paypal registered hook displayBeforePayment but does not implement hook handler	
Module genzo_krona registered hook actionAdminGenzoKronaPlayersListingResultsModifier but does not implement hook handler	
Module stripe registered hook Header but does not implement hook handler	
Module stripe registered hook displayPaymentRequestButton but does not implement hook handler	
Module stmegamenu registered hook actionProductDelete but does not implement hook handler	
Module stmegamenu registered hook hookActionObjectProductDeleteAfter but does not implement hook handler	
Module stiosslider registered hook displayLeftColumn but does not implement hook handler	
Module stbanner registered hook displayFooter but does not implement hook handler	
Module stfeaturedcategories registered hook displayHomeSecondaryLeft but does not implement hook handler	
Module stblog registered hook displayAnywhere but does not implement hook handler
 

What I soppose to do now?
Contact modules developer? Some module are core one's.

p.s. maybe the results was equal with 1.3...

Edited by Beeta

4 answers to this question

Recommended Posts

  • 0
Posted

This is not a bug just sloppiness in making of modules. Modules registered hooks for which they do not support these hooks.
In Modules -> Positions, you can safely unhook these modules from these hooks.

  • 0
Posted

It may or may not be bug in module. Usually, it's just some copy'n'paste issue, or forgotten piece of code. But I have seen modules that had spelling mistake in handler function, and therefore didn't work as designed. Fixing those spelling mistakes actually fixed some bug in the module (in this particular case, it was cache invalidation)

In any case, module author should look at it. 

For core modules, we have already fixed most of these issues. For example, blockcategories is fixed in version 3.0.3

https://github.com/thirtybees/blockcategories/commit/f242b651ff078c1b69474f4a028fa2fe336fe695

 

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...