fixing string compare