zend-mail icon indicating copy to clipboard operation
zend-mail copied to clipboard

Published 20 hours ago • verified publisher icon zendframework

When set -f option, no set -f option automatically

Open tzmfreedom opened this issue 6 years ago • 5 comments

Zend\Mail\Transport\SendMail always append envelope-from parameter.
But I want to set any envelope-from.
In this PR, only when not set option -f, set -f option automatically. This means if I set [email protected] or -bs [email protected], Zend\Mail\Transport\Sendmail doesn't set -f option.

tzmfreedom avatar Jun 13 '19 13:06 tzmfreedom

without getting deeper to the actual changes, have some thoughts:

  1. seems weird to have a negative parameter. how about set positive parameter which is b default true, and can be set as false.
  2. maybe have a specific option (envelope related) rather generic "no override" as behavior could be different if want/want not to override some other parameter.

glensc avatar Jun 18 '19 10:06 glensc

Thank you for you reply and advice !

I rethink, revise my code and changed this PR goal.

On this PR, only when not set option -f, set -f option automatically.
This means if I set [email protected] or -bs [email protected], Transport\Sendmail doesn't set -f option.
So there is no negative parameter, no additional method.

tzmfreedom avatar Jun 19 '19 12:06 tzmfreedom

you should update PR title as well. and the body is a bit hard to read as well.

glensc avatar Jun 19 '19 15:06 glensc

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/17.

weierophinney avatar Dec 31 '19 21:12 weierophinney

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:

  • Squash all commits in your branch (git rebase -i origin/{branch})
  • Make a note of all changed files (`git diff --name-only origin/{branch}...HEAD
  • Run the laminas/laminas-migration tool on the code.
  • Clone laminas/laminas-mail to another directory.
  • Copy the files from the second bullet point to the clone of laminas/laminas-mail.
  • In your clone of laminas/laminas-mail, commit the files, push to your fork, and open the new PR. We will be providing tooling via laminas/laminas-migration soon to help automate the process.

weierophinney avatar Dec 31 '19 21:12 weierophinney