minor logic error all set now