Webklex

I just looked at some rfcs: - https://www.rfc-editor.org/rfc/rfc3501#section-6.3.11 - https://datatracker.ietf.org/doc/html/rfc9051#section-6.3.12 The response could look like this: ``` TAG3 OK APPEND completed ``` ..or: ``` TAG3 OK [APPENDUID 38505 3955] APPEND...

Figured out why `$date` is required - is required for the legacy protocol. ```php \imap_append($this->stream, $folder, $message, $flags, $date); ``` In order to use the same interface, this method has...

I start to realize, that a bool cast might not be a good idea at all. Usually all commands can return something (if a newer imap protocol version is used)....

@szymekjanaczek Yes it's currently optional as well: https://github.com/Webklex/php-imap/blob/cc3e72f8517b0bf169a91d9fe5831ebd5740874d/src/Connection/Protocols/LegacyProtocol.php#L388-L397 But this means it has to be there - otherwise you wont be able to set it. Or am I missing something?...

Hi @epocci , thanks for your report. I'm not sure if this is a bug or rather expected behavior. If we implement your suggested change, it might cause more confusion....

@szymekjanaczek suggested to use something similar to [Laravel Resources](https://laravel.com/docs/9.x/eloquent-resources): > Like Resources in Laravel - but firstly we have to check the compatibility between responses of each operation. In the...

Sounds good to me :+1: I've meant that every class / object you create during the runtime will need some additional system resources. At some point this adds up and...

Hi @szymekjanaczek , yes I agree on both 1.) and 2.) however I'm having some trouble with [a]. This would add a php-module dependency in the mix, which is currently...

@ainesophaur , sounds reasonable to me. I wouldn't mind kicking `php*-imap` out entirely. What are possible downsides? Folks which rely on the legacy mode are stuck with v4.x, header decoding...

By kicking out `php*-imap` we also reduce the amount of code that has to be refactored and tested. Win / Win ?