Posted by mcurry on August 17, 2008 at 4:04pm
Jump to:
| Project: | Classified Ads |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | minor |
| Assigned: | milesgillham |
| Status: | closed (fixed) |
Issue Summary
This is a fairly big module, so we should investigate breaking up the module to work with the new mechanism.
Comments
#1
Aye.
#2
This 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
#3
Silly 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
#4
Automatically closed -- issue fixed for 2 weeks with no activity.