diff --git a/bug.php b/bug.php index dac7d5544..69d361b19 100644 --- a/bug.php +++ b/bug.php @@ -6,70 +6,28 @@ $pageTitle = 'Bug'; $jscript_modules[] = $js.'/ckeditor.js'; if (filter('op') == 'send') { - $dati = $post; - - // Parametri e-mail - $replace = [ - 'Server SMTP' => 'email_host', - 'Username SMTP' => 'email_username', - 'Porta SMTP' => 'email_porta', - 'Sicurezza SMTP' => 'email_secure', - 'Password SMTP' => 'email_password', - ]; - $rs = $dbo->fetchArray("SELECT * FROM zz_settings WHERE sezione = 'Email'"); - foreach ($rs as $r) { - if (!empty($replace[$r['nome']])) { - $dati[$replace[$r['nome']]] = $r['valore']; - } - } - // Preparazione email - $mail = new PHPMailer(); - - // Se non specificato l'host uso le impostazioni di invio mail di default del server - if (!empty($dati['email_host'])) { - $mail->IsSMTP(); - $mail->IsHTML(); - $mail->SMTPDebug = 2; - - $mail->Host = $dati['email_host']; - $mail->Port = $dati['email_porta']; - - // Controllo se รจ necessaria l'autenticazione per il server di posta - if (!empty($dati['email_username'])) { - $mail->SMTPAuth = true; - $mail->Username = $dati['email_username']; - $mail->Password = $dati['email_password']; - } - - if (in_array(strtolower($dati['email_secure']), ['ssl', 'tls'])) { - $mail->SMTPSecure = strtolower($dati['email_secure']); - } - } - - $mail->WordWrap = 50; + $mail = new Mail(); // Mittente - $mail->From = $dati['email_from']; + $mail->From = $post['email_from']; $mail->FromName = $_SESSION['username']; - $mail->AddReplyTo($dati['email_from']); + $mail->AddReplyTo($post['email_from']); // Destinatario - $mail->AddAddress($dati['email_to']); + $mail->AddAddress($post['email_to']); // Copia - if (!empty($dati['email_cc'])) { - $mail->AddCC($dati['email_cc']); + if (!empty($post['email_cc'])) { + $mail->AddCC($post['email_cc']); } // Copia nascosta - if (!empty($dati['email_bcc'])) { - $mail->AddBCC($dati['email_bcc']); + if (!empty($post['email_bcc'])) { + $mail->AddBCC($post['email_bcc']); } $mail->Subject = 'Segnalazione bug OSM '.$version.' ('.(!empty($revision) ? 'R'.$revision : tr('In sviluppo')).')'; - $mail->AltBody = tr('Questa email arriva dal modulo bug di segnalazione bug di OSM'); - $body = $dati['body'].'

'.tr('IP').': '.get_client_ip()."
\n"; // Se ho scelto di inoltrare i file di log, allego if (!empty($post['log']) && file_exists($docroot.'/logs/error.log')) { @@ -86,6 +44,8 @@ if (filter('op') == 'send') { $_SESSION['infos'][] = tr('Backup del database eseguito ed allegato correttamente!'); } + $body = $post['body'].'

'.tr('IP').': '.get_client_ip(); + // Se ho scelto di inoltrare le INFO del mio sistema if (!empty($post['info'])) { $body .= $_SERVER['HTTP_USER_AGENT'].' - '.getOS(); @@ -93,6 +53,8 @@ if (filter('op') == 'send') { $mail->Body = $body; + $mail->AltBody = 'Questa email arriva dal modulo bug di segnalazione bug di OSM'; + // Invio mail if (!$mail->send()) { $_SESSION['errors'][] = tr("Errore durante l'invio della segnalazione").': '.$mail->ErrorInfo; @@ -100,8 +62,6 @@ if (filter('op') == 'send') { $_SESSION['infos'][] = tr('Email inviata correttamente!'); } - $mail->SmtpClose(); - if (!empty($post['sql'])) { delete($backup_file); } diff --git a/modules/interventi/actions.php b/modules/interventi/actions.php index 9053bbe40..304d24f27 100644 --- a/modules/interventi/actions.php +++ b/modules/interventi/actions.php @@ -578,32 +578,11 @@ switch (post('op')) { $testo_email = post('body'); $allegato = post('allegato'); - $mail = new PHPMailer(); - - $mail->IsSMTP(); - $mail->Host = get_var('Server SMTP'); - - if (get_var('Username SMTP') != '') { - $mail->SMTPAuth = 1; - $mail->Username = get_var('Username SMTP'); - $mail->Password = get_var('Password SMTP'); - } - - if (get_var('Sicurezza SMTP') == 'SSL') { - $mail->SMTPSecure = 'ssl'; - } elseif (get_var('Sicurezza SMTP') == 'TLS') { - $mail->SMTPSecure = 'tls'; - } - - $mail->Port = get_var('Porta SMTP'); + $mail = new Mail(); $mail->AddReplyTo($from_address, $from_name); $mail->SetFrom($from_address, $from_name); - $mail->WordWrap = 60; // a capo dopo 60 caratteri - $mail->IsHTML(true); // invio mail in formato HTML - $mail->AltBody = strip_tags($testo_email); - $mail->AddAddress($destinatario, ''); // se ho impostato la conferma di lettura @@ -616,11 +595,11 @@ switch (post('op')) { $mail->MsgHTML($testo_email); - if ($allegato != '') { + if (!empty($allegato)) { $mail->AddAttachment($allegato); } - if (!$mail->Send()) { + if (!$mail->send()) { $_SESSION['errors'][] = tr("Errore durante l'invio dell'email").': '.$mail->ErrorInfo; } else { $dbo->query('UPDATE in_interventi SET data_invio=NOW() WHERE id='.prepare($id_record)); diff --git a/src/Mail.php b/src/Mail.php new file mode 100644 index 000000000..6870aa8e7 --- /dev/null +++ b/src/Mail.php @@ -0,0 +1,76 @@ + Settings::get('Server SMTP'), + 'username' => Settings::get('Username SMTP'), + 'port' => Settings::get('Porta SMTP'), + 'port' => Settings::get('Password SMTP'), + 'secure' => Settings::get('Sicurezza SMTP'), + ]; + + // Preparazione email + $this->IsHTML(true); + + if (!empty($config['host'])) { + $this->IsSMTP(true); + + // Impostazioni di debug + $this->SMTPDebug = 3; + $this->Debugoutput = function ($str, $level) { + $this->infos[] = $str; + }; + + // Impostazioni dell'host + $this->Host = $config['host']; + $this->Port = $config['port']; + + // Impostazioni di autenticazione + if (!empty($config['username'])) { + $this->SMTPAuth = true; + $this->Username = $config['username']; + $this->Password = $config['password']; + } + + // Impostazioni di sicurezza + if (in_array(strtolower($config['secure']), ['ssl', 'tls'])) { + $this->SMTPSecure = strtolower($config['secure']); + } + } + + $this->WordWrap = 78; + } + + public function send() + { + global $logger; + + if (empty($this->AltBody)) { + $this->AltBody = strip_tags($this->Body); + } + + $result = parent::send(); + + $this->SmtpClose(); + + // Segnalazione degli errori + foreach ($this->infos as $info) { + $logger->addRecord(\Monolog\Logger::ERROR, $info); + } + + return $result; + } +}