add attr_accessible
add exception
fix