Bug Tracker

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3666 closed enhancement (fixed)

Remove dead code from $(fn) shortcut for $(document).ready(fn)

Reported by: geary Owned by: flesler
Priority: trivial Milestone: 1.3
Component: core Version: 1.2.6
Keywords: Cc: geary
Blocked by: Blocking:


The code for the $(fn) shortcut is:

return jQuery( document )[ jQuery.fn.ready ? "ready" : "load" ]( selector );

It's unclear what the intent of this code was, but the jQuery.fn.ready test has no effect. jQuery.fn.ready always exists, so the code is the equivalent of:

return jQuery( document ).ready( selector );

That also matches people's expectation that $(fn) is the exact equivalent of $(document).ready(fn).

The attached patch removes this dead code.

Attachments (1)

readyshortcut.patch (402 bytes) - added by geary 8 years ago.
Patch to remove dead code from $(fn)

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by geary

Patch to remove dead code from $(fn)

comment:1 Changed 8 years ago by flesler

  • Cc geary added
  • Owner set to flesler
  • Status changed from new to assigned

comment:2 Changed 8 years ago by flesler

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

Applied at [5997]. Thanks.

comment:3 Changed 8 years ago by flesler

  • Type changed from bug to enhancement
Note: See TracTickets for help on using tickets.