The Bakery Everything CakePHP

Great work, bud! I was about to use the upload behavior, but I'll try yours first. Looks quite powerful.
Thanks!

edit: couldn't make it work yet :(. I used the examples you provide, but nothing happened. I've been peaking in the code all afternoon, and found some things, but not sure how to handle them. Basically, I found the code never entered the conditional in line 365 in the the behavior file. Changed the "$attachment['file']" to "$attachment['name']['file']" in that line, and entered, but couldn't save yet... At least now I got a nice error (File was not accepted for transfer. Not going to save this record. Source file argument is invalid or file is of unknown type)...
I'll keep trying/seeing this and let you know. :)

This is a great piece of code. Got so much features already built in. Hats off to the developers and all the contributors.

But there is a few things I would like to mention here on how to get it working. It took me a whole day just to figure it out.

The first part is the database. The script included in the download is missing a the auto increment extra. So you should change the file in config/sql/attachments.php so that the line with:

'id' => array('type'=>'integer', 'null' => false, 'default' => NULL, 'key' => 'primary'),

is changed to:

'id' => array('type'=>'integer', 'null' => false, 'default' => NULL, 'key' => 'primary', 'extra' => 'auto_increment'),

Note the auto increment part at the end. Once that is done you can create it with bake. The command is just (running from your current cake root directory):
cake/console/cake schema run create attachments

That should set up your database if all your configuration is right.

And then I find that I need to change the file models/behaviors/attachment.php. In there under the function setup I have to add:
        
        if(!is_array($config)){
            $config=$this->defaultSettings;
        }
Right at the beginning of the function so that there would be a default if there wasn't one passed by the user (And actually I don't have a clue how to pass one actually).

And then in the same file under the beforeSave function I have to add the following code:

foreach($attachment as $key=>$data){
    if(is_array($data)){
        foreach($data as $skey=>$sdata){
            $uh[$skey][$key]=$sdata;
        }
    }
}        
$attachment=$uh;

before line 365 just before the test:

