if this is the last attribute, it shouldn't append a "," no?