[jbosstools-issues] [JBoss JIRA] Commented: (JBIDE-8474) Merge deltacloud trunk into maintenance 3.2.1

[ https://issues.jboss.org/browse/JBIDE-8474?page=com.atlassian.jira.plugin... ] Rob Stryker commented on JBIDE-8474: ------------------------------------ Another example of an improper change: - public static IHost createHost(String connectionName, String hostname, IRSESystemType systemType, + public static IHost createHost(String username, String connectionName, String hostname, IRSESystemType systemType, In this case, you changed the method signature in RSEUtils.createHost(etc). The proper fix would have been public static IHost createHost(String connectionName, String hostname, IRSESystemType systemType, + ISystemRegistry systemRegistry) throws Exception { + return createHost(DeltaCloudRSEConstants.USERNAME, connectionName, hostName, systemType, systemRegistry); + } + public static IHost createHost(String username, String connectionName, String hostname, IRSESystemType ISystemRegistry systemRegistry) throws Exception { In my fix, the original signature remains in the class and merely redirects to the new signature with a sensible default for the newly added parameter. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira