Code Duplication    Length = 10-10 lines in 2 locations

admin/letter.php 1 location

@@ 364-373 (lines=10) @@
361
362
            for ($upl = 0 ;$upl < 5; ++$upl) {
363
                $uploader = new XoopsMediaUploader($uploaddir, $xnewsletter->getConfig('xn_mimetypes'), $xnewsletter->getConfig('xn_maxsize'), null, null);
364
                if ($uploader->fetchMedia(@$_POST['xoops_upload_file'][$upl])) {
365
                    //$uploader->setPrefix("xn_") ; keep original name
366
                    $uploader->fetchMedia($_POST['xoops_upload_file'][$upl]);
367
                    if (!$uploader->upload()) {
368
                        $errors = $uploader->getErrors();
369
                        redirect_header('javascript:history.go(-1)', 3, $errors);
370
                    } else {
371
                        $uploaded_files[] = ['name' => $uploader->getSavedFileName(), 'type' => $uploader->getMediaType()];
372
                    }
373
                }
374
375
            }
376

letter.php 1 location

@@ 484-493 (lines=10) @@
481
            }
482
            $uploader = new XoopsMediaUploader($uploaddir, $xnewsletter->getConfig('xn_mimetypes'), $xnewsletter->getConfig('xn_maxsize'), null, null);
483
            for ($upl = 0 ;$upl < 5; ++$upl) {
484
                if ($uploader->fetchMedia($_POST['xoops_upload_file'][$upl])) {
485
                    //$uploader->setPrefix("xn_") ; keep original name
486
                    $uploader->fetchMedia($_POST['xoops_upload_file'][$upl]);
487
                    if (!$uploader->upload()) {
488
                        $errors = $uploader->getErrors();
489
                        redirect_header('javascript:history.go(-1)', 3, $errors);
490
                    } else {
491
                        $uploaded_files[] = ['name' => $uploader->getSavedFileName(), 'origname' => $uploader->getMediaType()];
492
                    }
493
                }
494
            }
495
496
            // create items in attachments