In Lieu of the auto class loading what is in Drupal 7 here is a minor patch that gives a hook to allow the loading of additional classes to be included.

I added this in so that when I backported my beanstalkd to D6 it will now work.

CommentFileSizeAuthor
0001-Add-module_invoke_all-to-allow-the-loading-of-other-.patch719 bytesgordon

Comments

alex_b’s picture

alex_b commented

Seems reasonable.

Have you had a look at http://drupal.org/project/autoload ? I am asking without having actually reviewed autoload.

Would using autoload be an alternative to this patch? If so, why would we or would we not use it for drupal_queue in this instance?

gordon’s picture

gordon commented

I had not actually see it, but then again using a whole module to replace a single line in this module I think is a bit of overkill.

and also for how long this module is really going to be used, just having a single hook will make it easy to do some profiling of the queues with more real world data which is why I have backported my beanstalkd module from Drupal 7

alex_b’s picture

alex_b commented
Status: Needs review » Reviewed & tested by the community

#2: Thanks for elaborating. I wanted to know whether you had a specific technical reason why you didn't use autoload.

Will commit in a bit.

alex_b’s picture

alex_b commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks.

http://drupal.org/cvs?commit=299918

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.