commit 5289ca399aaa2d30278232a5bc92f015b31cbe17
author Edwin Kempin <edwin.kempin@sap.com> Fri May 11 08:09:08 2012 +0200
committer Edwin Kempin <edwin.kempin@sap.com> Fri May 11 08:09:08 2012 +0200
tree 42b67fc515a9d7243e1a54fb9d7bd817f6bf4932
parent 5de34d9713108fb46e74c12b96db2e3e6be3c672

Fix equals method of DynamicMap.NamePair

Equals was wrongly implemented so that the lookup of functions from
the map failed.

Change-Id: Id996f4eb52e33defb93579eeecb7ef3825e95b3f
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>

gerrit-extension-api/src/main/java/com/google/gerrit/extensions/registration/DynamicMap.java

diff --git a/gerrit-extension-api/src/main/java/com/google/gerrit/extensions/registration/DynamicMap.java b/gerrit-extension-api/src/main/java/com/google/gerrit/extensions/registration/DynamicMap.java
index f114afd..8cac117 100644
--- a/gerrit-extension-api/src/main/java/com/google/gerrit/extensions/registration/DynamicMap.java
+++ b/gerrit-extension-api/src/main/java/com/google/gerrit/extensions/registration/DynamicMap.java
@@ -147,7 +147,7 @@
     public boolean equals(Object other) {
       if (other instanceof NamePair) {
         NamePair np = (NamePair) other;
-        return pluginName.equals(np) && exportName.equals(np);
+        return pluginName.equals(np.pluginName) && exportName.equals(np.exportName);
       }
       return false;
     }