-
Notifications
You must be signed in to change notification settings - Fork 111
Treat quote as value if already in quote using another delimiter #226
base: master
Are you sure you want to change the base?
Conversation
@silvadiego is this same problem? can you cherry-pick testcase from there? |
test/Header/ListParser.php
Outdated
<?php | ||
/** | ||
* @see https://github.com/zendframework/zend-mail for the canonical source repository | ||
* @copyright Copyright (c) 2005-2018 Zend Technologies USA Inc. (https://www.zend.com) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2019 perhaps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@silvadiego is this same problem? can you cherry-pick testcase from there?
Yes, i will do that when i get home.
@weierophinney can we get this merged? as noted in #224, this is regression from changes from #147 resolution i.e from your 8aac5f7 commit |
port to this branch: zendframework#226
@Xerkus ping? |
@silvadiego I don't know what's this project merge policy (hardly anybody accepts changes here), but good practice is to squash useless commits to one commit and |
This repository has been closed and moved to laminas/laminas-mail; a new issue has been opened at laminas/laminas-mail#18. |
This repository has been moved to laminas/laminas-mail. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|
Fixes the issue as described in #222.