Re: app-fileservices no files in Request?

Stephen Davidson <[email protected]>
Newsgroups gmane.comp.java.keel.user
Message-ID <[email protected]>
Hi Nathan.
Nathan Kontny wrote:

>Stephen, 
>
>So I see that the StrutsClientConnector is now instead creating instances of
>the
>BinaryWrapper in the request.  I changed the FileUploadModel to look for this
>wrapper instead of the "file_data" key.  Attached is the patch if you are
>interested.  
>
>I also negated the logic of the file_type map, as wouldn't it be easier for
>users to specify the files that should be allowed (jpg, png, etc.) than have to
>specify every file that shouldn't be allowed?  For example, the patch now has
>this: 
>  
>
Depneds on what you are doing.  The guys (over at Platinum solutions) 
that originally wrote this need to deny certain file types.  And some 
other people that use Keel also need to deny certain file types.  This 
is not unusual (ever seen Samba in action?).  You are also correct in 
that sometime people only want certain filetypes uploaded (although you 
are the first case in question to submit a patch).  In my case, there 
were no restrictions on files to be uploaded, which is why I never 
"bothered" with this code.  I will have a look at you patch when 
addressing the app-fileservices issues tonight.

>-Nate
>
>
>Quoting Stephen Davidson <[email protected]>:
>
>  
>
>>Nathan Kontny wrote:
>>    
>>
>>>I am using the latest app-fileservices and pretty close to the latest of
>>>everything else from CVS.  When I use the file.upload-example, I see that
>>>inside the FileUploadModel during the example it doesn't find any
>>>      
>>>
>>"file_data"
>>    
>>
>>>request parameters.  ( from the code: if(oneKey.indexOf("_file_data_") >
>>>      
>>>
>>0){ ).
>>    
>>
Hmm, that answers earlier question.  Oh, FileUploadModel.  Missed 
something somewhere, did I?  I will have a look today, but I think what 
is happening is that my Model was interecepting this call, and resetting 
the parameter name for FileUploadModel.  I really do need to do some 
docs on this, including why everything behaves the way it does.

I will send you a copy this evening.

>>> 
>>>
>>>I am just using the example just as it comes out of the box, with a jpg of
>>>      
>>>
>>27KB.
>>    
>>
>>> Is this app working for anyone else right now? 
>>>
>>>-Nate
>>>      
>>>
Yes, me.   I will put together some examples tonight....

Regards,
Steve

-- 
Java/J2EE Developer/Integrator
Chair, Dallas/FortWorth J2EE Sig
214-724-7741
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.