diff --git a/src/main/java/seedu/waddle/logic/commands/AddCommand.java b/src/main/java/seedu/waddle/logic/commands/AddCommand.java index 71b307666c8..3f03420fab6 100644 --- a/src/main/java/seedu/waddle/logic/commands/AddCommand.java +++ b/src/main/java/seedu/waddle/logic/commands/AddCommand.java @@ -57,6 +57,7 @@ public CommandResult execute(Model model) throws CommandException { } model.addItinerary(toAdd); + System.out.println("number of days in a newly created itinerary with duration: " + this.toAdd.getDays().size()); return new CommandResult(String.format(MESSAGE_SUCCESS, toAdd)); } diff --git a/src/main/java/seedu/waddle/logic/commands/EditCommand.java b/src/main/java/seedu/waddle/logic/commands/EditCommand.java index f7b25357b13..b0c0be52899 100644 --- a/src/main/java/seedu/waddle/logic/commands/EditCommand.java +++ b/src/main/java/seedu/waddle/logic/commands/EditCommand.java @@ -78,6 +78,7 @@ private static Itinerary createEditedItinerary(Itinerary itineraryToEdit, Date updatedStartDate = editItineraryDescriptor.getStartDate().orElse(itineraryToEdit.getStartDate()); ItineraryDuration updatedDuration = editItineraryDescriptor.getDuration() .orElse(itineraryToEdit.getDuration()); + System.out.println("the updated duration is: " + updatedDuration); People updatedPeople = editItineraryDescriptor.getPeople().orElse(itineraryToEdit.getPeople()); Budget updatedBudget = editItineraryDescriptor.getBudget().orElse(itineraryToEdit.getBudget()); diff --git a/src/main/java/seedu/waddle/model/item/Duration.java b/src/main/java/seedu/waddle/model/item/Duration.java index 2ae8f9679cd..fe25b72da06 100644 --- a/src/main/java/seedu/waddle/model/item/Duration.java +++ b/src/main/java/seedu/waddle/model/item/Duration.java @@ -8,7 +8,6 @@ public class Duration { public static final String MESSAGE_CONSTRAINTS = "Duration should only contain a positive number."; private final int duration; - private final boolean isNull; /** * Constructs a {@code Duration}. @@ -16,14 +15,8 @@ public class Duration { * @param duration A valid duration. */ public Duration(String duration) { - if (duration == null) { - this.isNull = true; - this.duration = 0; - } else { - checkArgument(isValidDuration(duration), MESSAGE_CONSTRAINTS); - this.duration = Integer.valueOf(duration); - this.isNull = false; - } + checkArgument(isValidDuration(duration), MESSAGE_CONSTRAINTS); + this.duration = Integer.valueOf(duration); } public int getDuration() { @@ -45,10 +38,6 @@ public static boolean isValidDuration(String test) { } return value >= 0; } - - public boolean isNull() { - return isNull; - } @Override public String toString() { return String.valueOf(duration); diff --git a/src/main/java/seedu/waddle/model/itinerary/Itinerary.java b/src/main/java/seedu/waddle/model/itinerary/Itinerary.java index 795d751560a..252405aec68 100644 --- a/src/main/java/seedu/waddle/model/itinerary/Itinerary.java +++ b/src/main/java/seedu/waddle/model/itinerary/Itinerary.java @@ -66,12 +66,6 @@ public Country getCountry() { return country; } - /* TODO: implement method - public Date getEndDate() { - return duration.getEndFromStart(startDate); - } - */ - public Date getStartDate() { return startDate; } @@ -263,10 +257,14 @@ public String toString() { return builder.toString(); } - public void setDays(List days) { - for (int i = 0; i < getDuration().getValue(); i++) { - if (i < days.size()) { - this.days.add(i, days.get(i)); + public void setDays(List dayList) { + for (int i = 0; i < dayList.size(); i++) { + if (i < getDuration().getValue()) { + this.days.set(i, dayList.get(i));; + } else { + for (Item item : dayList.get(i).deleteDay()) { + addItem(item); + } } } }