Uploaded image for project: 'WildFly Core'
  1. WildFly Core
  2. WFCORE-4327

Switched order of method arguments - ModuleLoadService

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • 9.0.0.Beta1, 9.0.0.Final
    • None
    • Server
    • None

      Switched order of method arguments - ModuleSpecProcessor line 287
      Introduced in https://github.com/wildfly/wildfly-core/pull/3661

      method call

      return ModuleLoadService.install(phaseContext.getServiceTarget(), moduleIdentifier,
         dependencies, localDependencies, userDependencies);
      

      ModuleLoadService.install signature:

      public static ServiceName install(final ServiceTarget target, final ModuleIdentifier identifier,
        final List<ModuleDependency> systemDependencies,
        final List<ModuleDependency> userDependencies,
        final List<ModuleDependency> localDependencies) {   
      

      https://github.com/wildfly/wildfly-core/blob/master/server/src/main/java/org/jboss/as/server/deployment/module/ModuleSpecProcessor.java#L287
      So localDependencies and userDependencies should be switched.

              yborgess1@redhat.com Yeray Borges Santana
              rsvoboda@redhat.com Rostislav Svoboda
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: