You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Dan Haywood (JIRA)" <ji...@apache.org> on 2017/04/19 07:44:42 UTC
[jira] [Updated] (ISIS-1595) Configuration property for JDOQL
VARIABLES clause mismatch
[ https://issues.apache.org/jira/browse/ISIS-1595?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dan Haywood updated ISIS-1595:
------------------------------
Fix Version/s: 1.15.0
> Configuration property for JDOQL VARIABLES clause mismatch
> ----------------------------------------------------------
>
> Key: ISIS-1595
> URL: https://issues.apache.org/jira/browse/ISIS-1595
> Project: Isis
> Issue Type: Bug
> Affects Versions: 1.14.0
> Reporter: Sander Ginn
> Fix For: 1.15.0
>
>
> The [migration notes for 1.14.0|http://isis.apache.org/migration-notes.html#_migration-notes_1.13.0-to-1.14.0] states that the configuration property to check for JDOQL {{VARIABLES}} class name validity is:
> {{isis.reflector.validator.jdoqlVariablesClause}}
> However, the [corresponding facet factory|https://github.com/apache/isis/blob/master/core/metamodel/src/main/java/org/apache/isis/objectstore/jdo/metamodel/facets/object/query/JdoQueryAnnotationFacetFactory.java] defines the property differently:
> {{public static final String ISIS_REFLECTOR_VALIDATOR_JDOQL_VARIABLES_CLAUSE_KEY = "isis.reflector.validator.variablesFromClause";}}
> I would argue that the documentation's version is better, since the factory version is rather ambiguous with respect to the property that defines JDOQL {{FROM}} clause validation. I will raise a PR that changes the constant to match the documentation.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)