Bug Tracker

Ticket #77 (closed bug: fixed)

Opened 8 years ago

Last modified 7 years ago

JS 'warnings' generated in firefox

Reported by: alistair.potts@… Owned by:
Priority: undecided Milestone:
Component: ajax Version:
Keywords: "" Cc: ""
Blocking: Blocked by:

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

Change History

comment:1 Changed 8 years ago by anonymous

  • Component changed from ajax to core

comment:2 Changed 8 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.

comment:3 Changed 8 years ago by john

  • Version set to 1.0

comment:4 Changed 8 years ago by hultmann

  • Status changed from closed to reopened
  • Resolution worksforme deleted

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

comment:5 Changed 8 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.

comment:6 Changed 8 years ago by anonymous

  • Priority set to blocker
  • Resolution set to invalid
  • Status changed from reopened to closed
  • Component set to ajax
  • Type set to bug

SPAMPOLICE

comment:7 Changed 8 years ago by john

  • Priority set to blocker
  • Resolution set to fixed
  • Status changed from reopened to closed
  • Component set to ajax
  • Type set to bug
Note: See TracTickets for help on using tickets.