fixed basic attack logic