[STR-1674] Cancellation does not work as intended without form

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.0
• Fix Version/s: 1.4.0
• Component/s: Core
• Environment: Operating System: All; Platform: All
• Bugzilla Id: 22357
• Assignee: Paul Benedict
• Reporter: Patrick Dumontel
• Created: 13/Aug/03 00:49
• Updated: 07/Jul/07 15:05
• Resolved: 07/Jul/07 15:05

Description

If the action mapping does not contain a form bean name, e.g.:

<action path="/this/action"
type="action.class.name"/>

using isCancelled() in that action will always return false. The code in
org.apache.struts.action.RequestProcessor.processPopulate returns if the form
is null before setting the Globals.CANCEL_KEY attribute.

It seems to me that since a request parameter is used to detect the cancel
button (the return request.getParameter(Constants.CANCEL_KEY) != null line),
the setting of the request attribute should not depend on whether the action
mapping uses a form.