Cleaner code! You will now be able to work with this file a-lot easier.
<?php
include_once dirname(__FILE__) . '/config/config.php';
$id = $_POST['id'];
$active = $_POST['active'];
$name = $_POST['name'];
$menu_file = $_FILES['menu_file'];
$i = 0;
foreach($id as $key) {
if(empty($active[$i])) {
$active[$i] = 'off';
}
$active_post = $active[$i];
$name_loop = $name[$i];
$file_name = $menu_file['name'][$i];
$file_type = $menu_file['type'][$i];
$file_size = $menu_file['size'][$i];
$file_error = $menu_file['error'][$i];
$file_temp = $menu_file['tmp_name'][$i];
if(!empty($file_name)) {
if($file_type !== 'application/pdf') {
header('Location: ' . dirname(__FILE__) . '/index.php?page=Menus&error=Incorrect_File_Type');
exit;
} else {
if($file_size > 5242880) {
header('Location: ' . dirname(__FILE__) . '/index.php?page=Menus&error=Incorrect_File_Size');
exit;
} else {
$today = date('d-m-Y');
$uploaddir = '/home/penmillh/en/includes/menus/';
$upload_name = str_replace(' ', '', $name_loop) . $today . '.pdf';
$uploadfile = $uploaddir . $upload_name;
if($file_error !== 'UPLOAD_ERR_OK') {
header('Location: ' . dirname(__FILE__) . '/index.php?page=Menus&error=File_Upload_Error');
exit;
} elseif($file_error === 'UPLOAD_ERR_OK') {
move_uploaded_file($file_tmp, $uploadfile);
// You may want to concider a LIMIT in this query? Also, escape them inputs!!
$query = "UPDATE `menus` SET `active` = '" . $active . "', `menu` = '" . $name_loop . "', `location` = '" . $upload_name . "' WHERE `id` = '" . $value . "'";
$query = mysql_query($query) or die(mysql_error());
header('Location: ' . dirname(__FILE__) . '/index.php?page=Menus&error=File_Upload_Success');
exit;
}
}
}
}
if(empty($file_name)) {
// Again, you may want a LIMIT here? And you need to escape them inputs!!
$query = "UPDATE `menus` SET `active` = '" . $active_post . "' WHERE `id` = '" . $key . "'";
$query = mysql_query($query) or die(mysql_error());
/*
* header('Location: ' . dirname(__FILE__) . '/index.php?page=Menus&error=Menu_Change_Success');
*/
}
$i++;
}
?>
James.
EDIT: Forgot to add the
dirname(__FILE__)
.