Uploaded image for project: 'Magnolia Groovy Module'
  1. Magnolia Groovy Module
  2. MGNLGROOVY-65

Improvements to the InstallGroovyFile task

    XMLWordPrintable

Details

    • Bug
    • Resolution: Won't Do
    • Neutral
    • None
    • 1.2.4
    • None
    • None

    Description

      InstallGroovyFile could be improved:

      • There are "many" tasks that do something very similar (i.e load a file into the repo) - this should be extracted in an abstract core task rather than copy pasted all over the place
      • The 2-args constructor is completely unnecessary, and will not work - pattern will be null, and everything will fail miserably.
      • I should be able to specify whether I want the matching resources to have their is-a-script and enabled flags to true or false.
      • Lastly, the name of the class - suffix it with Class ?

      As a side note, I've been wondering why ExecuteGroovyFileTask is in a different package ?

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              fgrilli Federico Grilli
              gjoseph Magnolia International
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD