Try adding this to your action

static final Pattern pattern = Pattern.compile("^.+\\.((csv)|(xls))$", 
Pattern.CASE_INSENSITIVE);

In your validateUpload method have something like

        Matcher matcher = pattern.matcher(fileName);
        if (!matcher.matches()) {
            addActionError("File type not allowed");
            return;
        }

Remove |(xls) if you dont want to allow xls files


--- On Fri, 2/27/09, Richard Gundersen <r.gunder...@salford.ac.uk> wrote:

> From: Richard Gundersen <r.gunder...@salford.ac.uk>
> Subject: RE: String manipulation
> To: "Struts Users Mailing List" <user@struts.apache.org>
> Date: Friday, February 27, 2009, 11:50 AM
> Hmmm, no that also prevents genuine csv files from being
> selected too. 
> 
> Also investigating the accept attribute on the s:file tag
> to see if that
> will help...
> 
> Any help appreciated.
> 
> -----Original Message-----
> From: Richard Gundersen [mailto:r.gunder...@salford.ac.uk] 
> Sent: 27 February 2009 10:27
> To: Struts Users Mailing List
> Subject: RE: String manipulation
> 
> Oh, got it
> 
> @FieldExpressionValidator(expression="(accountFileFileName.endsWith('.cs
> v')", message="",
> key="validate.invalidFilename",
> fieldName="accountFileFileName")
> 
> Forgot you could just call methods in OGNL :)
> 
> -Richard
> 
> -----Original Message-----
> From: Richard Gundersen [mailto:r.gunder...@salford.ac.uk] 
> Sent: 27 February 2009 10:16
> To: Struts Users Mailing List
> Subject: String manipulation
> 
> Hi
> 
> I'm trying to validate that a filename has a .csv
> extension. I would
> like to use @FieldExpressionValidator on the field but I
> can't get the
> syntax right, and searching the docs I can't find much
> on String
> manipulation in OGNL. 
> 
> Could anyone help me out please?
> 
> Thanks
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
> For additional commands, e-mail:
> user-h...@struts.apache.org
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
> For additional commands, e-mail:
> user-h...@struts.apache.org
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
> For additional commands, e-mail:
> user-h...@struts.apache.org


      

---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
For additional commands, e-mail: user-h...@struts.apache.org

Reply via email to