setDebug does not expose handler

[editorassistant]

Backward-compatible change in smtp.php, line 411:

if (is_callable($this->debug)) {
$this->_smtp->setDebug(true, $this->debug);
} else {
$this->_smtp->setDebug(true);
}

[schengawegga]

Hi @editorassistant,

thanks for your message.
$this->debug is only a boolean property.
As i understand, you want to set the debugging-handler in Net_SMTP?
Give me a little bit to think about your suggested code.
Maybe it would be better to add a property like $this->debug_handler?
I will think about that.

[editorassistant]

My suggestion would be to follow the Net_SMTP convention since the class property is commented as "Turn on Net_SMTP debugging?"

…

On Sun, Jul 16, 2023 at 3:40 PM Armin Graefe ***@***.***> wrote: Hi @editorassistant <https://github.com/editorassistant>, thanks for your message. $this->debug is only a boolean property. As i understand, you want to set the debugging-handler in Net_SMTP? Give me a little bit to think about your suggested code. Maybe it would be better to add a property like $this->debug_handler? I will think about that.

-- EDAS Conference Services - edas.info <http://edas.info>

[schengawegga]

@editorassistant Added PR #30

[CloCkWeRX]

Fixed by #30