Iri2pref #63
Iri2pref #63
Conversation
|
|
||
| } else { | ||
|
|
||
| return $html; |
There was a problem hiding this comment.
Could this have an HTML link, too, with $html as both the href and the link text?
There was a problem hiding this comment.
No problem! Looks like I will have to modify the t/serializer-sparqlhtml.t too since some tests expect the plain IRIs. Or perhaps you want this change only when there is URI::NamespaceMap instance?
There was a problem hiding this comment.
I'd like it done whether or not there's a namespace map.
I was planning on merging your test file with t/serializer-sparqlhtml.t anyway. So if you don't want to do that, I can take care of it during the merge.
|
Hi Zoran. This looks like a great PRC patch. I added one comment asking for a small change. Other than that, this looks good to merge. Thanks! |
|
Hi Greg, |
Hi Greg,
This is my first attempt to address the #54 issue as part of the CPAN PRC :-)
Looking forward to your feedback!
Thanks,
Zoran