You'd think so, but the rule is different.
One removes based on connector.sourceId and the other removes based on connector.targetId Guess I could bring out the commons-collection's Predicates again to make a common base method. ;-)

- Joakim

Brett Porter wrote:

On 20/10/2007, at 8:48 AM, [EMAIL PROTECTED] wrote:

+ // [MRM-520] Proxy Connectors are not deleted with the deletion of a Repository. + List<ProxyConnectorConfiguration> proxyConnectors = getProxyConnectors(); + for ( ProxyConnectorConfiguration proxyConnector : proxyConnectors )
+        {
+ if ( StringUtils.equals( proxyConnector.getSourceRepoId(), cleanupRepository.getId() ) )
+            {
+ archivaConfiguration.getConfiguration().removeProxyConnector( proxyConnector );
+            }
+        }

Shouldn't this duplication be in the common base class?

- Brett

--
Brett Porter - [EMAIL PROTECTED]
Blog: http://www.devzuz.org/blogs/bporter/



--
- Joakim Erdfelt
 [EMAIL PROTECTED]
 Open Source Software (OSS) Developer

Reply via email to