Bug Tracker

Ticket #10701 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Problems with submit forms using submit function

Reported by: anonymous Owned by: dmethvin
Priority: low Milestone: 1.7.1
Component: event Version: 1.7
Keywords: Cc:
Blocking: Blocked by:

Description

On 1.7 form is posting even if onsubmit returns false. In 1.6.4 everithing was working good.

You can test it here:  http://jsfiddle.net/cxSmk/1/

When you click on "Submit" using 1.7 it will ask you to download bin file.

Firefox 7.0.1

Change History

comment:1 Changed 3 years ago by timmywil

  • Priority changed from undecided to low
  • Resolution set to invalid
  • Status changed from new to closed
  • Component changed from unfiled to event

"return false" only works if you use jQuery to bind the handlers.

 http://jsfiddle.net/timmywil/cxSmk/3/

comment:2 Changed 3 years ago by dmethvin

  • Status changed from closed to reopened
  • Resolution invalid deleted

Yeah it just needs an extra check in .trigger().

The reason this got through, however, is that we consider inline handlers a wontfix item and don't have any unit tests for them. I will fix it but also point out that inline handlers do not always work reliably in jQuery because they're not under our control.

comment:3 Changed 3 years ago by dmethvin

  • Owner set to dmethvin
  • Status changed from reopened to assigned
  • Milestone changed from None to 1.7.1

comment:4 Changed 3 years ago by Dave Methvin

  • Status changed from assigned to closed
  • Resolution set to fixed

Fix #10701, .preventDefault if an inline handler returns false.

Baby unicorns are slapped each time you use inline handlers, so do it sparingly.

Changeset: 90c907e8b65bc9d323863fc57389f2dca611292d

Note: See TracTickets for help on using tickets.