Issue #7613 Override Configurations.add(Configuration) (#7631)

Signed-off-by: Jan Bartel <[email protected]>
jetty · Feb 22, 2022 · 3d9f346 · 3d9f346
1 parent 808b74f
commit 3d9f346
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/jetty-webapp/src/main/java/org/eclipse/jetty/webapp/Configurations.java b/jetty-webapp/src/main/java/org/eclipse/jetty/webapp/Configurations.java
@@ -265,6 +265,12 @@ public Configurations(Configurations classlist)
             .map(c -> c.getClass().getName())
             .toArray(String[]::new));
     }
+
+    @Override
+    public boolean add(Configuration configuration)
+    {
+        return addConfiguration(configuration);
+    }
 
     public void add(Configuration... configurations)
     {
@@ -428,7 +434,7 @@ public Iterator<Configuration> iterator()
         return getConfigurations().iterator();
     }
 
-    private void addConfiguration(Configuration configuration)
+    private boolean addConfiguration(Configuration configuration)
     {
         String name = configuration.getClass().getName();
         // Is this configuration known?
@@ -462,14 +468,14 @@ private void addConfiguration(Configuration configuration)
             if (r != null)
             {
                 if (r.getName().equals(configuration.getClass().getName()))
-                    return; //skip the addition, a replacement is already present
+                    return false; //skip the addition, a replacement is already present
             }
 
             if (c.getClass().getName().equals(configuration.getClass().getName()))
-                return; //don't add same one twice
+                return false; //don't add same one twice
         }
 
-        _configurations.add(configuration);
+        return _configurations.add(configuration);
     }
 
     @Override