I’ve used the single image upload code, but I get the following error

Notice: Undefined variable: write in cake/app/controllers/components/upload.php on line 286

Warning: unlink(/cake/app/webroot/img/uploads/) [function.unlink]: Is a directory in cake/app/controllers/images_controller.php on line 42

I’m new to cake and php, but I think the $write variable hasn’t been defined.

Please help

Thanks

You could more easier use a array for storing multiple file-uploads. So instead of using $html->file (or $form->file in cake 1.2), create a custom element:

< ?php for($x=0;$x

In the controller you can use:

data[image] as $image) {
   $this->Image->create();
   $this->Image->save($image);
}
?>

Consider the controller code pseudo, since the array does not yet have the required structure for a valid insert into our Image table. Also there is no file handling implemented yet.

Class ‘AppController’ not found in C:\wamp\www\multipleimage\multiple-uploads-11062007\app\controllers\images_controller.php on line 2

Send me a reply and send this function

Remove the implode array line and then do a foreach,

very important to set the id to null or in each loop it will overwrite the date from the previous one and only add the last one.

if (!empty($array)){
	//save the data

	foreach ($array as $value) {
		$this->Image->id = null;
		$this->data['Image']['images'] = $value;
		$this->Image->save($this->data);
	}	

} else {
	$this->Session->setFlash('No images have been uploaded!');
	$this->redirect('/images/index');
	exit();
}

$this->redirect('/images/index');

First off, thanks for posting these components. I removed a bunch of the code so I could use them for general file uploads rather then just images.

I ran into a few things on the way I thought I should share.
The upload() function does not always return true or false depending on if the upload succeeds. This is important because in the code the $result is relied on in the image controller. I moved some things around, but at the bottom of the function I added this line which should take care of the issue:

// if no errors, then return true
return $this->error === false ? true : false ;

Also in the upload function, you go through the effort of adding an ending slash to the $directory variable being passed in, and assign it to $this->_destination. However, a few lines later when you need to use the slash appended directory, you use the $directory variable again rather then the freshly appended $this->_destination

$fileName = ($name == NULL) ? $this->uniquename($this->_destination......

The last thing I found, this may not be a big issue for some people, but in the uniquename function, and in the upload function, you hard code the “/” to separate directories. This is not a problem if you are on *nix, but windows has a problem with this. To get around this problem, just replace the slash with PHP constant DIRECTORY_SEPARATOR.

Thanks again for the work!

foreach($array as $arr):
   $this->Image->query("INSERT INTO images(id,imgSrc) VALUES(NULL,"'.$arr.'")");
endforeach;

Great work though.