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