Merge pull request #253 from keithxun/branch-update-DG

Update delete member sequence diagram

docs/diagrams/DeleteSequenceDiagram.puml

@@ -4,15 +4,14 @@ skinparam ArrowFontStyle plain
 
 box Logic LOGIC_COLOR_T1
 participant ":LogicManager" as LogicManager LOGIC_COLOR
-participant ":Parser" as Parser LOGIC_COLOR
-participant ":DeleteCommandParser" as DeleteCommandParser LOGIC_COLOR
-participant "d:DeleteCommand" as DeleteCommand LOGIC_COLOR
-participant "r:CommandResult" as CommandResult LOGIC_COLOR
+participant ":HallPointerParser" as Parser LOGIC_COLOR
+participant ":DeleteMemberCommandParser" as DeleteMemberCommandParser LOGIC_COLOR
+participant "dmc:DeleteMemberCommand" as DeleteMemberCommand LOGIC_COLOR
+participant "result:CommandResult" as CommandResult LOGIC_COLOR
 end box
 
 box Model MODEL_COLOR_T1
-participant "m:Model" as Model MODEL_COLOR
-participant "member:Member" as Member MODEL_COLOR
+participant "model:Model" as Model MODEL_COLOR
 end box
 
 [-> LogicManager : execute("delete_member 1")
@@ -21,57 +20,51 @@ activate LogicManager
 LogicManager -> Parser : parseCommand("delete_member 1")
 activate Parser
 
-create DeleteCommandParser
-Parser -> DeleteCommandParser
-activate DeleteCommandParser
+create DeleteMemberCommandParser
+Parser -> DeleteMemberCommandParser
+activate DeleteMemberCommandParser
 
-DeleteCommandParser --> Parser
-deactivate DeleteCommandParser
+DeleteMemberCommandParser --> Parser
+deactivate DeleteMemberCommandParser
 
-Parser -> DeleteCommandParser : parse("1")
-activate DeleteCommandParser
+Parser -> DeleteMemberCommandParser : parse("1")
+activate DeleteMemberCommandParser
 
-create DeleteCommand
-DeleteCommandParser -> DeleteCommand
-activate DeleteCommand
+create DeleteMemberCommand
+DeleteMemberCommandParser -> DeleteMemberCommand
+activate DeleteMemberCommand
 
-DeleteCommand --> DeleteCommandParser :
-deactivate DeleteCommand
+DeleteMemberCommand --> DeleteMemberCommandParser : dmc
+deactivate DeleteMemberCommand
 
-DeleteCommandParser --> Parser : d
-deactivate DeleteCommandParser
+DeleteMemberCommandParser --> Parser : dmc
+deactivate DeleteMemberCommandParser
 'Hidden arrow to position the destroy marker below the end of the activation bar.
-DeleteCommandParser -[hidden]-> Parser
-destroy DeleteCommandParser
+DeleteMemberCommandParser -[hidden]-> Parser
+destroy DeleteMemberCommandParser
 
-Parser --> LogicManager : d
+Parser --> LogicManager : dmc
 deactivate Parser
 
-LogicManager -> DeleteCommand : execute(m)
-activate DeleteCommand
+LogicManager -> DeleteMemberCommand : execute(model)
+activate DeleteMemberCommand
 
-DeleteCommand -> Model : getFilteredMemberList()
+DeleteMemberCommand -> Model : deleteMember(member)
 activate Model
 
-Model --> DeleteCommand : List<Member>
-deactivate Model
-
-DeleteCommand -> Model : deleteMember(member)
-activate Model
-
-Model --> DeleteCommand
+Model --> DeleteMemberCommand
 deactivate Model
 
 create CommandResult
-DeleteCommand -> CommandResult
+DeleteMemberCommand -> CommandResult
 activate CommandResult
 
-CommandResult --> DeleteCommand
+CommandResult --> DeleteMemberCommand : result
 deactivate CommandResult
 
-DeleteCommand --> LogicManager : r
-deactivate DeleteCommand
+DeleteMemberCommand --> LogicManager : result
+deactivate DeleteMemberCommand
 
-[<--LogicManager
+[<-- LogicManager
 deactivate LogicManager
 @enduml