fixed setvalue