[#420] Fehler beim Löschen im Mail-Log

[achiboy]

Wenn ich im Mail-Log einige Einträge auswähle und dann lösche, gibt es einen Fehler.
In der Datei ...\contenido\includes\include.mail_log.php auf Zeile 42 wird anstelle des erwarteten Array der Wert NULL verarbeitet.

Meine Analysen haben ergeben, dass die $_REQUEST['idmails'] einen nicht validen JSON-Wert enthält.

Ist es richtig, dass die ...\contenido\scripts\includes\mail_log_overview.js wie folgt geändert werden müsste?
Bitte beachtet Zeilen 177 + 186

alt:

Code: Alles auswählen

173        function actionDeleteSelectedEmails() {
174            var idmails = [];
175            $_root.find(_options.markMailsSelector + ':checked').each(function(pos, element) {
176                var idmail = $(element).closest('tr').data('idmail');
177                idmails.push('[' + idmail + ']');
178            });
179            idmails = idmails.join(',');
180            if (!idmails.length) {
181                return;
182            }
183
184            Con.showConfirmation(_options.text_deleteMultipleConfirmation, function() {
185                console.log('actionDeleteSelectedEmails idmails', idmails);
186                submitDeleteEmailForm(idmails);
187            });
188        }

neu:

Code: Alles auswählen

173        function actionDeleteSelectedEmails() {
174            var idmails = [];
175            $_root.find(_options.markMailsSelector + ':checked').each(function(pos, element) {
176                var idmail = $(element).closest('tr').data('idmail');
177                idmails.push(idmail);
178            });
179            idmails = idmails.join(',');
180            if (!idmails.length) {
181                return;
182            }
183
184            Con.showConfirmation(_options.text_deleteMultipleConfirmation, function() {
185                console.log('actionDeleteSelectedEmails idmails', idmails);
186                submitDeleteEmailForm('[' + idmails + ']');
187            });
188        }

[xmurrix]

Hallo achiboy,

danke dir für den Fehlerbericht und die Lösung.

Der Fehler ist behoben:
https://github.com/CONTENIDO/CONTENIDO/ ... 69be982255