It would be useful if the hook implementations could be loaded from a separate file rather than requiring them to be in the main module file, maybe look into how CTools does it.

Comments

bforchhammer’s picture

We already have an implementation of hook_hook_info(), which allows ET hook implementations to be declared in MODULE.entity_translation.inc...

plach’s picture

Yes, I totally want to reduce our memory footprint in various way before 1.0, but this is not my main priority now.

DamienMcKenna’s picture

No problem, I saw what you did for core so I totally understand having higher priorities right now :) Kudos!

plach’s picture

I meant that we should focus on getting more tests and features done before focusing on optimization :)

plach’s picture

Title: Allow hooks to be loaded from their own file » Minimize memory footprint of Entity Translation
Version: 7.x-1.0-beta1 » 7.x-1.x-dev
Category: feature » task