[TAPESTRY-339] LinkSubmit can cause browser back button problems

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.0.5
• Fix Version/s: 3.0.5
• Environment: Any
• Assignee: Jesse Kuhnert
• Reporter: Nick Westgate
• Created: 13/Jun/05 18:49
• Updated: 30/Apr/06 16:53
• Resolved: 29/Mar/06 00:56

Description

If LinkSubmit is mixed with other submit components in the same form, erroneous submits can occur from pages accessed via the browser back button. This happens because LinkSubmit's hidden variable is never reset.

Suggested fix (line marked +++) in LinkSubmit.java, renderComponent method:

// make sure the submit function is on the page (once)
if (cycle.getAttribute(ATTRIBUTE_FUNCTION_NAME) == null)
{
+++ body.addInitializationScript("document." + formName + "._linkSubmit.value = null;");
body.addBodyScript(
"function submitLink(form, elementId)

{ form._linkSubmit.value = elementId; if (form.onsubmit == null || form.onsubmit()) form.submit(); }

");
cycle.setAttribute(ATTRIBUTE_FUNCTION_NAME, this);
}

It would be great if the corrected LinkSubmit submission mechanism detail was factored out into a separate class, so that other submit components could be built using it. For example, Glen Stampoultzis's ButtonSubmit component.

Issue Links

• duplicates: TAPESTRY-678 LinkSubmit does not work correctly when the browser's back button is used and the form resubmitted normally