use self instead of this
fix style
update task details