You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Jacques Le Roux (JIRA)" <ji...@apache.org> on 2016/06/08 08:33:20 UTC

[jira] [Commented] (OFBIZ-7218) Refactor components regarding groovy file locations

    [ https://issues.apache.org/jira/browse/OFBIZ-7218?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15320242#comment-15320242 ] 

Jacques Le Roux commented on OFBIZ-7218:
----------------------------------------

Following [Nicolas's comment|https://issues.apache.org/jira/browse/OFBIZ-6810?focusedCommentId=15201384] I also suggested to move minilang scripts from 
component://componentname/script/org/ofbiz 
to 
component://componentname/minilang
I'll create a Jira for that



> Refactor components regarding groovy file locations
> ---------------------------------------------------
>
>                 Key: OFBIZ-7218
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7218
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Trunk
>            Reporter: Deepak Dixit
>            Assignee: Deepak Dixit
>
> This is a placeholder issue to capture all tasks and related issues regarding the refactoring of the directory structure of .groovy files.
> See related discussion: http://ofbiz.markmail.org/message/25dse4jke2fp64mx
> Nicolas Malin's comments from OFBIZ-6810
> {quote}
> http://markmail.org/thread/kyhgfykfiwhzllll#query:+page:1+mid:hemokyhcnpe4w5rn+state:results
> At the question :
>     Regard to the issue, we have no problem for the java and minilang files but the groovy need a community decision.
>     A) All groovy script under the same path src/main/groovy/...
>     B) Separate Groovy compiled and scripted like this src/main/groovy/... (pre-compiled Groovy) src/main/scripts/... (parsed Groovy scripts)
>     C) Separate Groovy compiled and scripted like this groovy/... (pre-compiled Groovy) groovyScripts/... (parsed Groovy scripts, groovyScripts rather that script because other script languages could be used in the future)
>     D) other if you have an ingenious idea
> the result is :
>     Total are :
>     A) -5
>     B) 0
>     C) +7
>     D) No other proposal
> Now we can move all groovy script present on component://componentname/webapp/componentname/WEB-INF/actions/script.groovy to component://componentname/groovyscript/script.groovy
> {quote}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)