data-players / SyReen

https://app.syreen.fr
0 stars 1 forks source link

Erreur sur page projet #97

Closed BenoitCier closed 1 year ago

BenoitCier commented 1 year ago

J'ai une erreur non blocante sur la page projet : Absolute IRI "syreen:locationInformation" confused with prefix "syreen".

srosset81 commented 1 year ago

Cela arrive lorsqu'un sujet est enregistré comme une string "syreen:predicate" alors que ça devrait être un URI. @VincentFarcy J'imagine que c'est un nouveau predicat. Il est encore utilisé ? J'avais créé une migration pour fixer ce type de problèmes, il suffirait de l'adapter à ce prédicat et de la relancer. Je peux m'en occuper. https://github.com/assemblee-virtuelle/activitypods/blob/next/boilerplate/services/migration.service.js#L17-L36

VincentFarcy commented 1 year ago

Salut, "syreen:predicate" n'évoque rien pour moi et je n'en trouve pas trace dans le code. Tu peux tenter la migration @srosset81 ?

srosset81 commented 1 year ago

Je voulais parler de "syreen:locationInformation"

srosset81 commented 1 year ago

Je peux faire la migration, mais si les données continuent d'être enregistrer incorrectement, l'erreur va se reproduire. Il faut donc s'assurer que ça ne soit plus le cas.

VincentFarcy commented 1 year ago

Ok il est bien toujours utilisé. Il faut que je regarde pourquoi c'est enregistré en string

srosset81 commented 1 year ago

Il faut aussi que tu l'ajoutes dans le context https://activitypods.org/context.json Voir ce qui a été fait ici: https://github.com/assemblee-virtuelle/activitypods/commit/b41554786f895ce9263fd5bcb59716d19462d54d

VincentFarcy commented 1 year ago

Salut @srosset81 , donc si je comprends bien, il faut que j'intervienne dans le repo activitypods pour mettre à jour les info concernant les données de l'ontologie Syreen. C'est bien ça ?

srosset81 commented 1 year ago

@VincentFarcy Je crois avoir ajouté ce dont on a besoin par rapport à la dernière version. Mais à vérifier.

syreen:hasPart: {
  @type: "@id"
},
syreen:partOf: {
  @type: "@id"
},
syreen:depictedBy: {
  @type: "@id"
},
syreen:hasLocation: {
  @type: "@id"
},
syreen:hasCategory: {
  @type: "@id"
},
syreen:hasUnit: {
  @type: "@id"
},
syreen:hasStage: {
  @type: "@id"
},
syreen:hasProjectType: {
  @type: "@id"
},
syreen:costPricePublication: {
  @type: "xsd:boolean"
},
syreen:marketValuePublication: {
  @type: "xsd:boolean"
},
VincentFarcy commented 1 year ago

@srosset81 , J'ai refait le tour :

string :
-    'syreen:label': 'Titre',
-    'syreen:alternativeLabel': 'Sous-titre',
-    'syreen:description': 'Présentation',
-    'syreen:locationInformation': 'Complément d\'adresse',

date :
-    'syreen:startDate': "Date de début prévisionnelle",    
-    'syreen:endDate': "Date de fin prévisionnelle"
-    'syreen:startDate': "Date prévisionnelle de disponibilité",

relation :
-    'syreen:hasProjectType': "Type de projet",
-    'syreen:hasStage': 'Phase',
-    'syreen:hasLocation': 'Localisation',
-    'syreen:hasCategory': "Categorie",
-    'syreen:hasUnit': "Unité",
-    'syreen:depictedBy': 'Image',
-    'syreen:hasPart'
-    'syreen:partOf'

numeric :
-    'syreen:quantity': "Quantité",
-    'syreen:sellingPrice': "Prix de vente",
-    'syreen:estimatedNewValue': "Valeur Neuve Estimée (€)",
-    'syreen:agePercentage': "Vétusté (%)",
-    'syreen:interestPercentage': "Intérêt (%)",
-    'syreen:performancePercentage': "Performance (%)",
-    'syreen:marketValue': "Valeur sur le marché",
-    'syreen:hourlyPrice': "Taux horaire (€)",
-    'syreen:dismantlingHours': "Nombre d'heures de démontage",
-    'syreen:collectionHours': "Nombre d'heures de collecte",
-    'syreen:refurbishmentAndSaleHours': "Nombre d'heures de reconditionnement/vente",
-    'syreen:additionalCosts': "Frais annexe pour le lot (Transport/Consommable) (€)",
-    'syreen:costPrice': "Coût de revient",

boolean
-    'syreen:publishMarketValue': "Publier cette valeur",
-    'syreen:publishCostPrice': "Publier cette valeur",

=> ok pour les relations => les booleans doivent être renommées => A voir pour les autres types de variable !

srosset81 commented 1 year ago

Merci pour ce travail @VincentFarcy ! J'ai mis à jour le fichier de context, qui est automatiquement déployé: https://github.com/assemblee-virtuelle/activitypods/commit/38985f58f1723c8033d562e24bf8bb6cb0c1e7f3 Pour les valeurs numériques, normalement Fuseki est assez intelligent pour les prendre en compte.

simonLouvet commented 1 year ago

@srosset81 & @VincentFarcy est ce que c'est résolu?

VincentFarcy commented 1 year ago

@simonLouvet Je viens de tester avec un nouveau user de test : pour moi c'est ok on peut clore.