search

ABI-Team-30 / Fresnel-Forms

A Protégé plugin for generating and adapting MediaWiki user interfaces for source ontologies
http://is.cs.ou.nl/OWF/index.php5/Fresnel_Forms
3 stars 1 forks source link

2ary namespace voor alles op wiki en Fresnel #66

Closed LloydRutledge closed 9 years ago

LloydRutledge commented 9 years ago

Moet nog voor property pagina's. Is al in veel fresnel. Bijvoorbeeld uit FOAF moet Property:accountName worden Property:foaf-accountName.

AlexMekkering commented 9 years ago

Ingevoerd in r453.

De gehele wijziging heeft nogal wat impact die ik nu even niet kan overzien. @jheijning , kun jij dit met jou berg aan kennis en ervaring over de wiki syntax aan een review onderwerpen? Ik ben met name geïnteresseerd of de smwImport pages zo goed zijn of dat daar de namespaces ook dieper geïntegreerd moeten worden.

Ook is hiermee de gehele fresnel2wiki verwerking een stuk schaalbaarder geworden omdat er minder objecten/lijsten & strings bijgehouden worden en dus minder geheugen gebruikt wordt.

jbachh commented 9 years ago

Ziet er weer een stuk professioneler uit :-)

RDF export ziet er nog steeds zo uit zoals het moet (voor zover mijn kennis rijkt): rdfexport voorbeeld

De pagenames hebben nu de prefix met een "-", wat de requirement was die we wilden toevoegen: import voorbeeld

infobox laat nog steeds goede labels zien: page

Alleen de form rijen hebben nu labels die niet helemaal zijn zoals je wilt lijkt me: form voorbeeld Ik stel voor om een extra argument prop.getLocalName() aan formRow() te geven voor de form label om dit op te lossen. Simpele wijziging die ik zo meteen zal submitten, review jij hem even Alex? r455

AlexMekkering commented 9 years ago

Review goedgekeurd, samen met alle strubbelingen rond #68 en #69