{"action":"opened","issue":{"url":"https://api.github.com/repos/se-edu/addressbook-level4/issues/284","repository_url":"https://api.github.com/repos/se-edu/addressbook-level4","labels_url":"https://api.github.com/repos/se-edu/addressbook-level4/issues/284/labels{/name}","comments_url":"https://api.github.com/repos/se-edu/addressbook-level4/issues/284/comments","events_url":"https://api.github.com/repos/se-edu/addressbook-level4/issues/284/events","html_url":"https://github.com/se-edu/addressbook-level4/issues/284","id":204785450,"number":284,"title":"Inconsistent parameter name in Parser method","user":{"login":"PierceAndy","id":8639005,"avatar_url":"https://avatars.githubusercontent.com/u/8639005?v=3","gravatar_id":"","url":"https://api.github.com/users/PierceAndy","html_url":"https://github.com/PierceAndy","followers_url":"https://api.github.com/users/PierceAndy/followers","following_url":"https://api.github.com/users/PierceAndy/following{/other_user}","gists_url":"https://api.github.com/users/PierceAndy/gists{/gist_id}","starred_url":"https://api.github.com/users/PierceAndy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PierceAndy/subscriptions","organizations_url":"https://api.github.com/users/PierceAndy/orgs","repos_url":"https://api.github.com/users/PierceAndy/repos","events_url":"https://api.github.com/users/PierceAndy/events{/privacy}","received_events_url":"https://api.github.com/users/PierceAndy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-02-02T05:24:09Z","updated_at":"2017-02-02T05:24:09Z","closed_at":null,"body":"In [Parser](https://github.com/se-edu/addressbook-level4/blob/master/src/main/java/seedu/address/logic/parser/Parser.java#L48), user input strings are split into `commandWord` and `arguments`, where `commandWord` stores the command string such as \"add\" or \"edit\", and `arguments` stores everything else that comes after `commandWord`.\r\n\r\n[`Parser:parseIndex`](https://github.com/se-edu/addressbook-level4/blob/master/src/main/java/seedu/address/logic/parser/Parser.java#L280-L292) takes the string index extracted from `arguments`, tries to convert it into an integer, and returns `Optional<Integer>`.\r\n\r\nHowever, the method signature is currently `parseIndex(String command)`, which is not accurate as it does not deal with the `commandWord`. It could be changed to `parseIndex(String index)`."}}
{"actor":{"display_login":"PierceAndy"}}