should this be a property instead of method?