diff options
| author | Aleksander Machniak <alec@alec.pl> | 2013-04-05 12:48:36 +0200 |
|---|---|---|
| committer | Aleksander Machniak <alec@alec.pl> | 2013-04-05 12:48:36 +0200 |
| commit | 4dfae96dcdfba93a8b03db2d0422766b20adf77d (patch) | |
| tree | 64c989a18f2ac892606f7cea00c15d8d17402a73 /program/steps/mail/copy.inc | |
| parent | 443b92a7ee19e321b350750240e0fc54ec5be357 (diff) | |
Fix copying messages to a folder with "empty" name e.g. "0", better error handling
Diffstat (limited to 'program/steps/mail/copy.inc')
| -rw-r--r-- | program/steps/mail/copy.inc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/program/steps/mail/copy.inc b/program/steps/mail/copy.inc index a72378b0e..876657485 100644 --- a/program/steps/mail/copy.inc +++ b/program/steps/mail/copy.inc @@ -24,10 +24,10 @@ if (!$OUTPUT->ajax_call) return; // move messages -if (!empty($_POST['_uid']) && !empty($_POST['_target_mbox'])) { - $uids = get_input_value('_uid', RCUBE_INPUT_POST); +if (!empty($_POST['_uid']) && strlen($_POST['_target_mbox'])) { + $uids = get_input_value('_uid', RCUBE_INPUT_POST); $target = get_input_value('_target_mbox', RCUBE_INPUT_POST, true); - $mbox = get_input_value('_mbox', RCUBE_INPUT_POST, true); + $mbox = get_input_value('_mbox', RCUBE_INPUT_POST, true); $copied = $RCMAIL->storage->copy_message($uids, $target, $mbox); @@ -47,7 +47,7 @@ if (!empty($_POST['_uid']) && !empty($_POST['_target_mbox'])) { } // unknown action or missing query param else { - exit; + $OUTPUT->show_message('internalerror', 'error'); } // send response |