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)