I don't like the look of this bit:

active != "0"

isn't it supposed to be "<>" for not equals in SQL?

also, use brackets when you've got complicated ANDs and ORs, it'll help you see it better, like:

SELECT * FROM users WHERE 
  (state = "FL" OR state = "NY")
AND 
 (active <> 0)
[download]

and are you supposed to be quoting number values?

I've now reached the end of my SQL knowledge, but is there a difference between active=0 and active="0"? If it's the contents of a number-type field?
--