The Bakery Everything CakePHP

I was working on something similar, but using an Image model to store the file info and associate the image with parent model using a belongsTo association. Your solution is much nicer.

I was also able to modify the behavior to make it work for both image and file uploads.

Thanks!

I have just started my adventure with cake and could You tell how to validate avantar extension . For example when I'am trying to upload zip file I wish to receive flash message: "wrong type of file".

Thanks for this piece of code, Yevgeny!

How do I go about passing just one of the dimensions and having the behavior create the image such that the aspect ratio is preserved?
I tried not passing 'height' for the versions, but I got errors and the version images were not created.

Thanks in advance!

...unfortunatly this behaviour isnt working on recursive associations. Though this isnt caused by the behaviour itself but by the fact that CAKE doesnt seem to call behaviours recursivly it is still a big limitation. I wonder if there is a way to make behaviours work on recursive associated models without modifying the CAKE core files.

Hi SkieDr!

There is no info about the license and I would like to modify this behaviour to use it in a personal project, may I?

BTW, good job mate, this code is very useful :-)

Use the beforeValidate() function below to check if the uploaded image is really an image.. invalidates if not...

Just paste it somewhere in the behavior above.

Haven't tested it that much but it works fine for me!

http://www.pastebin.be/6872
cheers!

Would be great to move the GD stuff out of __resize() and put them into __resize_gd2() and then create __resize_imagemagick().

This would allow the use of the far superior imagemagick over gd, and would also be easily extend with other image lib functions if needed.

p.s. Awesome code, thanks!

I have some questions about defining the upload folder used in these 2 functions.

function __getFolder(&$model, $record) {
extract($this->settings[$model->name]);
return $baseDir .'/'. Inflector::camelize($model->name) .'/'. $record[$model->primaryKey] . '/';
}
function __getFullFolder(&$model, $field) {
extract($this->settings[$model->name]);
return WWW_ROOT . IMAGES_URL. $baseDir .DS. Inflector::camelize($model->name) .DS. $model->id .DS;
}


Where do $baseDir and IMAGES_URL get defined?

For some reason, the file type is being stored in the DB field rather than the file name. I'm using CakePHP 1.2.0.6311-beta

How do I go about passing just one of the dimensions and having the behavior create the image such that the aspect ratio is preserved?
I tried not passing 'height' for the versions, but I got errors and the version images were not created.
It is quite easy to modify the behavior to make it generate the other dimension when only one is passed. In __resize() method just before the $width and $height declaration add this code:


        if (!isset($fieldParams['height']) or !$fieldParams['height']) {
            $fieldParams['height'] = (int) (($fieldParams['width'] / $size[0]) * $size[1]);
        } elseif (!isset($fieldParams['width']) or !$fieldParams['width']) {
            $fieldParams['width'] = (int) (($fieldParams['height'] / $size[1]) * $size[0]);
        }


Is that possible at all ?

There is problem with afterFind method that leads to impossibility of accessing this behavior through association (using https://trac.cakephp.org/ticket/2056 or http://groups.google.com/group/cake-php/browse_thread/thread/5f3d90e3686f191b/c3bb61dfb902b8c9?lnk=gst&q=afterfind#c3bb61dfb902b8c9 patch that makes it possible).
This method should return $results instead of boolean.
Change method declaration from:
function afterFind(&$model, &$results, $primary) to:
function afterFind(&$model, $results, $primary) And the last statement in it from:
return true; to:
return $results;

Perhaps in afterFind(), this code:


} else {
    foreach ($fields as $field => $fieldParams) {
        if (isset($results[$model->name][$field]) && ($results[$i][$model->name][$field]!='')) {
            $value=$results[$i][$model->name][$field];
            $results[$model->name][$field]=$this->__getParams(&$model, $field, $value, $fieldParams, $results[$model->name]);
        }
    }
}

should be changed to this:


} else {
    foreach ($fields as $field => $fieldParams) {
        if (isset($results[$model->name][$field]) && ($results[$model->name][$field]!='')) {
            $value=$results[$model->name][$field];
            $results[$model->name][$field]=$this->__getParams(&$model, $field, $value, $fieldParams, $results[$model->name]);
        }
    }
}


I need to delete a photo and I have modified beforeSave(), in the view I put a delete field ("delete_"+field name) like this:

echo $form->input('delete_fot1',array('type' => 'checkbox'));

