oops had a few silly mistakes - I can see where the problem now lies but am not sure how to fix it