Merge pull request #203 from bennchong/master

Raised Issues

docs/DeveloperGuide.adoc

@@ -551,7 +551,7 @@ thrown with the relevant message to the user.
 .Interactions between the UI Component and Security Component for the login Command
 image::SecurityLoginSequenceDiagram.png[width="800"]
 
-The figure above shows the steps when login is called after Figure 19 has completed (since login is a command too).
+The figure above shows the steps when login is called after parsing the command (since login is a command too).
 
 AppUsers contains an array of
 AccountCredentials, which has the attribute of Username and Password, and methods such isPasswordValid(). Here it also
@@ -1052,7 +1052,7 @@ These instructions only provide a starting point for testers to work on; testers
 
 . Logging in with a test account
 
-.. Test case: `login u\test pw\test` +
+.. Test case: `login u/test pw/test` +
    Expected: The login is successful and the main GUI appears. The `Me` panel should show a person card panel with `test` in the name field.
 
 . Registering a new account
@@ -1076,7 +1076,7 @@ All following sections assume that the user is currently logged into FreeTime.
 
 . Editing your details
 
-.. Test case: `edit p/91234567 e/johndoe@example.com t/CS2101` +
+.. Test case: `edit p/91234567 e/johndoe@example.com g/CS2101` +
    Expected: The details of the user in the `Me` panel is updated to specified details.
 
 . Showing command history

docs/UserGuide.adoc

@@ -94,7 +94,7 @@ CAUTION: You are required to to enter a password that contains more or equal to
 To ensure that your account is well-protected, do use a password that is not common.
 
 === Registering: `register` (`re`)
-Command: `register u/[Username] pw/[Password] e/[Email] p/[MobilePhone] a/[Address]`
+Command: `register u/USERNAME pw/PASSWORD e/[EMAIL] p/[MOBILEPHONE] a/[ADDRESS]`
 
 Examples:
 

docs/team/bennchong.adoc

@@ -40,9 +40,6 @@ Thus you are freed from the pains of finding a common free slot in huge groups.
 == Summary of contributions
 In this section, I would be briefly telling you about my contributions to this project, and any interesting points that you could find useful.
 
-
-GIVE THE REASON WHY. USE LONGER SENTENCE BELOW
-
 * *Major enhancement*: Added *the ability to authenticate users* within _FreeTime_
 ** What it does: Allows the users to have unique access and control to his own timetable
 ** Justification: To be able to prevent others from editing your own timetable, and to facilitate different privacy levels.

src/main/java/seedu/address/logic/commands/LogoutCommand.java

@@ -7,7 +7,7 @@
 import seedu.address.security.SecurityAuthenticationException;
 
 /**
- * Lists all persons in the address book to the user.
+ * Logout the user
  */
 public class LogoutCommand extends Command {
 

src/main/java/seedu/address/logic/commands/RegisterCommand.java

@@ -31,15 +31,15 @@ public class RegisterCommand extends Command {
             + PREFIX_ADDRESS + "ADDRESS "
             + "[" + PREFIX_TAG + "TAG]...\n"
             + "Example: " + COMMAND_WORD + " "
-            + PREFIX_USERNAME + "John Doe "
+            + PREFIX_USERNAME + "John Goe "
             + PREFIX_PASSWORD + "testpassword "
             + PREFIX_PHONE + "98765432 "
             + PREFIX_EMAIL + "johnd@example.com "
             + PREFIX_ADDRESS + "311, Clementi Ave 2, #02-25 "
             + PREFIX_TAG + "CS2113T ";
 
     public static final String MESSAGE_SUCCESS = "New person registered: %1$s";
-    public static final String MESSAGE_DUPLICATE_PERSON = "This person already exists in the address book";
+    public static final String MESSAGE_DUPLICATE_PERSON = "This person already exists in FreeTime";
 
     private final Person toAdd;
     private final String password;

src/main/java/seedu/address/logic/parser/AddressBookParser.java

@@ -31,7 +31,6 @@
 import seedu.address.logic.commands.UndoCommand;
 import seedu.address.logic.commands.UnfriendCommand;
 import seedu.address.logic.parser.exceptions.ParseException;
-import seedu.address.security.SecurityAuthenticationException;
 
 /**
  * Parses user input.
@@ -70,12 +69,11 @@ public CommandsEnum parseCommand(String userInput) throws ParseException {
     }
 
     /**
-     * //TODO
-     * @return
-     * @throws ParseException
-     * @throws SecurityAuthenticationException
+     * Parses the arguments
+     * @return the command to be executed
+     * @throws ParseException when arguments are not valid
      */
-    public Command parseCommandArguments() throws ParseException, SecurityAuthenticationException {
+    public Command parseCommandArguments() throws ParseException {
 
         switch (commandWord) {
         case RegisterCommand.COMMAND_WORD:

src/main/java/seedu/address/model/ModelManager.java

@@ -74,7 +74,6 @@ public TimeTable getTimeTable() {
 
     /** Raises an event to indicate the model has changed */
     private void indicateAddressBookChanged() {
-        //TODO Check whether this actually works when modifying your own data.
         if (user != null) {
             matchUserToPerson(user.getName().toString());
         }
@@ -268,7 +267,6 @@ public void clearUser() {
 
     @Override
     public User getUser() {
-        //TODO Can you do this? Must you create a new object to be returned instead?
         return this.user;
     }