Support all IMAP message headers if needed. Remove Doctrine dependency.

[xeoncross]

(Background: I deal with a lot of messages which have custom headers.)

This commit solves two problems:

1. The Doctrine ArrayCollection object wastes resources in each message object for a simple MIME header array.
2. I need custom headers not returned by the default imap_header() function.

[ddeboer]

Your support for imap_fetchheader() looks good.

But is the ArrayCollection overhead noticeable to you?

And can you please update your code to conform to the PSR-2 coding standards? Thanks!

[xeoncross]

1. The overhead of ArrayCollection is noticeable in my tests (I don't remember exactly how many kB's per-message it adds). The parameter array isn't very complex so there is no loss of functionality switching (other than needing to do an isset check first) so I think it's a good trade-off in this case.
2. What part of the code isn't PSR-2 standard?

[ddeboer]

Space after if

[ddeboer]

no loss of functionality switching (other than needing to do an isset check first)

All right, let's get rid of the ArrayCollection (even if it means breaking BC for the getParameter method).

What part of the code isn't PSR-2 standard?

Actually only the missing spaces after if, now I look at your code more closely. ;)

[OndraM]

👍

[ddeboer]

How is this rawHeaders useful? I don’t see a way to access it.

[xeoncross]

Well, $rawHeaders contains extra MIME headers that $headers does not contain. I honestly don't remember if I was extending this class or just marked this as public or what.

[mean-cj]

@ddeboer +1
Please merge this feature for get raw of all header.
Thank.

[Slamdunk]

Implemented in #202 due to heavily refactored code.
Thank you very much for your help