fixed unvared variables