You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Radu Cotescu (JIRA)" <ji...@apache.org> on 2017/08/24 13:47:00 UTC
[jira] [Created] (SLING-7085) Remove code duplication
Radu Cotescu created SLING-7085:
-----------------------------------
Summary: Remove code duplication
Key: SLING-7085
URL: https://issues.apache.org/jira/browse/SLING-7085
Project: Sling
Issue Type: Bug
Components: Scripting
Affects Versions: Scripting HTL Java Compiler 1.0.0, Scripting HTL Compiler 1.0.0, Scripting HTL Engine 1.0.20
Reporter: Radu Cotescu
Assignee: Radu Cotescu
Fix For: Scripting HTL Compiler 1.0.12, Scripting HTL Java Compiler 1.0.12, Scripting HTL Engine 1.0.38
The modularisation of the HTL modules implemented in SLING-5787 introduced some code duplication:
* https://github.com/apache/sling/blob/1aa2c8be782ecb858de9030501e67edc4aba1357/bundles/scripting/sightly/compiler/src/main/java/org/apache/sling/scripting/sightly/impl/compiler/CompileTimeObjectModel.java
* https://github.com/apache/sling/blob/1aa2c8be782ecb858de9030501e67edc4aba1357/bundles/scripting/sightly/java-compiler/src/main/java/org/apache/sling/scripting/sightly/render/AbstractRuntimeObjectModel.java
* https://github.com/apache/sling/blob/1aa2c8be782ecb858de9030501e67edc4aba1357/bundles/scripting/sightly/engine/src/main/java/org/apache/sling/scripting/sightly/impl/engine/runtime/SlingRuntimeObjectModel.java
To reduce code duplication {{CompileTimeObjectModel}} should be exported as {{ObjectModel}} and extend its methods with the specifics of each module needing the functionality.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)