Skip to content

Conversation

@derickr
Copy link
Member

@derickr derickr commented Nov 5, 2024

No description provided.

… to ezcMailParserOptions

By setting this option like:

  $parser = new ezcMailParser();
  $parser->options->parseMultipartMixedTextAttachmentsAsFiles = true;

The parser returns text parts as attachments, but only if they are sub-parts of
multipart/mixed parts.

This is distinct from the 'parseTextAttachmentsAsFiles' option, which does it
for *all* text parts.
@derickr derickr changed the title Add test for PR #95 Add new option 'parseMultipartMixedTextAttachmentsAsFiles' to ezcMailParserOptions Nov 6, 2024
@derickr
Copy link
Member Author

derickr commented Nov 7, 2024

Ping @eileenmcnaughton @seamuslee001 :-)

@eileenmcnaughton
Copy link
Contributor

thank you @derickr - you are amazing

I have applied this locally & our unit test still passes with it so it all looks good - I'm just waiting on our full test suite to run civicrm/civicrm-core#31450

@derickr derickr closed this in bebae73 Mar 12, 2025
@derickr derickr merged commit d9302d2 into master Mar 12, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants