jQuery: The Write Less, Do More JavaScript Library

Ticket #77 (closed bug: fixed)

Opened 2 years ago

Last modified 1 year ago

JS 'warnings' generated in firefox

Reported by: alistair.potts@… Assigned to: anonymous
Type: bug Priority: blocker
Milestone: Component: ajax
Version: Keywords: ""
Cc: "" Needs:

Description

This isn't a bug - but it's really annoying! The firefox javascript console is really useful for ensuring you're writing good code - but at the moment jquery triggers 4 warnings. This is annoying because you can't immediately tell whether it's your own code or jquery, which means firing up the errors console etc. etc. I believe a lot of web developers use the js console, so I would suggest the trivial fixes are worthwhile.

The warnings are (rev 110) "function does not always return a value" * jQuery.hasWord * handle event * function jQuery

"undefined property" ret[0] (line 602) - just put a check in first to see if ret has length.

Alistair

Attachments

Change History

Changed 2 years ago by anonymous

  • component changed from ajax to core

Changed 2 years ago by john

  • status changed from new to closed
  • resolution set to worksforme

I can't duplicate this - I think it's been fixed in SVN.

Changed 2 years ago by john

  • version set to 1.0

Changed 2 years ago by hultmann

  • status changed from closed to reopened
  • resolution deleted

Warnings appear when javascript.options.strict=true (about:config)

Changed 2 years ago by john

  • status changed from reopened to closed
  • resolution set to fixed

Ok, I went through and fixed all the problems. These have all been fixed in SVN rev 440.

Changed 2 years ago by anonymous

  • priority set to blocker
  • status changed from reopened to closed
  • component set to ajax
  • resolution set to invalid
  • type set to bug

SPAMPOLICE

Changed 2 years ago by john

  • priority set to blocker
  • status changed from reopened to closed
  • component set to ajax
  • resolution set to fixed
  • type set to bug
Note: See TracTickets for help on using tickets.