Details

      Description

      The problem is in fact the get Class.forName thing here. This cannot work I think we should rather re-delegate this to the domain manager... This is a damnd critical issue...

      @SuppressWarnings("unchecked")
          private List<Method> getServiceMethods(ServiceId service) {
              if (service == null) {
                  return Collections.emptyList();
              }
              Class<?> connectorInterface;
              try {
                  connectorInterface = Class.forName(service.getServiceClass());
              } catch (ClassNotFoundException e) {
                  throw new IllegalArgumentException(e);
              }
              WiringService wireingService = OpenEngSBCoreServices.getWiringService();
              if (wireingService.isConnectorCurrentlyPresent((Class<? extends Domain>) connectorInterface)) {
                  submitButton.setEnabled(true);
                  return Arrays.asList(connectorInterface.getMethods());
              }
              error("No service found for domain: " + connectorInterface.getName());
              submitButton.setEnabled(false);
              return new ArrayList<Method>();
          }

        Gliffy Diagrams

        Error rendering 'com.meetme.plugins.jira.gerrit-plugin:gerritreviewsmodule'. Please contact your JIRA administrators.

          Activity

          Hide
          Christoph Gritschenberger added a comment -

          temporary fix implemented under OPENENGSB-1803
          permanent solution may need some discussion

          Show
          Christoph Gritschenberger added a comment - temporary fix implemented under OPENENGSB-1803 permanent solution may need some discussion
          Hide
          Christoph Gritschenberger added a comment -

          fixed permantently.
          It's handled by the domainprovider now.

          Show
          Christoph Gritschenberger added a comment - fixed permantently. It's handled by the domainprovider now.

            People

            • Assignee:
              Christoph Gritschenberger
              Reporter:
              Andreas Pieber
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: