cauca-9-1-1 / survip-webapi

SURVI-Prevention Web API

survip-webapi

Installing WKHTMLTOPDF

Linux (Ubuntu)

From root directory run :

sh install.sh

Windows

Download and run the wkhtmltopdf.exe file. Set the destination folder to "C:\Program Files\wkhtmltopdf"

https://wkhtmltopdf.org/downloads.html

You will probably need to add wkhtmltopdf.exe to your Windows System Environmental Settings Path (this is a little different for each Windows version).

Right click on your "My Computers" icon, click Advanced Settings, under Environmental Settings edit the PATH and add the directory path for the wkhtmltopdf.exe file to your PATH.

Mac

curl -O http://wkhtmltopdf.googlecode.com/files/wkhtmltopdf-0.9.9-OS-X.i368
mv wkhtmltopdf-0.9.9-OS-X.i368 /usr/local/bin/wkhtmltopdf
chmod +x /usr/local/bin/wkhtmltopdf

survip-webapi's Issues

Implements the refresh token

Ajout de l'importation des Provinces

Créer la méthode d'import dans le controller ainsi que dans le service(basé sur une liste de provinces)
Création des services de validation et d'import.
Création des Units tests.

Intégrer les pictogrames à la BD

Add imported fields to entities which will be used in importation process

Ajouter une classe abstraite BaseImportedModel qui héritera de BaseModel et qui aura ces champs supplémentaires :
o IdExtern : string
o ImportedOn : dateTime?
o HasBeenModified : bool
Tous les modèles qui seront importés hériteront désormais de cette nouvelle classe abstraite plutôt que de BaseModel.

Créer la migration.

Et on modifie tous les services pour que l'ajout, la suppression et la modification renseignent HasBeenModified).

Faire fonctionner les fichiers de configurations hosting.json et appsettings.json

L'API ASP.NET Core ne tiens pas compte de hosting.json par défaut pour pouvoir controller le port et l'ip sur lesquels le service écoute sur le réseau.

InspectionQueryService - Optimisation SQL à faire

InspectionQuestionService.GetInspectionQuerySummaryListLocalized.

• Les SingleOrDefault dans le select de la query sont évalué localement.

Améliorer les données initiales

Lorsque l'on execute la migration de données suite à un reset de la BD les permission sont déactiver pour l'usager admin.
Il faudrait valider que cet usager et voire même le groupe Administration ai accès à pratiquement tout.

Également hardcoder tous les ids plutôt que d'utiliser l'incrémentation des guid tel qu'on le fait actuellement.

Migration setup

Setup la migration au lieu de la in house méthode utilisée pour générer la BD

WebuserFireSafetyDepartment has two fields for the FireSafetyDepartmentId

Because there's no mapping, EF doesn't know it has to use IdFireSafetyDepartment from the model and create a field named fire_safety_department_id in the database.

Error on approving inspection

Error when approving inspection on the building detail.
If the building detail is not the same of the inspection_buidling_detail one we got an error of foreignkey that blocked the approvment.

Reproduce only on one inspection -> put more validation when copying data from the inspection

Refonte de la structure du questionnaire afin de rendre tout ça plus flexible

Idée:

• Avoir un champ qui indique qu'une question permet le multiple?
• Avoir un champ qui indique le nombre maximum d’occurrence?
• Etc.

On pourrait avoir un type de question qui serait "réponses enfants". Ce type de question aurait alors une série de question enfant.

Mais quand on regarde la configuration actuelle, même les questions qui ont rapport à un détecteur de fumée ont de l'influence sur les suivantes. Si on répond oui à "Est-il alimenté a un pile", on a pas la même question suivante que si on répond non.

ASDKFJ;SDLKJ

• ASDF

Validation on survey when creating new inspection

If the inspection configuration is not set up for the target risk level, the API fail the transaction of putting it to ready status.
Ex: create an inspection for a risk level not configured.

la désactivation du questionnaire ne désactive pas ses collections enfant.

