string operation fixed