Merge pull request #45 from CS2103AUG2016-W10-C1/fix/access

Restore access modifiers for EditCommand, TaskMan & Task files
CS2103AUG2016-W10-C1 · Oct 19, 2016 · b34c510 · b34c510
2 parents 19dc846 + 683de70
commit b34c510
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/seedu/taskman/logic/commands/CompleteCommand.java b/src/main/java/seedu/taskman/logic/commands/CompleteCommand.java
@@ -59,7 +59,7 @@ public CommandResult execute() {
 	                    activityToComplete.getSchedule().get(),
 	                    activityToComplete.getFrequency().get());
 	        	try {
-					task.status = new Status(STATUS_COMPLETE);
+					task.setStatus(new Status(STATUS_COMPLETE));
 				} catch (IllegalValueException e) {
 					// TODO Auto-generated catch block
 					e.printStackTrace();

diff --git a/src/main/java/seedu/taskman/logic/commands/EditCommand.java b/src/main/java/seedu/taskman/logic/commands/EditCommand.java
@@ -26,7 +26,7 @@ public class EditCommand extends Command {
     public static final String MESSAGE_SUCCESS = "Task updated: %1$s";
     public static final String MESSAGE_DUPLICATE_TASK = "A task with the same name already exists in TaskMan";
 
-    protected final ArgumentContainer argsContainer;
+    private final ArgumentContainer argsContainer;
     private Activity beforeEdit;
     private Activity afterEdit;
     private Activity.ActivityType activityType;
@@ -123,7 +123,7 @@ private void initMembers(ArgumentContainer argsContainer) throws IllegalValueExc
         }
     }
 
-    protected static class ArgumentContainer {
+    private static class ArgumentContainer {
         public final int targetIndex;
         public String title;
         public String deadline;

diff --git a/src/main/java/seedu/taskman/model/TaskMan.java b/src/main/java/seedu/taskman/model/TaskMan.java
@@ -149,7 +149,7 @@ public boolean completeActivity(Activity key) throws UniqueActivityList.Activity
        			key.getSchedule().get(),
        			key.getFrequency().get()
        			);
-        	task.status = new Status("complete");
+        	task.setStatus(new Status("complete"));
 			this.addActivity(new Activity(task));
             return true;
         } else {

diff --git a/src/main/java/seedu/taskman/model/event/Task.java b/src/main/java/seedu/taskman/model/event/Task.java
@@ -14,7 +14,7 @@
 public class Task extends Event implements ReadOnlyTask, MutableTagsEvent {
 
     private Deadline deadline;
-    public Status status;
+    private Status status;
 
     public Task(@Nonnull Title title, @Nonnull UniqueTagList tags,
                 @Nullable Deadline deadline,