if(isset($attachment['file'])) {

because it seems the arrangments of the array is wrong for the Transfer object. But apart from that it works pretty well. Thank you for this great work.

heh, even with Abdullah fixes, im still fighting with this, but learning a lot while doing it.
Im stuck in a mime type detection problem... since the files are being saved in the temp folder as "randomname.tmp", the detection is not made thru the extension, and cant get it working. I'll try in a production lamp server asap, since im testing in windows. :)

Can you print what error you actually got? Maybe we can work it out. I'm still new to CakePHP so I value the practice.

But one thing that I can think off the top of my head is that if you don't require mime type detection then why don't you just switch it off?

And about the error you got before
File was not accepted for transfer. Not going to save this record. Source file argument is invalid or file is of unknown type
With my fix you might still get it too. Because after the remapping of the array, some of the arrays would be empty and the program would try to copy an empty file. No biggie though because if you just turn off debugging and suppress the message it should still reroute you back to your index page.
And more thing I forgot to mention, but you'll only realize this after you got the upload working and editing it again to add another file. In the tutorial it doesn't mention that the element actually require the Number helper. So in your controller you should have the line:

var $helpers=array('Number');


This is just warning for those risking their lives with the latest commits from SVN (1.2.r6469).

Behaviors have changed. They are no longer accessed as arrays in the model, but as BehaviorsCollection Object.

So you need to change components/attachment.php, line 76 to:
    if(!isset($controller->{$controller->modelClass}->Behaviors->Attachment)) 

Hope this helps someone.

Attachments look very useful, but can not get it to work.
I get this error:

Attachment behavior is not loaded for Example model [APP/controllers/components/attachment.php, line 77]
I a working with the examples provided in the source and the latest beta: 1.2.0.6311

Any ideas?

I downloaded attachments 0.42 and the latest cake from svn but still get the same problem.

Hi all, this compoment looks great, but I can't get it working. -:(
I encounter similar problems that others reported. First, the code never enter this line:

if(isset($attachment['file'])) {

because of the errors that Abdullah Zainul Abidin reported.
Debugging attachment show this:

[ExmapleAttachment] => Array
(
[47e2d69465f77] => Array
(
[name] => Array
(
[file] => xxx.jpg
)

[type] => Array
(
[file] => image/jpeg
)

[tmp_name] => Array
(
[file] => \\xampplite\\tmp\\php237.tmp
)

[error] => Array
(
[file] => 0
)

=> Array
(
[file] => 22460
)

)

)

After implementing Abdullah fix (e.g then isset($attachment['file']) would eval to true), the code enters the loop, but then it complains about mimetype (e.g because it's \\xampplite\\tmp\php237.tmp) during setTemporary.
Even after I disable mimetype check, I got stuck during setDestination because of extension checking (e.g. c:\\xxx.jpg) failed because it didn't started with 'file://'???

I am running on Windows, Firefox.
Using this line of code:

input($assocAlias.'.'.$id.'.file',array('label' => false,'type' => 'file'));?>
generating this line on the view:

Anything suspicious? Does anyone get this componenet working? Please share your knowledge. Thanks.

Some issues I've noticed so far trying out v0.42 on cake 1.2.6598 1.2.0.6613

I had to comment out lines 76-80 in the component as it was causing the app to timeout (possibly because modelClass isn't there, I believe it's modelNames (array))
I also had to change the php opening tag from the short tag format in the element on line 50 as I purposely don't enable short tag support to maximize code portability.

I had better luck debugging with die(debug($foo)) because of timeouts and file permissions that are probably due to my particular setup - but if anyone else is having timeouts and issues of this behaviour not working, try replacing trigger_errors with dies/debugs and adding dies/debugs before function return values where you think you might be having problems. I'm sure there's more elegant ways, but I did get this working finally for my app using .42 and 6613 on php 5.2.5 with apache 2.2.8, thanks David for all your efforts.

This is a warning thrown by the component if the model corresponding to the controller doesn't have the Attachment behavior in it's $actsAs array.

So let's say you have an Examples Controller with the Attachment Component loaded. Then you should also have an Example Model with the Attachment Behavior in it's $actsAs array.

hope that helps
I'm getting this error as well while I have 'Attachment' defined in both my components and and actsAs arrays.

I'm getting this error as well while I have 'Attachment' defined in both my components and and actsAs arrays.

The error will disappear when you download the nightly build and the 0.42 attachments. At least this worked for me.

Later, I got another error about PATHINFO_FILENAME but fixed it with upgrading php from 5.1.x to 5.2.x ( http://www.php.net/pathinfo )

I hope tihs will help you.

However, I still get an error about the temporary file 'has incorrect mimetype'.

As far as I can see, the error occurs in the
private function setTemporary($data,$error = null)
when making the call:
$this->check('tmp',array('extension')

Any help will be appreciated.

When loading thumbs from cache, I get massive page loads. to fix this i changed the following in the component

} elseif($cached = Cache::read($cacheKey,$this->cacheConfig)) {
$content = $cached;


to

} elseif($cached = Cache::read($cacheKey,$this->cacheConfig)) {
echo $content = $cached;
exit;


Thank you for this great piece of code!

I'm new to cakephp (actually, this is my first comment on the bakery and all, so please be kind ^_^)
I had an hard time while getting this behaviour to work on windows;
i'll share my experience hoping it can help someone else in the same situation.

The test were made using Win XP and Wamp2.0 -> http://www.wampserver.com/en/ with Apache 2.2.8 - MySQL 5.0.51b - PHP 5.2.6

attm 0.42, cake core to the latest svn (6816), app beta 6311


I spent some time tracking down my fault (other than using Windows). And the fault was.. I was using Windows.
If you don't want to run every time into the "Could not determine temporary file type" error, you'll have to take into consideration some path issues.

First, in {APP}/models/behaviours/attachment.php:
edit the default settings property:

private $defaultSettings = array( [..]
adding the right path to the allowed paths key, i.e:

'allowPaths' => array(TMP,'/tmp/',WWW_ROOT,'C:\\wamp\\tmp\\', 'C:/wamp/tmp/' )

This will save you an headhache ^_^

check your php.ini for the actual value of tmp - the Transfer class (in {APP}/Vendors) will then able to get the right temp uploaded file.

Second - After that, you'll notice, there is another problem.
The isLocalFile method (and therefore the isUploadedLocalFile method too) always evaluates to false.
The problem lies -i think- in the
parse_url($file)
function used in conjunction with the windows filesystem.
$url['scheme'] shoul be set to "file" (or null) for a local file.

In my case (try pr($url['scheme'])..) it was set to "C", because the tmp file is something like "C:\wamp\tmp\phpB11.tmp" !!!
(the drive letter before the colon is interpreted as the scheme)

I didn't dig deeply into the issue; just came up with the following workaround.
$url['scheme'] may be set to any drive (A: trough Z:), so just accept any single char string ^_^:

(in yourAppName/vendors/transfer.php):

    private function isLocalFile($file)
    {
        if(!is_string($file)) {
            //pr("filename is not a string". $file);
            return false;
        }
        
        $url = parse_url($file);
        //pr("url " . $url . "<br>". $file);
        
        if(isset($url['scheme']) && $url['scheme'] == 'file') {
            $file = $file = substr($file,7);
            //pr("scheme = file (substr 7)" . $file);

        // CHANGE THIS /////////////////
        //} elseif(!empty($url['scheme'])) {
        
        // TO:        //////////////
        } elseif(!empty($url['scheme']) && strlen($url['scheme']) != 1) {
            
            //pr("url scheme " . $url['scheme'] );
            //pr("scheme not = file " . $file);
            return false;
        }
         
        if(is_file($file) && is_executable($file)) {
            $this->errors[] = 'File is a local file but executable bit is set.';
            //pr('is executable'. $this->errors);
            return false;
        }
        
        return true;
    }

You're almost done. The temp files now pass the checks on windows too :)
But there is anoter problem

Third - make sure the PECL file_info extension is properly installed.
AND bug free..
see:
http://pecl.php.net/bugs/bug.php?id=7555

It's possible to set the second parameter of the finfo_open function, i.e.

finfo_open(FILEINFO_MIME,"C:/wamp/php/extras/magic"

for every occurrence in {APP}/vendors/mimetype.php,
but you don't want to have a class that would'nt work when you commit to the (probabably linux) production server, don't you?

THE solution that worked for me is there:
http://www.php.net/manual/en/fileinfo.installation.php

Note:
it seems that the php.ini setting

mime_magic.magicfile = "C:\wamp\bin\php\extras\magic.mime"

does not work on windows!

You HAVE TO set a (system) ENVIRONEMENT VARIABLE called MAGIC -
set it to
C:\wamp\bin\php\extras\magic
if the main magic database file is
C:\wamp\bin\php\extras\magic.mime

Restart the system (yes, we are on windows)
And you're done!


---
Other


You may also need/want to bypass the mimetype check, by setting the second parameter ($skip) to
array([..], 'mimetype')
on every call of the $Transfer->check method.
I.e.:

if(!$this->check('tmp',array('extension','mimetype'))) {
                return false;
            } 


I had another problem with the security component; if included it gave me errors while updating a record with more than one attached file. (maybe this is not a windows only issue)


Finally, I think it's a good practice to set the configuration for the behaviour (see the examples provided with the behaviout)


'Attachment' => array(
            'base' => '{APP}webroot',
            'dirname' => 'files/organizations',
            'filename' => '{BASENAME}',
            'overwriteExisting' => true,
            'maxSize' => '8M',
        ),

Thanks again David,
and to windows users, happy baking (with attachments)!

Thank you very much, David! This makes attaching any kind of File to a Record so much easier. I don't have to worry about this or that.

I like the flexibility and the implementation, too.

I'm excited to see how development continues.

a little bug in attachment.php behaviour
line 511 should be:
if(!$this->Attachment->delete($attachment['id'])) {

Sorry for my stupid/basic question, but I can't figure out how to use this behaviour if I want only to deal with checks on extensions and not on mime-type. I have to install cakephp on a hosting service not supporting finfo nor mime_content_type. What can I do??

Hi,

I want to use it with AjaxHelper. so that I have tried it using $ajax->form() and $ajax->submit(). but it doesn't work.

Normally(w/o Ajax) its working gr8.

Please reply ASAP.

Hi,

I want to use it with AjaxHelper. so that I have tried it using $ajax->form() and $ajax->submit(). but it doesn't work.

Normally(w/o Ajax) its working gr8.

Please reply ASAP.
In reply to kapcee above: You can't (easily) send file using Ajax. There are work arounds, but you won't be able to do it by simply using the AjaxHelper. This is not a bug about CakePHP or this behavior, but a browser "feature". Just google 'send file ajax' and you will understand.

Hi,

I am trying to make Attachmet behavior work on my website with no success. When I try the Example Movie Module,Controller and Views I get
Fatal error: Class 'Imagick' not found in /home/....
I checked that ImageMagick is installed on my shared-web server.

Well I am kind of lost here. Please help me.
First of all installation steps are different in README file and here. So which document I have to follow for the working example.

For example in README file Step 4 and 5 aren't mentioned here.
Step 4
------
Add default media plugin configuration by appending this line to your app's core.php:
...
require_once(APP.'plugins'.DS.'media'.DS.'config'.DS.'core.php');

Step 5
------
Define MEDIA and MEDIA_URL in your config/bootstrap.php or config/core.php:
...
define('MEDIA','/path/to/your/app/webroot/media/');
define('MEDIA_URL','media/');


as well as var $actsAs = array('Attachment'); isn't mentioned in READ ME file.
Could anyone provide a simple example for uploading PDF, AVI , JPEG files.

Thanks for your efforts.

Hi,

I have currently got as far as deploying the latest release of this on on 1.2 final and am currently successfully adding the fields 'model`, `group`, `alternative`, `cattery_id`, `modified`, `created`, but no file is uploaded and no dirname, basename or checksum are being written, with no errors being reported.
Any suggestions would be much appreciated.

Many thanks in advance,

Jon

Great package BTW...

But has anybody had the issue where xls files are recognised as application/msword mime type? It doesn't seem to be the code it seems to be how the magic.mime/php deals with mime types. Any ideas anyone?

BTW it happens on my Windows XP test server (WAMP) and my Centos 5.2 production server.

hi,

great stuff, thanks.

I just have a problem: MyModel hasOne Attachment, and i have a view with MyModel Fields plus the Attachment File Upload.

View Template:

echo $form->input('Attachment.file',array('type'=>'file'));
If the file upload is empty MyModel gets saved and the Validation of Attachment isn't triggered (Validation of Attachment works fine when a file is selected)

How can i adjust my models to require a file to be attached and error on empty fileUpload?