Merge pull request #237 from annnniexu/master

fixed totalexpense diagram
AY2324S2-CS2113-T12-4 · Apr 15, 2024 · 31bc348 · 31bc348
2 parents 5c924d3 + 5536ba8
commit 31bc348
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/docs/DeveloperGuide.md b/docs/DeveloperGuide.md
@@ -184,7 +184,7 @@ Step 2. After the user has added new travel activities into the `travelActivityL
 
 Step 3. The method will loop through each activity in `travelActivities` and add up all the expenses for the given type. If no type is provided, it will default to TravelActivity type.
 
-![totalexpense.png](DiagramImages/totalexpense.png)
+![TotalExpenseDiagram.png](DiagramImages/TotalExpenseDiagram.png)
 
 ### Tag feature
 

diff --git a/docs/DiagramImages/TotalExpenseDiagram.png b/docs/DiagramImages/TotalExpenseDiagram.png
diff --git a/docs/plantUML diagrams/totalexpense.puml b/docs/plantUML diagrams/totalexpense.puml
@@ -4,17 +4,26 @@
 autonumber
 actor User
 User -> OmniTravel : "totalexpense /type Food"
+activate OmniTravel
 OmniTravel -> Parser : totalExpenseCommand(line, list)
+activate Parser
    Parser -> TravelActivityList : totalExpense(type)
+   activate TravelActivityList
    loop size of travelActivities ArrayList
       opt activity type matches the type the user inputed
          TravelActivityList -> TravelActivity: getExpense()
+         activate TravelActivity
          TravelActivity --> TravelActivityList:
+         deactivate TravelActivity
          TravelActivityList -> TravelActivityList: total += expense
 
       end
    end
+
 TravelActivityList --> Parser
+deactivate TravelActivityList
 Parser --> OmniTravel
+deactivate Parser
 OmniTravel --> User
+deactivate OmniTravel
 @enduml