VocPub SHACL violation for isDefinedBy

[leskneebone]

Testing reveals some unexpected behaviour for SHACL VocPub 5.1.x validator. "vocPubInvalid_idDefinedBy.ttl" is attached for testing.

Steps

1. Data to validate > Upload > "vocPubInvalid_idDefinedBy.ttl"
2. SHACL Shapes > Use Validators > VocPub > Add > 5.1.x validator > close
3. SHACL Shapes > Validate

Validation results throw 9 results: 4 violations, 5 warnings, 0 informations (screenshot attached)

Issues:

a. For http://linked.data.gov.au/def/data-access-rights/closed-access-rights, a violation is thrown and is about skos:inScheme. Requirement 2.2.3 states that a skos:Collection "SHOULD indicate that it is within the vocabulary by use of the skos:inScheme predicate" - not "MUST". A warning may be more appropriate here.

b. For http://linked.data.gov.au/def/data-access-rights/open-access-rights, a violation is thrown and is about rdfs:isDefinedBy. Requirement 2.2.3 states that a skos:Collection should indicate with rdfs:isDefinedBy predicate "If it is defined for the first time in the vocabulary". This requirement is conditional. A warning or perhaps an Information may be more appropriate here.

vocPubInvalid_idDefinedBy.ttl.zip

[leskneebone]

Validator wrongly includes two occurrences of sh:minCount 1 ; statement.

Proposed change:

<Requirement-2.2.3>
    a sh:NodeShape ;
    sh:targetClass skos:Collection ;
    sh:property [
        sh:path skos:inScheme ;
        sh:maxCount 1 ;
        sh:message "Requirement 2.2.3 A Collection exists within a vocabulary SHOULD indicate that it is within the vocabulary by use of the skos:inScheme predicate. If it is defined for the first time in the vocabulary, it should also indicate this with the rdfs:isDefinedBy predicate. This message is about skos:inScheme" ;
    ] ,
    [
        sh:path rdfs:isDefinedBy ;
        sh:maxCount 1 ;
        sh:message "Requirement 2.2.3 A Collection exists within a vocabulary SHOULD indicate that it is within the vocabulary by use of the skos:inScheme predicate. If it is defined for the first time in the vocabulary, it should also indicate this with the rdfs:isDefinedBy predicate. This message is about rdfs:isDefinedBy" ;
    ] ;
    sh:severity sh:Warning ;
.