Skip to content

Commit 1f85a9e

Browse files
committed
Fix bugs
1 parent af4961e commit 1f85a9e

4 files changed

Lines changed: 17 additions & 17 deletions

File tree

docs/DeveloperGuide.md

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,11 @@ The ***Architecture Diagram*** given above explains the high-level design of the
3636

3737
<div markdown="span" class="alert alert-primary">
3838

39-
:bulb: **Tip:** The `.puml` files used to create diagrams in this document can be found in the [diagrams](https://github.com/se-edu/addressbook-level3/tree/master/docs/diagrams/) folder. Refer to the [_PlantUML Tutorial_ at se-edu/guides](https://se-education.org/guides/tutorials/plantUml.html) to learn how to create and edit diagrams.
39+
:bulb: **Tip:** The `.puml` files used to create diagrams in this document can be found in the diagrams folder. Refer to the [_PlantUML Tutorial_ at se-edu/guides](https://se-education.org/guides/tutorials/plantUml.html) to learn how to create and edit diagrams.
4040

4141
</div>
4242

43-
**`Main`** has two classes called [`Main`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/java/seedu/address/Main.java) and [`MainApp`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/java/seedu/address/MainApp.java). It is responsible for,
43+
**`Main`** has two classes called `Main` and `MainApp`. It is responsible for,
4444
* At app launch: Initializes the components in the correct sequence, and connects them up with each other.
4545
* At shut down: Shuts down the components and invokes cleanup methods where necessary.
4646

@@ -80,12 +80,12 @@ The sections below give more details of each component.
8080
for clarity.](images/UiClassDiagram.png)
8181

8282
**API** :
83-
[`Ui.java`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/java/seedu/address/ui/Ui.java)
83+
`Ui.java`
8484

8585
The UI consists of a `MainWindow` that is made up of parts e.g.`CommandBox`, `ResultDisplay`, `PersonListPanel`, `StatusBarFooter` etc.
8686
All these, including the `MainWindow`, inherit from the abstract `UiPart` class.
8787

88-
The `UI` component uses JavaFx UI framework. The layout of these UI parts are defined in matching `.fxml` files that are in the `src/main/resources/view` folder. For example, the layout of the [`MainWindow`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/java/seedu/address/ui/MainWindow.java) is specified in [`MainWindow.fxml`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/resources/view/MainWindow.fxml)
88+
The `UI` component uses JavaFx UI framework. The layout of these UI parts are defined in matching `.fxml` files that are in the `src/main/resources/view` folder. For example, the layout of the `MainWindow` is specified in `MainWindow.fxml`
8989

9090
The `UI` component,
9191

@@ -97,9 +97,9 @@ The `UI` component,
9797
![Structure of the Logic Component](images/LogicClassDiagram.png)
9898

9999
**API** :
100-
[`Logic.java`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/java/seedu/address/logic/Logic.java)
100+
`Logic.java`
101101

102-
1. `Logic` uses the `AddressBookParser` class to parse the user command.
102+
1. `Logic` uses the `MeetBuddyParser` class to parse the user command.
103103
1. This results in a `Command` object which is executed by the `LogicManager`.
104104
1. The command execution can affect the `Model` (e.g. adding a person).
105105
1. The result of the command execution is encapsulated as a `CommandResult` object which is passed back to the `Ui`.
@@ -122,7 +122,7 @@ Given below is another Sequence Diagram for interactions within the `Logic` comp
122122

123123
![Structure of the Model Component](images/ModelClassDiagramNew.png)
124124

125-
**API** : [`Model.java`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/java/seedu/address/model/Model.java)
125+
**API** : `Model.java`
126126

127127
The `Model`,
128128

@@ -131,7 +131,7 @@ The `Model`,
131131
* exposes an unmodifiable `ObservableList<Person>` that can be 'observed' e.g. the UI can be bound to this list so that the UI automatically updates when the data in the list change.
132132
* stores the meeting book data.
133133
* exposes an unmodifiable `ObservableList<Meeting>` that can be 'observed' e.g. the UI can be bound to this list so that the UI automatically updates when the data in the list change.
134-
* stores the connection between the person in the address book and the meeting in the meeting book. e.g. Tom and July both participate in the CS2103 Lecture.
134+
* stores the connection between the person in the contacts book and the meeting in the meeting book. e.g. Tom and July both participate in the CS2103 Lecture.
135135
* does not depend on any of the other three components.
136136
* (Some details are omitted)
137137

@@ -147,7 +147,7 @@ The `Model`,
147147

148148
![Structure of the Storage Component. The full diagram for MeetingBook is omitted to make the figure less cluttered.](images/StorageClassDiagram.png)
149149

150-
**API** : [`Storage.java`](https://github.com/se-edu/addressbook-level3/tree/master/src/main/java/seedu/address/storage/Storage.java)
150+
**API** : `Storage.java`
151151

152152
The `Storage` component,
153153
* can save `UserPref` objects in json format and read it back.
@@ -171,7 +171,7 @@ However, there were a few implementations details to consider. The implementatio
171171
discuss the one for contacts.
172172

173173
Currently, an essential attribute in the model are the filtered lists,
174-
which shows the results of searches and finds.
174+
which show the results of searches and finds.
175175
The indexes used in commands like "edit" are taken with respect to these filtered list.
176176
These filtered lists, however, are backed by immutable observable lists,
177177
which helps update the JavaFX GUI. I initially tried to make copies of these observable lists,

src/main/java/seedu/address/logic/commands/meetings/AddMeetingCommand.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public class AddMeetingCommand extends Command {
3232
+ PREFIX_START_TIME + "START TIME "
3333
+ PREFIX_END_TIME + "END TIME "
3434
+ "[" + PREFIX_DESCRIPTION + "DESCRIPTION] "
35-
+ "[" +PREFIX_PRIORITY + "PRIORITY] "
35+
+ "[" + PREFIX_PRIORITY + "PRIORITY] "
3636
+ "[" + PREFIX_GROUP + "GROUP]..."
3737
+ "[" + PREFIX_PERSON_CONNECTION + "INDEX OF PERSON RELATED]...\n"
3838
+ "Example: " + COMMAND_WORD + " "

src/main/java/seedu/address/logic/commands/persons/AddPersonCommand.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,9 @@ public class AddPersonCommand extends Command {
2323
public static final String MESSAGE_USAGE = COMMAND_WORD + ": Adds a person to the address book. "
2424
+ "Parameters: "
2525
+ PREFIX_NAME + "NAME "
26-
+ "[" +PREFIX_PHONE + "PHONE] "
27-
+ "[" +PREFIX_EMAIL + "EMAIL] "
28-
+ "[" +PREFIX_ADDRESS + "ADDRESS] "
26+
+ "[" + PREFIX_PHONE + "PHONE] "
27+
+ "[" + PREFIX_EMAIL + "EMAIL] "
28+
+ "[" + PREFIX_ADDRESS + "ADDRESS] "
2929
+ "[" + PREFIX_GROUP + "GROUP]...\n"
3030
+ "Example: " + COMMAND_WORD + " "
3131
+ PREFIX_NAME + "John Doe "

src/main/java/seedu/address/model/person/AddressBook.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
package seedu.address.model.person;
22

3-
import javafx.collections.ObservableList;
4-
import seedu.address.model.group.Group;
3+
import static java.util.Objects.requireNonNull;
54

65
import java.util.HashSet;
76
import java.util.List;
87
import java.util.Set;
98

10-
import static java.util.Objects.requireNonNull;
9+
import javafx.collections.ObservableList;
10+
import seedu.address.model.group.Group;
1111

1212
/**
1313
* Wraps all data at the address-book level

0 commit comments

Comments
 (0)