Bug Tracker

Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#7413 closed bug (invalid)

Browser inconsistencies return of jQuery.isEmptyObject

Reported by: rfernandes@… Owned by: rwaldron
Priority: low Milestone: 1.next
Component: core Version: 1.6b1
Keywords: Cc:
Blocked by: Blocking:

Description (last modified by snover)

  • Firefox 3.6 (and 4b7) returns:

jQuery.isEmptyObject("test") // false

  • IE7 returns:

jQuery.isEmptyObject("test") // true

Did not test other browsers or versions but can test if requested

As normal IE is the suspect here...

Change History (13)

comment:1 Changed 6 years ago by rwaldron

  • Owner set to rwaldron
  • Status changed from new to assigned

Confirmed!

comment:3 Changed 6 years ago by snover

  • Description modified (diff)

comment:5 Changed 6 years ago by john

  • Milestone changed from 1.5 to 1.4.5
  • Owner changed from rwaldron to john

Let's tackle this in 1.4.5 and use hasOwnProperty.

comment:6 Changed 6 years ago by john

  • Component changed from unfiled to core
  • Priority changed from undecided to low
  • Resolution set to fixed
  • Status changed from assigned to closed
  • Version changed from 1.4.3 to 1.4.4

Landed.

comment:7 Changed 6 years ago by john

  • Resolution fixed deleted
  • Status changed from closed to reopened

The patch causing problems in jQuery.param, backed out.

comment:8 Changed 6 years ago by snover

  • Owner changed from john to rwaldron
  • Status changed from reopened to assigned

comment:9 Changed 6 years ago by dmethvin

  • Milestone changed from 1.4.5 to 1.5

comment:10 Changed 5 years ago by timmywil

  • Milestone set to 1.6
  • Version changed from 1.4.4 to 1.6b1

Confirmed this bug is still present in 1.6b1. ​http://jsfiddle.net/timmywil/Q9nUZ/

comment:11 Changed 5 years ago by john

  • Milestone changed from 1.6 to 1.next

comment:13 Changed 5 years ago by rwaldron

  • Keywords needsdocs added
  • Resolution set to invalid
  • Status changed from assigned to closed

Best patch ever === no patch at all.

comment:14 Changed 5 years ago by addyosmani

  • Keywords needsdocs removed

Docs updated to emphasize $.isEmptyObject should always be called with a real object.

Note: See TracTickets for help on using tickets.