fix constructer