{"action":"opened","issue":{"url":"https://api.github.com/repos/CMPUT301F18T20/MedicalPhotoRecord/issues/60","repository_url":"https://api.github.com/repos/CMPUT301F18T20/MedicalPhotoRecord","labels_url":"https://api.github.com/repos/CMPUT301F18T20/MedicalPhotoRecord/issues/60/labels{/name}","comments_url":"https://api.github.com/repos/CMPUT301F18T20/MedicalPhotoRecord/issues/60/comments","events_url":"https://api.github.com/repos/CMPUT301F18T20/MedicalPhotoRecord/issues/60/events","html_url":"https://github.com/CMPUT301F18T20/MedicalPhotoRecord/issues/60","id":375687808,"node_id":"MDU6SXNzdWUzNzU2ODc4MDg=","number":60,"title":"User, Provider and Patient should throw an exception in their constructors about invalidUserID","user":{"login":"modelarious","id":22161638,"node_id":"MDQ6VXNlcjIyMTYxNjM4","avatar_url":"https://avatars1.githubusercontent.com/u/22161638?v=4","gravatar_id":"","url":"https://api.github.com/users/modelarious","html_url":"https://github.com/modelarious","followers_url":"https://api.github.com/users/modelarious/followers","following_url":"https://api.github.com/users/modelarious/following{/other_user}","gists_url":"https://api.github.com/users/modelarious/gists{/gist_id}","starred_url":"https://api.github.com/users/modelarious/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/modelarious/subscriptions","organizations_url":"https://api.github.com/users/modelarious/orgs","repos_url":"https://api.github.com/users/modelarious/repos","events_url":"https://api.github.com/users/modelarious/events{/privacy}","received_events_url":"https://api.github.com/users/modelarious/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"tnguyen10","id":35233304,"node_id":"MDQ6VXNlcjM1MjMzMzA0","avatar_url":"https://avatars0.githubusercontent.com/u/35233304?v=4","gravatar_id":"","url":"https://api.github.com/users/tnguyen10","html_url":"https://github.com/tnguyen10","followers_url":"https://api.github.com/users/tnguyen10/followers","following_url":"https://api.github.com/users/tnguyen10/following{/other_user}","gists_url":"https://api.github.com/users/tnguyen10/gists{/gist_id}","starred_url":"https://api.github.com/users/tnguyen10/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tnguyen10/subscriptions","organizations_url":"https://api.github.com/users/tnguyen10/orgs","repos_url":"https://api.github.com/users/tnguyen10/repos","events_url":"https://api.github.com/users/tnguyen10/events{/privacy}","received_events_url":"https://api.github.com/users/tnguyen10/received_events","type":"User","site_admin":false},"assignees":[{"login":"tnguyen10","id":35233304,"node_id":"MDQ6VXNlcjM1MjMzMzA0","avatar_url":"https://avatars0.githubusercontent.com/u/35233304?v=4","gravatar_id":"","url":"https://api.github.com/users/tnguyen10","html_url":"https://github.com/tnguyen10","followers_url":"https://api.github.com/users/tnguyen10/followers","following_url":"https://api.github.com/users/tnguyen10/following{/other_user}","gists_url":"https://api.github.com/users/tnguyen10/gists{/gist_id}","starred_url":"https://api.github.com/users/tnguyen10/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tnguyen10/subscriptions","organizations_url":"https://api.github.com/users/tnguyen10/orgs","repos_url":"https://api.github.com/users/tnguyen10/repos","events_url":"https://api.github.com/users/tnguyen10/events{/privacy}","received_events_url":"https://api.github.com/users/tnguyen10/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2018-10-30T20:51:36Z","updated_at":"2018-10-30T20:51:36Z","closed_at":null,"author_association":"COLLABORATOR","body":"Currently we have this:    \r\n\r\n***************START*************************\r\npublic User(String Userid, String email, String phoneNumber) {\r\n        this.UserID = Userid;\r\n        this.email = email;\r\n        this.phoneNumber = phoneNumber;\r\n}\r\n\r\npublic User(String UserID) {\r\n        try {\r\n                this.setUserID(UserID);\r\n        } catch (NonNumericUserIDException e) {\r\n            //TODO: Handle exception\r\n        }\r\n}\r\n******************END******************\r\n\r\nWe need this:\r\n\r\n***************START*************************\r\npublic User(String Userid, String email, String phoneNumber) throws NonNumericUserIDException {\r\n        this.setUserID(Userid);\r\n        this.email = email;\r\n        this.phoneNumber = phoneNumber;\r\n}\r\n\r\npublic User(String UserID) throws NonNumericUserIDException {\r\n        this.setUserID(UserID);\r\n}\r\n******************END******************\r\n\r\n\r\nIt seems small, but then you have to go change the test cases that use these constructors. As well as fix how the constructors in Patient and Provider class are calling the super and handle exceptions in all their test cases."}}
{"actor":{"display_login":"modelarious"}}