Ce point ne cause aucune erreur dans l'affichage car un questionnaire n'est pas réactivable.
Lorsque l'on supprime un questionnaire celui-ci est desactivé par le CrudService, cependant ce dernier ne descends pas en récursive dans les enfants de l'entité.
Modifications:

• Surcharger la fonction remove dans le service SurveyService
  -> Désactiver les entités suivantes:
  - survey_localization
  - survey_question
  - survey_question_localization
  - survey_question_choice
  - survey_question_choice_localization

Mise à jour des information de modifications: Localization

Comment gérer la mise à jour de localization:

• Pas de service distinct sous collection
• On ne sait pas si la localization a changé réellement, doit-on changer ces informations à chaque fois ?
• Doit-on gérer cela dans l'application au lieu de l'API ?

Import de table n'étant pas dans la RAO

Les entités ci-dessous n'ont pas de model d'import car la plupart sont différentes ou non existantes dans la RAO(pour le moment)
Building's picture.
BuildingSprinkler + sprinkler type.
BuildingAnomaly, BuildingAnomalyPicture + Picture
BuildingDetails + detail's types (ConstructionType, ConstructionFireResistanceType, RoofType, RoofMaterialType, SidingType, BuildingType) + implantation plan
BuildingParticularRisk + pictures

Ajout des placeholders de la configuration des rapports

Le premier lot de placeholders à être ajoutés seront les informations disponibles lors de la validation d'une inspection. On pourra par la suite raffiner la granularité des palceholders au besoin.

Ajouter une date de dernière modification et un usager aux tables

Dans BaseModel on ajoute:

• LastModifiedOn : DateTime?
• IdWebuserLastModifiedBy: Guid?

Créer la nouvelle migration.

Et on modifie tous les services pour que la suppression et la modification renseignent ces trois champs (LastModifiedOn, IdWebuserLastModifiedBy ).

IdWebuserLastModifiedBy seront aussi mis à jour sur la création.

When doing an inspection, the user should work on a copy of the inspection's buildings data.

So we need to copy the parent and its children when creating the inspection :

• BuildingDetail
• PNAPS, contacts, haz mat
• particular risks
• fire protection
• fire hydrants
• implantation's plan
• Etc.

There's no need to change anything on the mobile application for this as all of the controller are already correctly named (like InspectionBuildingPnapController), but we need to change all of these queries so they use the copied data or something.

Changer la stratégie de mapping des unités de mesure

Reloader le sketch enregistrer

Error when adding picture in building management

When adding a picture to a building the picture sent doesn't have a id.
Modification to do:

• Add Guid to the picture when added from the web page

Importer les bâtiments de St-Georges

Parent, enfant avec la relation.

Report's generation is throwing 500's on the test server

Requête de liste pour les voies pour les lookup

Il faut créer une qrequête spécifique pour les voies pour les controles de type lookup sans limitation. Les lookup gère le chargement des données et sont donc plus optimisé au nombreux résultats.

Améliorer les notions de statut de visite et d'inspection

Il faut clarifier les statuts de visites et d'inspection et leur utilisation.

Préparer la base de données de TEST

Pays, Provinces, Régions

Importer les SSI/MRC suivants:

• MRC Coaticook au complet;
• MRC Vaudreuil-Soulanges au complet;
• SSI Sutton / Lac Brôme;
• SSI Sainte-Anne-Des-Lacs.

On importe:

• Pays;
• Provinces;
• Régions;
• Les MRCS;
• SSI;
• Types de villes;
• Villes;
• Liens villes/SSIs;
• Rues;
• Batîments (+ alias principal dans building_localization).

Test

Test

Particular risk pictures are removed from DB instead of deactivated

When deleting a picture in particular risk of an inspection. The building particular risk picture is deactivated but the picture in inspection picture is deleted, so du to cascading destruction the particular risk picture id finally deleted.
Is that ok ?

Update DotNet Core, Asp.Net core & EF Core to 2.1 final

See this page to migrate from 2.0 to 2.1:
https://docs.microsoft.com/en-us/aspnet/core/migration/20_21?view=aspnetcore-2.1

Saving problem in unit of measure

Managing the field measure_type from discrimant :

• Using an enum + override in controller

Créer les classes intermédiaires pour l'importation

Pour tous nos modèles pour lesquels on permettra l'importation.

Les classes intermédiaires seront à peu près identiques aux modèles, sauf sans les relations et les clefs étrangères seront des string qui correspondront aux IdExtern.

Préparer le data pour la démo de l'ACSIQ.

Report generation : picture are not correctly inserted in the reports.

No picture are ever shown in the report even though the image frame is there.

We need to double check the code to make sure we're correctly generating the img tag.

Permettre aux systèmes externes d'être authentifié

Enregistrer et recharger les JSON des sketchs pour les images d'anomalies

t3w

t634w

Add survey to the inspection configuration

Sécurité lors du login

Il faudrait favoriser l’envoie du nom d’usager et de son mot de passe dans le « POST body » au lieu de l’url afin d’éviter que le mot de passe ne soit enregistré dans la cache.

Inspection course lanes validation in inspection review

Modify request of inspection building course lanes to return only active course lanes in getCompleteCourses, or create a new method to return the active.
When deleting a courseLane -> the course lane is removed from db instead of deactivated
-> InspectionBuildingCourseService -> SaveCompleteCourses

Valider le token d'accès avec la BD pour être sûr qu'il existe en BD même s'il est valide

Lorsqu'on valide le token, on ne valide aucunement via la BD si le token est encore valide, ce qui fait qu'on pourrait créer un token via un autre API qui utilise la même clef d'encodage et que ça passerait comme valide.

Il faudrait voir pour permettre la validation en BD du token. Je n'ai pas la moindre idée de comment intégrer ça dans le processus d'authentification par token d'ASP.net.

Delete the inspection's building data when deleting an inspection or one of its building

We need to delete everything in inspection_building (and children) when:

• Deleting an inspection.
• Removing a building from an inspection.

Obtention de la liste des placeholders

Il faut une liste sous forme de texte à afficher à l'utilisateur.

Enlever les méthodes non implémentées de l'API

Plusieurs méthodes sont non implémentées dans l'API, souvent à cause des méthode abstraite du BaseCrudService (Get(id), GetList()). Même si ces méthodes ne sont appelées que dans l'API test, il faut y apporter des validations pour que celle-ci ne causent pas d'erreur.
Modification:

• Ajouter un autre service BaseCrudServiceWithGet avec ces méthodes abstraite et modifier l'existant pour les enlever

Setup odata et refaire les call du tableau de bord pour l'utiliser

Add country importation

• Ajouter un controller ou utiliser le controller existant.
• Ajouter un service ou utiliser le service existant.

Bref, on prend l'objet intermédiaire et on le transforme en objet "prévention" et fait l'insert, update ou delete approprié.

Robustesse du serveur et gestion d'exceptions

Il y a peu de contrôle d’exception dans les différents services. Ceci implique qu’un accès au « Context » mal formulé, ou bien une donnée absente dans la base de données fait planter l’application à tout coups, et ce, avec très peu de verbosité.

Exemple : Dans la méthode « GetListActive » du fichier « WebuserService », le nom de famille est obtenu dynamiquement à l’aide de « lastname = users.Attributes.Where(a => a.AttributeName=="last_name") ». Par la suite, il n’y a pas de vérification afin de valider que « lastname » n’est pas nulle avant d’accéder à « lastname.AttributeValue ». Bien que tous les utilisateurs aient les attributs de bases (first_name, last_name, reset_password) associés, la requête retourne des valeurs nulles et l’absence de vérification lance une exception qui n’est pas attrapée.

Génération HTML to PDF - Remplacer WkWrap par une autre librairie

Alternative:

JsReport: https://stackoverflow.com/a/45840308/6347013 (https://jsreport.net/learn/dotnet)
DinkToPdf : https://github.com/rdvojmoc/DinkToPdf

Importer les rues de St-Georges de la RAO dans Survi-Prévention.

Integrer l'enregistrement des croquis