docky37 / alerts Goto Github PK

• Mapping with service ?
• Service to DB ?

• remove id in CoveredPerson class used in OPS#1 (done)
• rename birthDate as birthdate (done)
• extract service interfaces (done)
• use a PersonDTO class for OPS#1 et 2 (done)
• rename model DTO classes and move them in alerts/DTO (done)

• Install Spring Boot Starter MVC (2.2.5)
• Add JaCoCo, Findbugs
• Add Surfire report, JaCoCo report and FindBugs report

• add city field with "Culver" default value
• add zip field with "97451" default value
• modify PersonMapping to get an set these fields.

• Make the many to one join in PersonEntity class
• Create a Person class
• Create a mapping between Person & PersonEntity classes.

From the begining, Alerts application use the same AddressFirestation class both as DTO and Entity.

This feature has been developped in 2 parts and that does not respect the specs.
So need to be rewritten in one request.

• Insert address
• Returns child age in monthes if age < 2 years
• extract mapping in util class

Need health, info, metrics & report end points

• add MedicalRecords (model)
• add MedicalReportController & unit tests
• add MedicalReportService & unit tests

• Need to avoid duplicates in address (PB with POST request because omnipotent)
• Need to avoid duplicate persons (PB with POST request because omnipotent)
• Verify Put request: same FirstName + LastName in URI than in Content
• Verify mails format
• Verify phone format

• Create PersonControllerTest class with first test on Get request.
• Create PersonService class mocked in PersonControllerTest.
• Create PersonController with Get request.

After that same process test + feat for

• Get Person/lastName/firstName
• Push Person
• Put Person/lastName/firstName
• Delete

• In PersonService & MedicalRecordsService update method do not check if firstName & lastName are the same in content than in parameters !!! So it is possible to rename a Person !

Cette liste doit regrouper les personnes par adresse. Elle doit aussi inclure le nom, le numéro de téléphone et l'âge des habitants, etfaire figurer leurs antécédents médicaux (médicaments, posologie et allergies) à côté de chaque nom.
List of Person (names, phone, age, medical data)
Group by Address

• Create Person class
• Create FireStation Class
• Create MedicalRecords Class

• Written to return a List instead of one PersonInfoDTO so method is able to deal with namesake, and return more than one PersonInfoDTO.
• The mapping method for OPS#6 is a part of OPS#4 one.

I think I've to:

• remove duplicates,
• remove emptiy e-mail address.

I think we need to add a request to the person endpoint to get a list of person without e-mail address.

Norme HTTP. D’après la RFC 2616 (HTTP methods), PUT sert à
Créer une entité :
L’URI doit être celle de l’entité à créer . Si l’entité dans la requête est acceptée et créée côté serveur, le serveur doit renvoyer un code 201 (created).
Si l’entité peut avoir plusieurs URIs (exemple : /script/v1.2 ou /script/current), le serveur peut donc définir ces URIs alternatives comme pointant sur la nouvelle entité

Modifier une entité :
Si une entité avec la même URI que celle de la requête existe déjà, l’entité de la requête devrait la remplacer. Ce n’est pas une création, le code retourné par le serveur en cas de succès doit être 200 (OK) ou 204 (No content). En effet, en pratique un PUT va très rarement générer une réponse avec un contenu.

When I have done the refactoring of the addListMedicalRecord using MedicalMapping I have forgotten one thing, the join with personEntity !

EI: May expose internal representation by returning reference to mutable object

La méthode com.safetynet.alerts.DTO.PersonInfoDTO.getAllergies() risque d'exposer sa représentation interne en renvoyant com.safetynet.alerts.DTO.PersonInfoDTO.allergies
La méthode com.safetynet.alerts.DTO.PersonInfoDTO.getMedications() risque d'exposer sa représentation interne en renvoyant com.safetynet.alerts.DTO.PersonInfoDTO.medications
La méthode new com.safetynet.alerts.DTO.PersonInfoDTO(String, String, String, String[], String[], String) risque d'exposer sa représentation interne en stockant un objet externe modifiable dans com.safetynet.alerts.DTO.PersonInfoDTO.allergies
La méthode new com.safetynet.alerts.DTO.PersonInfoDTO(String, String, String, String[], String[], String) risque d'exposer sa représentation interne en stockant un objet externe modifiable dans com.safetynet.alerts.DTO.PersonInfoDTO.medications
La méthode com.safetynet.alerts.DTO.PersonInfoDTO.setAllergies(String[]) risque d'exposer sa représentation interne en stockant un objet externe modifiable dans com.safetynet.alerts.DTO.PersonInfoDTO.allergies
La méthode com.safetynet.alerts.DTO.PersonInfoDTO.setMedications(String[]) risque d'exposer sa représentation interne en stockant un objet externe modifiable dans com.safetynet.alerts.DTO.PersonInfoDTO.medications