[jira] [Comment Edited] (JENA-839) E_Regex only works with XSDstring.

["Claude Warren (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/JENA-839?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14263492#comment-14263492 ] 

Claude Warren edited comment on JENA-839 at 1/3/15 10:55 AM:
-------------------------------------------------------------

I'll try to build a list of all the interactions that are affected.

My thought on the concat are that it should be an xsd:string as you can't guarantee that the result of the concat will meet the requirements of the xsd:normalizedString.




was (Author: claudenw):
I'll try to build a list of all the interactions that are affected.

My thought on the concat are that it should be an xsd:string as you can guarantee that hte result of the concat will meet the requirements of the xsd:normalizedString.



> E_Regex only works with XSDstring.
> ----------------------------------
>
>                 Key: JENA-839
>                 URL: https://issues.apache.org/jira/browse/JENA-839
>             Project: Apache Jena
>          Issue Type: Bug
>          Components: ARQ
>    Affects Versions: Jena 2.12.1
>         Environment: Linux
>            Reporter: Claude Warren
>            Assignee: Claude Warren
>
> E_Regex eval verifies the argument is a string using: 
> {noformat}
> Node arg = NodeFunctions.checkAndGetStringLiteral("REGEX", args.get(0)) ;
> {noformat}
>  NodeFunctions.checkAndGetStringLiteral verifies that if the nodevalue is not null and is an XSDstring by the following statement:
> {noformat}
>  if ( dt != null && !dt.equals(XSDDatatype.XSDstring) )
>             throw new ExprEvalException(label + ": Not a string literal: " + nv) ;
> {noformat}
> this means that other string types (e.g. XSDnormalizedString) fail.
> I believe that the check should be to check that dt is an instanceof XSDBaseStringType.
> I am looking for comment before making the change.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)