It's to be expected that the validate won't be called, since it makes the same test as isCancelled() to check whether to run the form's validate.

The extra javascript for the onclick event stops any javascript validation from the validator plug-in from running. I doubt it's relevant here.

You really need to get into that RequestProcessor.processPopulate() method to debug it in any way you find to your liking. That's the crux of the matter here. You've got the 'cancelled' request parameter, but you haven't got the 'cancelled' request attribute, and it's that method that is meant to create the attribute.

Good luck,
Adam

Sashi Ravipati wrote:
Thanks for the attachment ,

but this is interesting

cancel button  works fine in Struts 1.1b2 and when I have
replaced the .jar files with Struts final 1.1 (Downloaded on July 3rd)
it
does not work.

Also the HTML version of the tags is different

in 1.1 Beta2 it is <input type="submit" name="org.apache.struts.taglib.html.CANCEL"
value="Cancel">


in Strits 1.1 final it is
<input type="submit" name="org.apache.struts.taglib.html.CANCEL"
value="Cancel" onclick="bCancel=true;">

Another problem I realised now is that the Validate method is also
called in case of cancel button which shouldnt happen.

Thanks



[EMAIL PROTECTED] 07/09/03 04:34PM >>>

I think the only way you will solve this is by downloading the source and putting some logging statnements in there. It's easy enough to use the struts source by putting the org/apache/struts/action/RequestProcessor.java in your own source directory and recompiling and deploying. I don't think you need the whole lot.


Here's my base action class for what its worth
Adam

Sashi Ravipati wrote:

I check the first thing u asked and it is same i.e
org.apache.struts.taglib.html.Constants.CANCEL_PROPERTY ==
org.apache.struts.taglib.html.CANCEL


and regarding the processPopulate(), I am not sure how I can do this.

I just downloaded the Struts 1.1 final again and tested but the result
is the same.


Could u send me the Code u have in your action class. I will try to

see


if I missed some thing.

Thanks




[EMAIL PROTECTED] 07/09/03 11:54AM >>>

Via logging is basically how I solved it - by figuring out where in struts I was going wrong.


What it means now is that RequestProcessor is not setting that

attribute


- you don't need to set it by hand, struts should.

It's just one little if clause. According to your logging, request.getParameter("org.apache.struts.taglib.html.CANCEL") has a value, right? The parameter is not null?

If the parameter is not null but the attribute is null, that means

that


you have either not got the same struts code as me (1.1 final) or your


RequestProcessor is not executing the processPopulate() method.

I would check 2 things:

that org.apache.struts.taglib.html.Constants.CANCEL_PROPERTY == "org.apache.struts.taglib.html.CANCEL" (i.e. that the parameter name

/


cancel button name in your HTML is correct)

and that RequestProcessor.processPopulate() is executing. This method also puts the data into the action form, so you could use that as a check unless you can grab the source code and put some logging in

there.


As you can tell I'm intrigued.
Adam

Sashi Ravipati wrote:


request.getAttribute(Globals.CANCEL_KEY) is returning null, so can I

set



it to not null and where Should I do this..

Can u explain how u solved ur problem..

Thanks




[EMAIL PROTECTED] 07/09/03 11:22AM >>>

I've solved my problem. Now you've checked the parameter, check to see


if the request.attribute is set with
request.getAttribute(Globals.CANCEL_KEY);

For some reason in the RequestProcessor it sets this attribute to true


when the request.parameter is present. Check it out. This is the one that the Action.isCancelled() checks. It has to be not null.


Adam


Sashi Ravipati wrote:



it says - Cancel

I don't know what I am missing which is making this tag not work..

Thanks







[EMAIL PROTECTED] 07/09/03 09:50AM >>>

Try logging what is in this cancel parameter (org.apache.struts.taglib.html.CANCEL) in the same place where you

are



calling isCancelled()

Actually I've just realised I'm suffering the same problem. I'll try

it




myself. Let me know what you find as well!

Adam

Sashi Ravipati wrote:




HTML is as shown below

<input type="submit" name="org.apache.struts.taglib.html.CANCEL"
value="Cancel" onclick="bCancel=true;">

[EMAIL PROTECTED] 07/09/03 09:21AM >>>

Sashi, what HTML does the tag produce?


Adam

Sashi Ravipati wrote:





I am trying this on Struts 1.1 final. Will this make any

difference..







[EMAIL PROTECTED] 07/08/03 05:11PM >>>

Odd, I can't reproduce that in 1.1-rc1. Unless you're extending something which might override the isCancelled(HttpServletRequest) method, or the html:cancel tag is not contained within an html:form tag, I can't think of anything else.

m

--- Sashi Ravipati <[EMAIL PROTECTED]> wrote:






This is how I have my cancel tag.

<html:cancel>
  <bean:message key="button.cancel"/>
</html:cancel>

[EMAIL PROTECTED] 07/08/03 02:27PM >>>

The complete content of your html:cancel tag is probably key to solving this problem.

m

--- Sashi Ravipati <[EMAIL PROTECTED]> wrote:






isCancelled(request) is always returning false
(Even when cancel button
is clicked)

Need some help...

Thanks








[EMAIL PROTECTED] 07/08/03 09:10AM >>>

I have a <html:cancel> in my jsp page. and in my action I have

if(isCancelled(request)){
return mapping.findForward("index"); }



But it is not working. What am I doing wrong here.

I







am using Struts 1.1
final

Thanks



__________________________________
Do you Yahoo!?
SBC Yahoo! DSL - Now only $29.95 per month!
http://sbc.yahoo.com



---------------------------------------------------------------------







To unsubscribe, e-mail:
[EMAIL PROTECTED]
For additional commands, e-mail:
[EMAIL PROTECTED]




__________________________________ Do you Yahoo!? SBC Yahoo! DSL - Now only $29.95 per month! http://sbc.yahoo.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail:

[EMAIL PROTECTED]




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]




--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]




--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]





---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to