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 ...


-- 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?
  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.

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

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

  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.

Last questions

Matukio Teilnehmerliste Anzeige
Hallo Ich habe ein Problem. auf. Ich nutze Matukio und im Frontend werden die Ze...
24 Replies
Posted on Friday, 06 October 2023
New World Season 4 introduces a new Season Pass
Amazon Games Unveils New World Season 4: Frost Eternal In an exciting announc...
0 Replies
Posted on Wednesday, 29 November 2023
  • New
    • #7.0.2
    Wrath of the Lich King Classic Edition is generati
    World of Warcraft: Wrath of the Lich King Classic Edition adds a controversial f...
    0 Replies
    Posted on Wednesday, 29 November 2023
  • New
    • #3rd party
    ccomment is stuck, saving forever
    Hi Guys, Sitename: traveltomtom.net is anyone experiencing problems with c...
    0 Replies
    Posted on Tuesday, 28 November 2023
  • New
  • ReCaptcha einrichten schlägt fehl...
    Hallo Zusammen, vielleicht kann mir jemand helfen, ich bin schon total am ve...
    2 Replies
    Posted on Wednesday, 22 November 2023