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

bforchhammer commented

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

plach
he/him
Primary language Italian
Location Venezia
commented

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

damienmckenna
Location TN, USA
commented

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

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented

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

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
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