Trying to fix passing formData from form to middleware

[gsarwohadi]

I needed to use 'formData' sent from the client and since the middleware isn't collecting it properly, I tried to make a fix.

This what I did:
Collect formData on 'field' formidable event in UploadHandler.prototype.post method to a req.fields object (created new object if undefined). Extend fileInfo with req.fields on 'end' event. So on upload.on("end", function (fileInfo)), I can have the extra property from formData in fileInfo.

Example use:

In the client:

<form id="formupload" action="/temp" method="post" enctype="multipart/form-data">
    <input type="hidden" name="user_id" value="some user id">
    <input id="fileupload" type="file" name="files[]" data-url="/temp" multiple>
</form>

In the server:

upload.on("end", function (fileInfo)
    {
        var filemanager = upload.fileManager({
            targetDir: __dirname + "/public/userfiles/" + fileInfo.user_id,
            targetUrl: "/userfiles/" + fileInfo.user_id
        });
        filemanager.move(fileInfo.name, "", function (err, result)
        {
            console.dir(result);
        })
    });

[tonyspiro]

Has this been added to the master yet? I'm running into the same issue. Thanks.

[tonyspiro]

Nevermind I just copied your version and it works great. Thanks!

[benmj]

+1

[aguidrevitch]

Merged in manually in 0.1.2 and published to npmjs.org. Please let me know if the issue is solved.

[joelyle]

I don't see this fix in 0.1.4

[aguidrevitch]

I've committed the code into 0.1.5 to address your issue. Now, each event handler gets request and response objects in addition to fileInfo. In your case you can use something like:

upload.on("end", function (fileInfo, req, res)
    {
        var filemanager = upload.fileManager({
            targetDir: __dirname + "/public/userfiles/" + req.param('user_id'),
            targetUrl: "/userfiles/" + req.param('user_id')
        });
        filemanager.move(fileInfo.name, "", function (err, result)
        {
            console.dir(result);
        })
    });

Please let me know if that works for you

[ghost]

Unless i'm missing something vital, then this isn't working at all. Regardless of what i do, then reg.params gives an empty object.