Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-7218

Refactor components regarding groovy file locations

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: ALL COMPONENTS
    • Labels:
      None

      Description

      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

      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

        Attachments

          Issue Links

          There are no Sub-Tasks for this issue.

            Activity

              People

              • Assignee:
                deepak Deepak Dixit
                Reporter:
                deepak Deepak Dixit
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: