diff options
| author | Aleksander Machniak <alec@alec.pl> | 2013-04-05 12:48:36 +0200 |
|---|---|---|
| committer | Aleksander Machniak <alec@alec.pl> | 2013-04-05 15:00:57 +0200 |
| commit | d2d8d498c67e7193dcdcebe7d806753ec02843fd (patch) | |
| tree | 883baec73071decf3531712aacc94c4451c593ab /program/steps/mail/copy.inc | |
| parent | e152227571fb5ae9fc5c800ac27efcd0242e94c4 (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 |