modified but not working