conception/update #93

Merged
mael.daim merged 7 commits from conception/update into master 1 year ago
Owner

This branch updated the old documentation.
I made an architecture diagram for the web application.
I also added a new markdomwn "Description.md" containing all class diagram and a brand new description for each of them.
Please in future update the documentation of the conception each time you change something because we are going to be marked on sunday the 21st.

This branch updated the old documentation. I made an architecture diagram for the web application. I also added a new markdomwn "Description.md" containing all class diagram and a brand new description for each of them. Please in future update the documentation of the conception each time you change something because we are going to be marked on sunday the 21st.
maxime.batista was assigned by mael.daim 1 year ago
samuel.berion was assigned by mael.daim 1 year ago
vivien.dufour was assigned by mael.daim 1 year ago
yanis.dahmane-bounoua was assigned by mael.daim 1 year ago
mael.daim added 4 commits 1 year ago
continuous-integration/drone/push Build is passing Details
e1056f4ade
updated old documentation
continuous-integration/drone/push Build is passing Details
f71835352d
updated editor's mvc conception
mael.daim added 1 commit 1 year ago
continuous-integration/drone/push Build is failing Details
14eaf110d2
fixed conflict
mael.daim added 1 commit 1 year ago
continuous-integration/drone/push Build is passing Details
c596bfba71
Fixed bugs
vivien.dufour approved these changes 1 year ago
maxime.batista requested changes 1 year ago
Now, let's discuss a much bigger part of the diagram.
In this part we find all the team logic. Actually, a team only have an array of members and a "TeamInfo".
The class "TeamInfo" exist to allows to split the data of the members.
- The class "TeamInfo" exist to allows to split the data of the members.
+ The class "TeamInfo" only exists to split the team's information data (name, id etc) from the members.
```diff - The class "TeamInfo" exist to allows to split the data of the members. + The class "TeamInfo" only exists to split the team's information data (name, id etc) from the members. ```
mael.daim marked this conversation as resolved
In general, we use the implementation "SimpleFunctionValidator".
We reconize the strategy pattern. Indeed, we need a family of algorithms because we have many classes that only differ by the way they validate.
Futhermore, you may have notices the ComposedValidator that allows to chain several Validator.
We naturally used the composite pattern to solve this problem.
- We naturally used the composite pattern to solve this problem.
+ We can see that this system uses the composite pattern
```diff - We naturally used the composite pattern to solve this problem. + We can see that this system uses the composite pattern ```
mael.daim marked this conversation as resolved
## Session's class diagram
![Session's class diagram](./assets/session.svg)
It encapsulates the PHP's array "$_SESSION" and kind of replaces it. With two interfaces that dictate how a session should be handled, and same for a mutable one.
- It encapsulates the PHP's array "$_SESSION" and kind of replaces it. With two interfaces that dictate how a session should be handled, and same for a mutable one.
+ It encapsulates the PHP's array "$_SESSION". With two interfaces that dictate how a session should be handled, and same for a mutable one.
```diff - It encapsulates the PHP's array "$_SESSION" and kind of replaces it. With two interfaces that dictate how a session should be handled, and same for a mutable one. + It encapsulates the PHP's array "$_SESSION". With two interfaces that dictate how a session should be handled, and same for a mutable one. ```
mael.daim marked this conversation as resolved
mael.daim added 1 commit 1 year ago
continuous-integration/drone/push Build is passing Details
d9862b85c0
applied required review's changement
mael.daim requested review from maxime.batista 1 year ago
mael.daim removed review request for maxime.batista 1 year ago
maxime.batista approved these changes 1 year ago
samuel.berion approved these changes 1 year ago
mael.daim merged commit b89d325415 into master 1 year ago
mael.daim deleted branch conception/update 1 year ago

Reviewers

vivien.dufour approved these changes 1 year ago
maxime.batista approved these changes 1 year ago
samuel.berion approved these changes 1 year ago
continuous-integration/drone/push Build is passing
The pull request has been merged as b89d325415.
Sign in to join this conversation.
No Milestone
No project
4 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: IQBall/Application-Web#93
Loading…
There is no content yet.