[
https://issues.apache.org/jira/browse/JENA-945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14547096#comment-14547096
]
Andy Seaborne commented on JENA-945:
------------------------------------
The description does not say why the proposed operations are needed.
I was not clear whether the need is because something can't be done (parsing
and serializaling is managing to use the public interface to create queries),
or needs to be done differently or the presence of the operations was missed.
The only thing that direct access to those members gives is being about to set
them to null, and then every access has to check, or to use an alternative to
{{ArrayList}}.
If you wish to add javadoc, then go ahead but additional operations to provide
similar function don't seem like a good idea to me.
> 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)
