search

zendframework / zend-mail

Mail component from Zend Framework
BSD 3-Clause "New" or "Revised" License
96 stars 111 forks source link

Consider removing dependency on zend-loader #185

Open acelaya opened 6 years ago

acelaya commented 6 years ago

I have seen that this package requires zendframework/zend-loader, just for one class, the Zend\Mail\Header\HeaderLoader.

However, this class basically defines a map to resolve header names to class names.

Would you be open to replace this by a simple array map in the Zend\Mail\Headers class? This would let us drop the requirement on zendframework/zend-loader (assuming I'm not missing anything...).

I could do it if you want.

Ocramius commented 6 years ago

LGTM: kill it 👍

On 15 Dec 2017 23:42, "Alejandro Celaya" notifications@github.com wrote:

I have seen that this package requires zendframework/zend-loader, just for one class, the Zend\Mail\Header\HeaderLoader.

However, this class basically defines a map to resolve header names to class names.

Would you be open to replace this by a simple array map in the Zend\Mail\Headers class? This would let us drop the requirement on zendframework/zend-loader (assuming I'm not missing anything...).

I could do it if you want.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/zendframework/zend-mail/issues/185, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJakA0Io3UFHVl5vZQRH2NYFXA4I7aHks5tAvXCgaJpZM4REFdX .

michalbundyra commented 4 years ago

This repository has been closed and moved to laminas/laminas-mail; a new issue has been opened at https://github.com/laminas/laminas-mail/issues/40.