fixed attr_reader