Closed jadelkhoury closed 4 years ago
@berezovskyi Can you safely review this without me sharing Java generated code?
Ignore the commited java files (from EMF). Main logic is in the resourcePropertyServices.mtl file.
@nilsjor @berezovskyi Please review 2nd commit. I removed support for RDF:Bag. I also renamed "n_a" to "none". It was not easy to re-label the options to something more intuitive. I would rather do this for many other parameters at once.
@berezovskyi What do you think of this naming? Last offer.
Sounds good!
Introduce an optional rdfCollectionRepresentation attribute on ResourceProperties which can be set to rdfList or rdfBag (or n/a)
NOTE: This is what I have tested. But I have NOT tested that the actual annotation produces the expected result from JenaModelHelper.
if set to "n_a", no affect on produced java code.
if set on properties with cardinality 0..1 or 1: no effect on produced hava code.
If set to "RDFBag" on a Property with a "many" cardinality:
if set to "RDFList" on a Property with a "many" cardinality: