Cannot approve an inspection anymore

[jocelyncardinal]

Describe the bug
Cannot approve an inspection anymore

Screenshots

[jocelyncardinal]

Related to issue #265

[iamkinetic]

Error code 500 from https://survipreventiontest.cauca.ca/api/Inspection/(*)/approve

[iamkinetic]

Ouin, celui-là semble être un bug qui n'a pas été causé par la mise à jour des librairies qu'on utilise.

Je ne comprends pas trop ce qui se passe avec ça, mais je travaille là dessus.

[iamkinetic]

Ok pour l'historique le problème est celui-ci

• Nous avons utilisés le property hiding dans l'héritage d'une des classes pour réussir à faire quelque chose de spécifique à l'importation.
• Nous utilisons également la "reflection" à quelques endroits dans le code de l'acceptation d'inspection pour avoir du code générique. La reflection retourne la propriété qui est cachée ET la propriété qui la cache au lieu de juste celle qui la cache.
• Comme le property hiding n'est pas exactement une très bonne pratique, je vais remplacer ça par une meilleure méthode (ie: contraintes de généricité via interface plutôt que par classe abstraite générique).

Pour le détail, c'est cette ligne de InspectionBuilding qui est la cause du problème:

public new ICollection<InspectionBuildingLocalization> Localizations { get; set; }

Je crois bien que ça été ajouté comme un genre de workaround pour faciliter l'héritage dans l'importation. La solution exacte était d'enlever cette ligne pour ne plus cacher la vraie propriété et de changer les contraintes de généricités des BaseBuildingContact, BaseBuildingHazardousMaterial et compagnie de BaseBuilding à IBaseBuilding (que je viens de créer à partir de BaseBuilding).

C'est définitivement un problème que je qualifierais de niveau avancé.

[iamkinetic]

Ok, la correction est terminée et les techs m'ont dit que l'API serait mis à jour d'ici midi.

[iamkinetic]

Ok, c'est fait, la correction est en ligne.

[jocelyncardinal]

Ok merci … Par contre je vois plus le bouton pour générer le rapport PDF ? Pas capable d’ajouter cette colonne [cid:image001.png@01D4BA24.B9730960] De : iamkinetic <notifications@github.com> Répondre à : CAUCA-9-1-1/survip-webapi <reply@reply.github.com> Date : vendredi 1 février 2019 11:47 À : CAUCA-9-1-1/survip-webapi <survip-webapi@noreply.github.com> Cc : "jocelyn.cardinal@hotmail.com" <jocelyn.cardinal@hotmail.com>, Author <author@noreply.github.com> Objet : Re: [CAUCA-9-1-1/survip-webapi] Cannot approve an inspection anymore (#269) Ok, c'est fait, la correction est en ligne. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCAUCA-9-1-1%2Fsurvip-webapi%2Fissues%2F269%23issuecomment-459787451&data=02%7C01%7C%7C802d521d5733445b8b5b08d68864e764%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636846364286748969&sdata=UX5IC7EW232laMYEt8JB8eXnnacU6U1fEk%2BtdwHDUZs%3D&reserved=0>, or mute the thread<https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAer3TRQb6VZz2WEJ8VEsTaSQFQGsS4bTks5vJG-JgaJpZM4adU00&data=02%7C01%7C%7C802d521d5733445b8b5b08d68864e764%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636846364286758961&sdata=B74enDY1UpPBHMGoN9YzErnvVmQKy2nWif77JjZaEBQ%3D&reserved=0>.

[iamkinetic]

Il y a un workaround pour ça. Dans ton browser, tu peux aller dans la console et coller cette commande :

localStorage.setItem("column-visible-mode3", "[false,true,true,true,false,false,false,false,false,false,true,true,true,false,false,false,false,false,false,false,false,false,false,false,true]")

Ce site montre comment afficher la console dans Safari sur Mac :
https://www.wickedlysmart.com/hfjsconsole/

La commande devrait te retourner "Undefined" comme ceci :

Après ça tu peux faire un refresh du browser et la colonne va s'afficher.