Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.This is a fairly big module, so we should investigate breaking up the module to work with the new mechanism.
Comments
Comment #1
milesgillham commentedAye.
Comment #2
milesgillham commentedThis has been done in -dev branch for ed_classified.module I've marked it as fixed in the CVS but only because strictly I've converted require_once to module_include. Further thought would need to be given as to how and when the includes are pulled in and that's a major structural issue I think. Presently they're all pretty much needed all the time.
Will leave this on the back-burner for the moment and bring to the fore as the module grows further and performance becomes a bigger concern.
Cheers,
Miles
Comment #3
milesgillham commentedSilly me, I was bitten by drupal_get_path() not available at module load time which module_load_include() relies on. I've made some structural changes in -dev so ed_classified should be a little more efficient in terms of inclusion overheads.
Cheers,
Miles