fix silly mistake of not assigning values in constructor