1. Hugh Messenger
  2. Bugs
  3. Friday, 27 February 2015
  4.  Subscribe via email
Hey Daniel,

I'm sure you are probably aware, but just tiny issue after the J! 3.4 update ...

Strict standards: Declaration of JMail::addAttachment() should be compatible with PHPMailer::addAttachment($path, $name = '', $encoding = 'base64', $type = '', $disposition = 'attachment') in /var/www/html/jomsoc/plugins/system/mandrill/mailer/mail.php on line 43

Sent you a PR ...

https://github.com/compojoom/cmandrill/pull/1

-- hugh
Responses (3)
Accepted Answer Pending Moderation
Hi there. Love your extension. But we are having the same problem since updating to 3.4. I've attached a screenshot.
Additionally, it is now affecting the site Frontend. So we can't keep the plugin portion switched on and have the site display correctly.
Anything you can do?
Thanks!
Rich
  1. more than a month ago
  2. Bugs
  3. # 1
Daniel Dimitrov
Support team
Accepted Answer Pending Moderation
Hey Richard, Hey Hugh,

@Hugh - thanks for the PR.

We'll be working on CMandrill in April, if everything goes according to the plan. We plan to implement the complete api (as it now supports bcc, more e-mail attachments and few other goodies) and we'll have a new stats view.
For now just set error reporting on your site to none, or add Hugh changes to the code.

Regards,
Daniel
Please if you use our extensions be so kind and review them at JED
Matukio | Hotspots Pro | CComment Pro
  1. more than a month ago
  2. Bugs
  3. # 2
Accepted Answer Pending Moderation
Richard,

You really should turn error reporting off on live production sites, only enable it on development sites or when actually trying to debug something. Enabling it not only causes "cosmetic" issues, it can break things in unpredictable ways (like causing Javascript errors when error reporting messes up JSON responses in AJAX calls), and it can expose information you don't want people knowing about your site (file paths, dataset naming, etc).

You can safely ignore this specific notice, it doesn't impact the functionality of CMandril at all. If it bugs you, just edit the file as per the PR referenced in my post, or download the fixed file from my fork on github

Hugh
  1. more than a month ago
  2. Bugs
  3. # 3
  • Page :
  • 1


There are no replies made for this post yet.
However, you are not allowed to reply to this post.
Powered by EasyDiscuss for Joomla!