[JENA-1393] FmtUtils.stringForURI does not compact URIs with ":" in localname

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Done
• Affects Version/s: Jena 3.1.1, Jena 3.2.0, Jena 3.3.0, Jena 3.4.0
• Fix Version/s: Jena 3.5.0
• Component/s: ARQ, Jena
• Assignee: Andy Seaborne
• Reporter: Michał Woźniak
• Created: 17/Sep/17 14:36
• Updated: 02/Nov/17 22:10
• Resolved: 02/Oct/17 20:51

Description

Issue: FmtUtils.stringForURI does not compact URI with ":" in localname as the compacted
prefixed form gets rejected by FmtUtils.validPNameChar(ch) function. However,
according to the SPARQL specification ";" is allowed in prefixed form, see:
https://www.w3.org/TR/sparql11-query/#rPN_LOCAL

Note it ":" was not allowed in localname in older versions of the document (before 2013):
https://www.w3.org/TR/2012/WD-sparql11-query-20120105/#rPN_LOCAL

Proposed Fix: add the following line to FmtUtils.validPNameChar(ch):

         if ( ch == ':' )    return true ;

Proposed Test add to TestFmtUtils:

    @Test
    public void stringForURI_colonInLocalname_shouldCompact() {
        String uri = aUri + "local:name";
        final String result = FmtUtils.stringForURI(uri, getPrefixMapping());
        assertEquals("zz:local:name", result);
    }

Issue Links

• links to: GitHub Pull Request #282