fixed attr assignment