[jboss-jira] [JBoss JIRA] (AS7-2024) JDBC Driver registry does not unregister properly

[Stefano Maestri (Updated) (JIRA)]

     [ https://issues.jboss.org/browse/AS7-2024?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stefano Maestri updated AS7-2024:
---------------------------------

    Fix Version/s: 7.1.0.Beta1

    
> JDBC Driver registry does not unregister properly
> -------------------------------------------------
>
>                 Key: AS7-2024
>                 URL: https://issues.jboss.org/browse/AS7-2024
>             Project: Application Server 7
>          Issue Type: Bug
>          Components: JCA
>    Affects Versions: 7.1.0.Alpha1
>            Reporter: Tristan Tarrant
>            Assignee: Stefano Maestri
>            Priority: Minor
>             Fix For: 7.1.0.Beta1
>
>
> org.jboss.as.connector.registry.DriverRegistryImpl in as7 trunk
> I think line 62 is a copy/paste bug:
> {code:title=DriverRegistryImpl.java} 
> @Override
>     public void unregisterInstalledDriver(InstalledDriver driver) {
>         if (driver == null)
>             throw new IllegalArgumentException(MESSAGES.nullVar("driver"));
>         DEPLOYMENT_CONNECTOR_REGISTRY_LOGGER.tracef("Removing deployment: %s", driver);
>         synchronized (drivers) {
>             drivers.put(driver.getDriverName(), driver);
>         }
>     }
> {code}
> should be
> {code:title=DriverRegistryImpl.java} 
> @Override
>     public void unregisterInstalledDriver(InstalledDriver driver) {
>         if (driver == null)
>             throw new IllegalArgumentException(MESSAGES.nullVar("driver"));
>         DEPLOYMENT_CONNECTOR_REGISTRY_LOGGER.tracef("Removing deployment: %s", driver);
>         synchronized (drivers) {
>             drivers.remove(driver.getDriverName());
>         }
>     }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira