Hi there,
first of all thank you for Swift Mailer, it seems to be a good project, really.
I am here just to report a bug in last release, exactely in file EasySwift and line 739.
Since 3 lines before you wrote
* @param string The encoding used (default is base64)
I think you should change the default null value from parameter $encoding and method addAttachment.
It is just because if someone is using this great wrapper for old Swift version and he doesn't add base64 encoding this wrapper will send a new Swift_Message_Attachment($data, $filename, $type, $encoding), with null value.
I am not sure the problem I had with PDF was there, anyway this is just a little doc or code problem that you could easily solve, changing the comment or using $encoding="base64" as default, as is in Swift_Message_Attachment constructor.
Cheers,
Andrea Giammarchi
EasySwift and the default encoding
Moderators: Chris Corbyn, General Moderators
Re: EasySwift and the default encoding
I would like tell you that I solved EasySwift attachment problems sending manually base64, as last encoding parameter.
This is just to confirm that you can easily fix this little bug changing EasySwift file, in line 739, this method declaration:
with this one
Kind Regards
This is just to confirm that you can easily fix this little bug changing EasySwift file, in line 739, this method declaration:
Code: Select all
public function addAttachment($data, $filename=null, $type="application/octet-stream", $encoding=null)Code: Select all
public function addAttachment($data, $filename=null, $type="application/octet-stream", $encoding='base64')- Chris Corbyn
- Breakbeat Nuttzer
- Posts: 13098
- Joined: Wed Mar 24, 2004 7:57 am
- Location: Melbourne, Australia
Re: EasySwift and the default encoding
Eek! I didn't even realise that class was still there. I should have deprecated this ages ago
Surprised it even still works at all 
Re: EasySwift and the default encoding
Its only a Wrapper, but it is working perfectly.
So please do not remove them from your next release
Cheers,
Andrea
So please do not remove them from your next release
Cheers,
Andrea
- Chris Corbyn
- Breakbeat Nuttzer
- Posts: 13098
- Joined: Wed Mar 24, 2004 7:57 am
- Location: Melbourne, Australia
Re: EasySwift and the default encoding
I'm afraid it will definitely be gone in next release. EasySwift is a v2 wrapper. Th next release is going to be v4 and I simply can't maintain something two versions oldandr3a wrote:Its only a Wrapper, but it is working perfectly.
So please do not remove them from your next release
Cheers,
Andrea