It seems like somewhat of a standard for Drupal 6 modules to use Autoload for their class loading. This is what the DBTNG module uses.

Also switching this over would be dependent on this patch for Autoload landing, to add support for drush: #547736: Autoload fails with drush.

Comments

alex_b’s picture

Good call. I won't have the time to jump onto this, but I'd happily accept a patch. Thanks for filing the issue.

dave reid’s picture

It would be ever better if #817496: Autoload via .info files gets in to automodal!

JoeMcGuire’s picture

Priority: Normal » Major
Status: Active » Needs review
StatusFileSize
new669 bytes

Looks like autoload is in use now. Only thing missing is declaring the dependancy. Patch attach adds this.

Marked as major as without this its possible to disable the autoload module but leave with the drupal_queue module enables breaking a site.

Status: Needs review » Needs work

The last submitted patch, drupal_queue-autoload-dependancy-3.patch, failed testing.

JoeMcGuire’s picture

Version: 6.x-1.0-alpha1 » 6.x-1.x-dev

That was a pretty standarded git patch. Lets try again by changing the issue to be against 6.x-1.x-dev.

JoeMcGuire’s picture

Status: Needs work » Needs review
JoeMcGuire’s picture

Green and passing! This is a tiny change and probably just needs maintainer approval.

dave reid’s picture

Status: Needs review » Needs work

Depending on Autoload actually is not required right now. It *works* with Autoload, but anything needing to invoke drupal_queue automatically should be depending on autoload.

JoeMcGuire’s picture

Status: Needs work » Closed (works as designed)

haha quite right unexpected design but useful and works as designed.