diff --git a/docs/DeveloperGuide.md b/docs/DeveloperGuide.md index 698bfbb0cbc..997b05ba059 100644 --- a/docs/DeveloperGuide.md +++ b/docs/DeveloperGuide.md @@ -312,6 +312,9 @@ The following sequence diagram shows how the gather emails by financial plan ope ### Expanded Find feature The enhanced find mechanism is facilitated by the `CombinedPredicate` and utilises the existing `FindCommand` structure. + +#### Implementation Overview + Here's a sequence diagram that demonstrates how `FindCommand` works. ![FindCommandSequenceDiagram](images/FindCommandSequenceDiagram.png) @@ -339,7 +342,7 @@ For now, we only allow for searching for `Name`, `FinancialPlan` and `Tag` field searched fields, but extending the feature to search in other fields is possible by creating the appropriate `Predicate` class and modifying the `FindCommandParser`. -#### Design Considerations: +#### Design Considerations **Aspect: How to implement find for multiple fields** * **Alternative 1 (current choice):** Use one unified command and format. diff --git a/src/main/java/seedu/address/model/person/predicates/PersonContainsKeywordsPredicate.java b/src/main/java/seedu/address/model/person/predicates/PersonContainsKeywordsPredicate.java index 43e47031784..1c0efe9dde8 100644 --- a/src/main/java/seedu/address/model/person/predicates/PersonContainsKeywordsPredicate.java +++ b/src/main/java/seedu/address/model/person/predicates/PersonContainsKeywordsPredicate.java @@ -5,7 +5,8 @@ import seedu.address.model.person.Person; /** - * Tests if a {@code Person} contains certain keywords in a certain field. + * Tests if a {@code Person} contains certain keywords in a certain field. Note that the implementing predicates are not + * required to test the validity of the keywords. */ public interface PersonContainsKeywordsPredicate extends Predicate {