Change Profile Attribute 'graduation' to 'yearInSchool'

My preference is that all one-liner ifs and fors have brackets.  I find it makes the code more readable.  I agree it's not critical, and even as far as formatting goes it's not a huge deal.  I'm not saying you have to change it, as this probably doesn't amount to an ant-hill as far as java standards go, but I still might express my preference from time to time.  You can do what seems best.

Unused imports is less important to me as IDEs auto-flatten the list of imports, so largely I don't even notice if there are unused or not.  I do try to clean those up when I see them, thought it's not worth it to go through the whole project searching for them.