Careful with your indentation.  Also, having all this code in one line makes it really hard to read.