and the beforeSave() could be like this:

    function beforeSave(&$model) {
        //$model->log($this->settings);
        extract($this->settings[$model->name]);
        if (empty($model->data[$model->name][$model->primaryKey])) {
            //$this->__addToListBottom(&$model);
        }
        
        $tempData = array();
        foreach ($fields as $key=>$value) {
            $field = ife(is_numeric($key), $value, $key);
            if (isset($model->data[$model->name][$field])) {
                if ($this->__isUploadFile($model->data[$model->name][$field])) {
                    $tempData[$field] = $model->data[$model->name][$field];
                    $model->data[$model->name][$field]=$this->__getContent($model->data[$model->name][$field]);
                } else {
                    //delete image 
                    if (isset($model->data[$model->name]["delete_".$field])) {
                        if($model->data[$model->name]["delete_".$field]==1){
                            
                            @unlink($this->__getFullFolder(&$model, $field).'/'.$field.'.png');
                            @unlink($this->__getFullFolder(&$model, $field).'/thumb_'.$field.'.png');
                            $model->data[$model->name][$field]="";
                            
                        }else{
                            unset($model->data[$model->name][$field]);
                        }
                    }else{
                        unset($model->data[$model->name][$field]);
                    }
                    
                }
            }
        }
         //debug($model->data);
         //debug($tempData);        
        $this->runtime[$model->name]['beforeSave'] = $tempData;         
        return true;
    } 


You wrote "thubnails" instead of thumbnails.
I can't find any other error: your behavior is great

great code but this will not work out of the box in

1.2.0.7125

perhaps add a little note at the top to see what version this was written for.

I noticed, while using this behavior, that sometimes my versions would not be created, and I could not figure out what was going on. So I dug into the code, and noticed at the very end of the file, in the __resize function, there was an if($resize) statement, basically canceling creation of the version if the original was the same size as the new version.

Although this is efficient, it is not really what's intended by most users. I'm not going to run checks on my versions to see if they exist or not, and if the original is the same size as my version before outputting it to the browser. So I added a bit of code to simply copy the original if it happens to be the same size as the version I am creating.

Add this ELSE to the final IF($resize) statement in the __resize function:

        else {
            // we still want our image w/prefix, even if it is the same size
            copy($url, $cachefile);
        }

and that solves the problem, all versions are created and you can use the version you created for a specific use without worry that the image won't exist.

Hi!

For some reason, the file type is being stored in the DB field rather than the file name. I'm using CakePHP 1.2.0.6311-beta
I have just the same problem - any solutions?

Hi again,

was an easy one, just changing

function __getContent($file) {
    return $file['type'];
}
to

function __getContent($file) {
    return $file['name'];
}
did the trick!

Bye

I'm receiving the following warning using this behavior.

Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file
As a newbie to php, i'm not really sure what it means or how to fix that. Any tips?

With the new version of CakePHP 1.2.1.8004 (maybe also before) makes the bugfix of comment #20 obsolete.
So this is the right version again for all who have changed the behavior according to comment #20

    function __getContent($file) {
        return $file['type'];
    }



Hello,

the line:

$folder = &new Folder($path = $folderName, $create = true, $mode = '777'); 

has to be changed in this way:

$folder = &new Folder($path = $folderName, $create = true, $mode = 0777); 

$mode has to be an integer. Better if 0755

Marco

It's works, the images (thumb, main, l and s) are in webroot/img/Model/... sucessfully.

But when I try to show this images in the view, returns a blank result.

Code:  <?php echo $html->image($item['Item']['avatar']['thumb']) ?> 
HTML:  <img src="/car_stuff/Item/34/thumb_avatar.jpg" alt="" /> 
"car_stuff" is my app name

Any ideas? Cheers!

To show the imgs in view.ctp
My imgs /app/webroot/img/MODEL_NAME/ITEM_ID

change the code:
 
function __getFolder(&$model, $record) {
        extract($this->settings[$model->name]);
        return  $baseDir .'/'. Inflector::camelize($model->name) .'/'. $record[$model->primaryKey] . '/'; 
to:

function __getFolder(&$model, $record) {
        extract($this->settings[$model->name]);
        return  $baseDir .'/img/'. Inflector::camelize($model->name) .'/'. $record[$model->primaryKey] . '/'; 

Is there someone here have a working source code for Cake 1.2.2.8120... I've tried the code above but I can't make the image appear on views.

I have the same problem Izzy, comment 24 and 25. Take a look

Izzy dizzy, I'm using Cake 1.2.2.8120 and the behavior works with some corrections. Take a look at comments #13 and #25.

I don't understand why it's so difficult to write proper documentation for the code one writes.