Parser for eml files?

[piernik]

Is it possible to use Your library as parser of eml files?
Now I'm using https://github.com/php-mime-mail-parser/php-mime-mail-parser

But it would be great to use only one library.

[Slamdunk]

Theoretically yes, an EML file can be treated like an EmbeddedMessage.

Would you be so kind to create PR containing a proof-of-concept?

[piernik]

I think It's not possible right now. EmbeddedMessage extends AbstractPart at the end which needs Imap resource as constructor param. All interesting methods are in AbstractPart class (as I think).

[Slamdunk]

You are right, I was referring that there is only one raw imap call in the whole EmbeddedMessage logic to serve the BasicMessageInterface API, and that call is easily overwritable to work with an EML file.

[piernik]

So it's not possible to use those methods without IMAP resource? Are You planning to adjust Your lib to use eml parser?

[Slamdunk]

It is possible, and I think pretty straightforward, to get what you ask.

Not soon though. I'll tag it in 1.2 milestone.

[piernik]

Would be great.

[piernik]

Whe do You expect deliver it??

[Slamdunk]

I don't know yet, I have personal issues that keep me off coding.

[piernik]

maybe a timeframe? I need to publish my project by the end of the year so by 15th I need to start testing. Will it be ready within 2 weeks??

[Slamdunk]

This is a free and open source software developed for free and maintained in our spare time, we can't guarantee any timeframe, so we don't give them.

[piernik]

I understad - I'll use different parser :/

[piernik]

Any news on that?

[Slamdunk]

Not yet

[pyatnitsev]

Is it actual? Do you need some help with PR?

[Slamdunk]

I'd like to be honest: I'm not working on this and I haven't even planned to.

Any PR is welcome: I think it should be easy to get a parser that returns a BasicMessageInterface implementation for a generic message.

Tests are mandatory: start the basic feature against tests/fixtures/plain_only.eml and submit a PR, after discussing the general approach a more comprehensive data set can be tested too.

[pyatnitsev]

Ok. Lets discuss an implementation of this feature.
I see it, as a separate class, maybe EmlParser with method parseEml() that have one param - path to eml object and returns instance of new class - ParsedMessage (implements BasicMessageInterface)

Right?

[pyatnitsev]

Another idea: are you realy want to get own implementation of eml parser, maybe better will be integration of https://github.com/php-mime-mail-parser/php-mime-mail-parser to get BasicMessageInterface object?

WDYT?

[Slamdunk]

https://github.com/php-mime-mail-parser/php-mime-mail-parser is a front-end for the Mailparse extension.

I think it would be good to use that extension too.

A less pretty but still valid idea is to provide an EML parser that just parses the headers with imap_rfc822_parse_headers function, leaving the raw body. This wouldn't require any other third-party library/extension.

[rikvdh]

@Slamdunk I was working on this as I was migrating from laminas-mail to here and I need this for testing.

I am quite far with this, some 'final' keywords in the AbstractMessage need to go in order to override them in a 'EmlMessage' class. It can decode headers now, no problem. But when decoding content, it relies on a structure-object generated by imap_fetchstructure.

I see a way with php-mime-mail-parser/php-mime-mail-parser or using mailparse directly. But is that desirable? I don't want to put effort in making it work when that route is not 'the way to go forward'.

See https://github.com/ddeboer/imap/compare/master...rikvdh:imap:feature/eml-import?expand=1