Uploaded image for project: 'Tools (JBoss Tools)'
  1. Tools (JBoss Tools)
  2. JBIDE-13733

Clean up JBossASHandler

    XMLWordPrintable

Details

    • Task
    • Resolution: Done
    • Major
    • 4.1.0.Alpha2
    • 4.1.0.Alpha1
    • server
    • None

    Description

      JBossASHandler is a pretty messy class with many confusing parts to it. Several things need to be changed.

      1) Why are default names for server required in this class? Don't runtime type and server type already do this in getDefaults() when the working copy is created?

      2) The code to check for nested runtimes is very messy and hard to understand.

      3) Why are ESB and Drools added separately? Shouldn't the check for all nested runtimes using all handlers be able to find this itself? It seems drools and esb have their own detectors. So why is this required here?

      4) The drools runtime seems to be not added to any model at all. It seems to just be returned. Is this only cosmetic? What is the purpose here? The drools detector would be adding this to the drools model, but we do not add it to any model, so what is the purpose?

      Attachments

        Issue Links

          Activity

            People

              snjeza_jira Snjezana Peco (Inactive)
              rob.stryker Rob Stryker (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: