We help IT Professionals succeed at work.

Check out our new AWS podcast with Certified Expert, Phil Phillips! Listen to "How to Execute a Seamless AWS Migration" on EE or on your favorite podcast platform. Listen Now

x

php file upload  overwrite

phillystyle123
on
Medium Priority
541 Views
Last Modified: 2012-05-07
hello -

i'm using some commercial php upload script that i purchased years ago.

the problem is it won't allow me to overwrite my images which i have to do for this particular application - it just creates new ones.

don't know a lot about php upload so i tried throwing an $overwrite=true variable in the code for kicks and nothing changed. here's the way the code for the variables:

// *** BEGIN Simply Upload ***
require_once("FXInc/uploadAction.inc");
$errMsg = "";
$action = true;
$noPath = true;
$rename = false;
$delete = false;
$overwrite = true;
$FX_successRedirect = "";
$FX_DirPath = "../images/ba/".$thumbs_dir."/thumbs/";
$FX_typearray = array("application","audio","image");
$FX_extarray = array();
$FX_size = "";
$FX_fields = array();
if ((isset($HTTP_POST_VARS["FX_upload"])) && ($HTTP_POST_VARS["FX_upload"] == "form1")) {
  require_once("FXInc/upload.inc");
}
// *** END Simply Upload ***
Comment
Watch Question

Commented:
well it look like the code you have has few years ;). The $HTTP_POST_VARS variable is deprecated for long time ;). Maybe you should consider pay someone to wrote or modify your script :).
But back to the point. You have to show us a little bit more of your code, fragment you show us is too small. We can't predict problem just from definition of variables ;)

Author

Commented:
thanks for taking a look at this szewkam:

this file looks relevant:

<?php
function renameIt($name,$path) {
  $re = "/\(\d+\)/";
  if (preg_match($re,$name,$matches)) {
    $num = substr($matches[0],1,strlen($matches[0])-2);
    $newName = preg_replace($re,"(".(++$num).")",$name);
  } else {
    $newName = str_replace(".","(1).",$name);
  }
  if (file_exists($path . $newName)) $newName = renameIt($newName,$path);
  return $newName;
}
 
$FX_uploadAction = $HTTP_SERVER_VARS['PHP_SELF'];
if (isset($HTTP_SERVER_VARS['QUERY_STRING'])) {
  $FX_uploadAction .= "?" . htmlentities($HTTP_SERVER_VARS['QUERY_STRING']);
}
?>

Open in new window

Unlock this solution with a free trial preview.
(No credit card required)
Get Preview

Commented:
you can also find a file where function renameIt is called, and just comment line with it or remove the call :)

Author

Commented:
thanks guys - solutionDriver - can you show me where your code would fit into my code?

this is what i'm guessing:
<?php
function renameIt($name,$path) {
unlink($path . $newName);
  return $name; // amendment: don't rename  -- phillystyle123, [date]
 
  $re = "/\(\d+\)/";
  $re = "/\(\d+\)/";
  if (preg_match($re,$name,$matches)) {
    $num = substr($matches[0],1,strlen($matches[0])-2);
    $newName = preg_replace($re,"(".(++$num).")",$name);
  } else {
    $newName = str_replace(".","(1).",$name);
  }
  if (file_exists($path . $newName)) $newName = renameIt($newName,$path);
 }
 
$FX_uploadAction = $HTTP_SERVER_VARS['PHP_SELF'];
if (isset($HTTP_SERVER_VARS['QUERY_STRING'])) {
  $FX_uploadAction .= "?" . htmlentities($HTTP_SERVER_VARS['QUERY_STRING']);
}
?>

Open in new window

Commented:
You may need to change $delete = false; to $delete = true; and to make sure that there is enough permissions for deletion. Overwrite is simply delete then create new.
Hello phillystyle123.

that's exactly right, you got it.

Best,

  sd

Author

Commented:
thanks solutionDriver!
Unlock the solution to this question.
Thanks for using Experts Exchange.

Please provide your email to receive a free trial preview!

*This site is protected by reCAPTCHA and the Google Privacy Policy and Terms of Service apply.

OR

Please enter a first name

Please enter a last name

8+ characters (letters, numbers, and a symbol)

By clicking, you agree to the Terms of Use and Privacy Policy.