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

Claude Warren commented on JENA-945:
------------------------------------

Are you indicating that you think we should not allow direct access to the 
relevant lists or just that it should be noted in the documentation.  I would 
prefer the latter.

The QueryBuilder requires the ability to modify the list.  That is, it may 
replace (positionally significant) a variable in the list.  So either direct 
access to the list, or get, add and replace methods need to be provided.

QueryBuilder is also in a different package. (o.a.j.arq.querybuilder).





> Add Seven methods to Query class
> --------------------------------
>
>                 Key: JENA-945
>                 URL: https://issues.apache.org/jira/browse/JENA-945
>             Project: Apache Jena
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: Jena 3.0.0
>            Reporter: Claude Warren
>            Assignee: Claude Warren
>            Priority: Minor
>
> The QueryBuilder uses reflection to perform some of its data manipulation.  
> This reflection will fail in an environment with Java Security Manager 
> enabled.
> This change is to add methods so that QueryBuilder no longer needs to use 
> reflection to get and set Query internal variables.
> The variables are:
> namedGraphURIs  -- get and set required. parameter/return: List<String>
> graphURIs -- get and set required. parameter/return: List<String>
> projectVars -- get and set required. parameter/return: VarExprList
> groupVars -- set required. parameter: VarExprList



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

Reply via email to