Miglioramento della segnalazione bug

bug.php

@@ -2,39 +2,27 @@
 
 include_once __DIR__.'/core.php';
 
-$pageTitle = tr('Bug');
+$mail = Mail::get();
-$jscript_modules[] = App::getPaths()['js'].'/ckeditor/ckeditor.js';
+$bug_email = 'info@openstamanager.com';
+
+$user = Auth::user();
 
 if (filter('op') == 'send') {
     // Preparazione email
     $mail = new Mail();
 
-    // Mittente
-    $mail->From = $post['email_from'];
-    $mail->FromName = $_SESSION['username'];
-    $mail->AddReplyTo($post['email_from']);
-
     // Destinatario
-    $mail->AddAddress($post['email_to']);
+    $mail->AddAddress($bug_email);
 
-    // Copia
+    // Oggetto
-    if (!empty($post['email_cc'])) {
+    $mail->Subject = 'Segnalazione bug OSM '.$version;
-        $mail->AddCC($post['email_cc']);
-    }
 
-    // Copia nascosta
+    // Aggiunta dei file di log (facoltativo)
-    if (!empty($post['email_bcc'])) {
-        $mail->AddBCC($post['email_bcc']);
-    }
-
-    $mail->Subject = 'Segnalazione bug OSM '.$version.' ('.(!empty($revision) ? 'R'.$revision : tr('In sviluppo')).')';
-
-    // Se ho scelto di inoltrare i file di log, allego
     if (!empty($post['log']) && file_exists($docroot.'/logs/error.log')) {
         $mail->AddAttachment($docroot.'/logs/error.log');
     }
 
-    // Se ho scelto di inoltrare copia del db
+    // Aggiunta della copia del database (facoltativo)
     if (!empty($post['sql'])) {
         $backup_file = $docroot.'/Backup OSM '.date('Y-m-d').' '.date('H_i_s').'.sql';
         backup_tables($backup_file);
@@ -44,11 +32,23 @@ if (filter('op') == 'send') {
         $_SESSION['infos'][] = tr('Backup del database eseguito ed allegato correttamente!');
     }
 
-    $body = $post['body'].'<hr><br>'.tr('IP').': '.get_client_ip();
+    // Aggiunta delle informazioni di base sull'installazione
+    $infos = [
+        'Utente' => $user['username'],
+        'IP' => get_client_ip(),
+        'Versione OSM' => $version.' ('.(!empty($revision) ? $revision : 'In sviluppo').')',
+        'PHP' => phpversion(),
+    ];
 
-    // Se ho scelto di inoltrare le INFO del mio sistema
+    // Aggiunta delle informazioni sul sistema (facoltativo)
     if (!empty($post['info'])) {
-        $body .= $_SERVER['HTTP_USER_AGENT'].' - '.getOS();
+        $infos['Sistema'] = $_SERVER['HTTP_USER_AGENT'].' - '.getOS();
+    }
+
+    // Completamento del body
+    $body = $post['body'].'<hr>';
+    foreach ($infos as $key => $value) {
+        $body .= '<p>'.$key.': '.$value.'</p>';
     }
 
     $mail->Body = $body;
@@ -62,6 +62,7 @@ if (filter('op') == 'send') {
         $_SESSION['infos'][] = tr('Email inviata correttamente!');
     }
 
+    // Rimozione del dump del database
     if (!empty($post['sql'])) {
         delete($backup_file);
     }
@@ -70,32 +71,22 @@ if (filter('op') == 'send') {
     exit();
 }
 
+$pageTitle = tr('Bug');
+$jscript_modules[] = App::getPaths()['js'].'/ckeditor/ckeditor.js';
+
 if (file_exists($docroot.'/include/custom/top.php')) {
     include $docroot.'/include/custom/top.php';
 } else {
     include $docroot.'/include/top.php';
 }
 
-$email_to = Settings::get('Destinatario');
+if (empty($mail['from_address']) || empty($mail['server'])) {
-$email_from = Settings::get('Indirizzo per le email in uscita');
-
-$mail = Mail::get();
-
-if (empty($email_to) || empty($email_from) || empty($mail['server'])) {
     echo '
 <div class="alert alert-warning">
     <i class="fa fa-warning"></i>
-        <b>'.tr('Attenzione!').'</b> '.tr('Per utilizzare correttamente il modulo di segnalazione bug devi configurare alcuni parametri riguardanti le impostazione delle email').'.';
+    <b>'.tr('Attenzione!').'</b> '.tr('Per utilizzare correttamente il modulo di segnalazione bug devi configurare alcuni parametri riguardanti le impostazione delle email').'.
 
-    if (empty($email_to) || empty($email_from)) {
+    '.Modules::link('Account email', $mail['id'], tr('Correggi account'), null, 'class="btn btn-warning pull-right"').'
-        echo Modules::link('Impostazioni', $dbo->fetchArray("SELECT `idimpostazione` FROM `zz_settings` WHERE sezione='Email'")[0]['idimpostazione'], tr('Correggi impostazioni'), null, 'class="btn btn-warning pull-right"');
-    }
-
-    if (empty($mail['server'])) {
-        echo Modules::link('Account email', $mail['id'], tr('Correggi account'), null, 'class="btn btn-warning pull-right"');
-    }
-
-    echo '
     <div class="clearfix"></div>
 </div>';
 }
@@ -103,50 +94,29 @@ if (empty($email_to) || empty($email_from) || empty($mail['server'])) {
 echo '
 <div class="box">
     <div class="box-header">
-			<h3 class="box-title"><i class="fa fa-bug"></i>'.tr('Segnalazione bug').'</h3></h3>
+        <h3 class="box-title"><i class="fa fa-bug"></i> '.tr('Segnalazione bug').'</h3>
     </div>
 
     <div class="box-body">
-			<form method="post" action="'.$rootdir.'/bug.php?op=send">
+        <form method="post" action="">
+            <input type="hidden" name="op" value="send">
+
             <table class="table table-bordered table-condensed table-striped table-hover">
                 <tr>
                     <th width="150" class="text-right">'.tr('Da').':</th>
-						<td>
+                    <td>'.$mail['from_address'].'</td>
-                            {[ "type": "email", "placeholder": "'.tr('Mittente').'", "name": "email_from", "value": "'.$email_from.'", "required": 1 ]}
-						</td>
                 </tr>
 
                 <!-- A -->
                 <tr>
                     <th class="text-right">'.tr('A').':</th>
-						<td>
+                    <td>'.$bug_email.'</td>
-                            {[ "type": "email", "placeholder": "'.tr('Destinatario').'", "name": "email_to", "value": "'.$email_to.'", "required": 1 ]}
-						</td>
-					</tr>
-
-					<!-- Cc -->
-					<tr>
-						<th class="text-right">'.tr('Cc').':</th>
-						<td>
-                            {[ "type": "email", "placeholder": "'.tr('Copia a').'...", "name": "email_cc" ]}
-						</td>
-					</tr>
-
-					<!-- Bcc -->
-					<tr>
-						<th class="text-right">'.tr('Bcc').':</th>
-						<td>
-                            {[ "type": "email", "placeholder": "'.tr('Copia nascosta a').'...", "name": "email_bcc" ]}
-						</td>
                 </tr>
 
                 <!-- Versione -->
                 <tr>
                     <th class="text-right">'.tr('Versione OSM').':</th>
-
+                    <td>'.$version.' ('.(!empty($revision) ? $revision : tr('In sviluppo')).')</td>
-						<td>
-                            {[ "type": "span", "placeholder": "'.tr('Versione OSM').'", "value": "'.$version.' ('.(!empty($revision) ? $revision : tr('In sviluppo')).')" ]}
-						</td>
                 </tr>
             </table>
 
@@ -156,7 +126,7 @@ echo '
                 </div>
 
                 <div class="col-md-4">
-                        {[ "type": "checkbox", "placeholder": "'.tr('Allega copia del database').'", "name": "sql", "value": "0", "extra":"'.((!$user['is_admin']) ? 'readonly' : '').'" ]}
+                    {[ "type": "checkbox", "placeholder": "'.tr('Allega copia del database').'", "name": "sql" ]}
                 </div>
 
                 <div class="col-md-4">
@@ -172,7 +142,9 @@ echo '
             <!-- PULSANTI -->
             <div class="row">
                 <div class="col-md-12 text-right">
-                        <button type="submit" class="btn btn-primary" id="send" disabled><i class="fa fa-envelope"></i> '.tr('Invia segnalazione').'</button>
+                    <button type="submit" class="btn btn-primary" id="send" disabled>
+                        <i class="fa fa-envelope"></i> '.tr('Invia segnalazione').'
+                    </button>
                 </div>
             </div>
         </form>

update/2_4.sql

@@ -278,6 +278,10 @@ INSERT INTO `zz_settings` (`idimpostazione`, `nome`, `valore`, `tipo`, `editable
 -- apilayer API key (per validazione piva)
 INSERT INTO `zz_settings` (`idimpostazione`, `nome`, `valore`, `tipo`, `editable`, `sezione`) VALUES (NULL, 'apilayer API key for VAT number', '', 'string', '1', 'Generali');
 
---
 -- Aggiorno query modulo movimenti per ordinarli in funzione della data del movimento
 UPDATE `zz_modules` SET `options` = 'SELECT |select| FROM `mg_movimenti` JOIN `mg_articoli` ON `mg_articoli`.id = `mg_movimenti`.`idarticolo` WHERE 1=1 HAVING 2=2 ORDER BY `Data` DESC' WHERE `zz_modules`.`name` = 'Movimenti';
+
+-- Rimozione impostazioni inutilizzate
+DELETE FROM `zz_settings` WHERE `zz_settings`.`nome` = 'Indirizzo per le email in uscita';
+DELETE FROM `zz_settings` WHERE `zz_settings`.`nome` = 'Destinatario';
+DELETE FROM `zz_settings` WHERE `zz_settings`.`nome` = 'Destinatario fisso in copia (campo CC)';