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

wise naming/implementation issues

    XMLWordPrintable

Details

    • Feature Request
    • Resolution: Done
    • Blocker
    • 4.1.0.Beta1
    • None
    • webservices
    • None

    Description

      the new addition of wise have a few quirks in naming and implementation.

      plugin org.jboss.tools.wise -> org.jboss.tools.wise.ui

      Reason: to match the rest of the plugins. Yes, it is mostly about bundling wise, but then it should on its own as org.jboss.tools.wise.core or org.jboss.tools.wise.runtime or similar. Not just org.jboss.tools.wise on its own. Since it also has helper code inside it I suggest to call it org.jboss.tools.ws.wise.ui and if one day it makes sense to isolate the jars in a wise specific plugin/bundle o.j.t.ws.wise or o.j.t.ws.wise.core it can just include it.

      org.jboss.wise.ui overlaps with wise framework package domain, should be org.jboss.tools.ws.wise.ui

      org.jboss.wise.ui.util.Activator - is this class actually needed ? if yes, shouldn't it be in org.jboss.ws.wise.ui.Activator ?

      https://github.com/jbosstools/jbosstools-webservices/blob/master/plugins/org.jboss.tools.wise/src/org/jboss/wise/ui/util/WiseUtil.java#L57 has a method named getSampleSOAPInputMessage2 in what is marked as public API. I suggest rename and javadoc it to something sensible or move it to an internal package.

      https://github.com/jbosstools/jbosstools-webservices/blob/master/plugins/org.jboss.tools.wise/src/org/jboss/wise/ui/util/WiseUtil.java#L85 and onward has a lot of ex.printstacktraces() - shouldn't these be handled/logged/ignored ?

      Attachments

        Activity

          People

            bfitzpat_rh Brian Fitzpatrick (Inactive)
            manderse@redhat.com Max Andersen
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: