-
Notifications
You must be signed in to change notification settings - Fork 193
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added option to include/exclude inline attachments #128
Conversation
@@ -389,16 +389,19 @@ public function getAddresses($name) | |||
* | |||
* @return Attachment[] | |||
*/ | |||
public function getAttachments() | |||
public function getAttachments($include_inline=false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Incorrect spacing between argument
$include_inline
and equals sign; expected 1 but found 0 - Incorrect spacing between default value and equals sign for argument
$include_inline
; expected 1 but found 0
@@ -448,9 +451,9 @@ public function getAttachments() | |||
* @return array Saved attachments paths | |||
* @throws Exception | |||
*/ | |||
public function saveAttachments($attach_dir) | |||
public function saveAttachments($attach_dir,$include_inline=false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Incorrect spacing between argument
$include_inline
and equals sign; expected 1 but found 0 - Incorrect spacing between default value and equals sign for argument
$include_inline
; expected 1 but found 0 - Expected 1 space between comma and argument
$include_inline
; 0 found
1 similar comment
I changed the default for |
Thanks a lot it's a very good job. But we are missing some test about include inline attachments to false |
1 similar comment
Tell me what you think of the new unit test. It processes all the sample message again but with $include_inline set to false. |
It will be fine like this for the moment. Thanks. I really need to work on a new version 3.0 with better improvement and better structure for test and parser. |
Added an optional argument to getAttachments and saveAttachments to include inline attachments (defaults to false). Also update the README file accordingly.