search

cividesk / com.cividesk.email.sparkpost

This extension allows CiviCRM to send emails and process bounces through the SparkPost service.
11 stars 30 forks source link

Autoloading Mail_Sparkpost #53

Open michaelmcandrew opened 6 years ago

michaelmcandrew commented 6 years ago

Hi there,

In https://github.com/cividesk/com.cividesk.email.sparkpost/blob/master/sparkpost.php#L130, the comment says "Don't know why it does not autoload ...".

AFAIK (though I haven't tested this) this is because MailSparkpost is not within CRM. I think CRM_Mail_Sparkpost would work.

Not a biggie. Just wanted to report since there was a comment there.

universalhandle commented 5 years ago

See https://docs.civicrm.org/dev/en/latest/extensions/info-xml/#classloader if you wish to autoload classes that don't quite match the expectations of CiviCRM's core autoloader.

nganivet commented 5 years ago

Thanks guys, will look at these and fix in next release.