[
https://issues.apache.org/jira/browse/JENA-662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14259840#comment-14259840
]
ASF GitHub Bot commented on JENA-662:
-------------------------------------
GitHub user kinow opened a pull request:
https://github.com/apache/jena/pull/16
JENA-662 Wrong space handling in FileModelAssembler
Pull request for [JENA-662](https://issues.apache.org/jira/browse/JENA-662)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kinow/jena JENA-662
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/jena/pull/16.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #16
----
commit 36ddaf2a4d13ca27cb8b3841d87e8304908f2ef0
Author: Bruno P. Kinoshita <[email protected]>
Date: 2014-12-29T04:46:54Z
JENA-662 Wrong space handling in FileModelAssembler
----
> Wrong space handling in FileModelAssembler
> ------------------------------------------
>
> Key: JENA-662
> URL: https://issues.apache.org/jira/browse/JENA-662
> Project: Apache Jena
> Issue Type: Bug
> Components: Jena
> Affects Versions: Jena 2.11.1
> Reporter: Natanael Arndt
> Priority: Minor
>
> The method {{getDirectoryName}} in
> {{com.hp.hpl.jena.assembler.assemblers.FileModelAssembler}} should also make
> sure that at some chars, like space should be urldecoded again. E.g. If the
> URI is {{file:///tmp/file%20name%20with%20space}} the corresponding file can
> not be found under {{/tmp/file%20name%20with%20space}} but under {{/tmp/file
> name with space}}.
> There might be some other chars which have to be taken into account, e.g.
> "%25" -> "%", but I don't think a urldecode over the whole string is correct.
> Sorry I can't provide an example yet, because I couldn't isolate this part of
> the code so far and I didn't try to write a test for this.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
