Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

f:hideProperties override f:showProperties (owf:sortProperties niet nodig) #25

Closed
LloydRutledge opened this issue Dec 11, 2014 · 2 comments
Assignees
Milestone

Comments

@LloydRutledge
Copy link
Contributor

fresnel:hideProperties werkt in Fresnel browsers om properties uit de fresnel:showProperties in dezelfde lens te overriden, en dus niet laten zien. We moeten dus ermee consequent zijn in het opslaan van Fresnel triple-bestanden. En ook in hoe de GUI werkt. Dus als iemand een propertie hidet met de GUI wordt die actie dan in de Fresnel-code ge-encodeerd als het toevoegen van die property in een hideProperties triple. Maar die oude property blijft in dezelfde, originele plek in de showProperties list om de volgorde te houden. Deze is consequent met Fresnel semantics. En daardoor is de voorgestelde owf:sortProperties niet nodig.

Vragen:

  • Kunnen we een hidden property unhiden?
  • Kunnen we een gewoon wissen, zodat die geen triple heeft met of hide- of showProperties voor die lens? Excuus, ik ben jullie antwoord hierop van onze scrum vergeten.
@AlexMekkering
Copy link
Member

Solved in r212, please review. This time, I'd like @jheijning to review because of the use of bound SPARQL queries in testFormattedPropertyFromFresnel 😉

@AlexMekkering AlexMekkering added this to the 1.0.6 milestone Dec 29, 2014
@jbachh
Copy link
Contributor

jbachh commented Dec 31, 2014

professionele job.
Paar heel kleine refactorings:
-JenaUtils: checkstyle javadoc issues verholpen.
-FresnelManager: javadoc "* Private constructor disabling instantiation and filling some initial requirements."
veranderd naar: "* Constructor filling some initial requirements."
-FresnelManagerTest: camelized enkele variable namen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants