You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Sander Ginn (JIRA)" <ji...@apache.org> on 2017/03/10 14:23:04 UTC

[jira] [Created] (ISIS-1595) Configuration property for JDOQL VARIABLES clause mismatch

Sander Ginn created ISIS-1595:
---------------------------------

             Summary: 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


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)