fixing string comparison