From b5fa04f048ddbe729c4a4bf8a20b2a87b7f67fb5 Mon Sep 17 00:00:00 2001
From: Tom Fennelly <tom.fennelly@gmail.com>
Date: Fri, 22 Jan 2016 11:03:54 +0000
Subject: [PATCH] Fixups to ClassicPluginStrategy

---
 .../main/java/hudson/ClassicPluginStrategy.java    | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/hudson/ClassicPluginStrategy.java b/core/src/main/java/hudson/ClassicPluginStrategy.java
index a869e9342e..a3b6a07e98 100644
--- a/core/src/main/java/hudson/ClassicPluginStrategy.java
+++ b/core/src/main/java/hudson/ClassicPluginStrategy.java
@@ -285,7 +285,7 @@ public class ClassicPluginStrategy implements PluginStrategy {
     }
 
     /**
-     * Get the list of all plugins that have ever been detached from Jenkins core.
+     * Get the list of all plugins that have ever been {@link DetachedPlugin detached} from Jenkins core.
      * @return A {@link List} of {@link DetachedPlugin}s.
      */
     @Restricted(NoExternalUse.class)
@@ -330,7 +330,19 @@ public class ClassicPluginStrategy implements PluginStrategy {
 
     /**
      * Information about plugins that were originally in the core.
+     * <p>
+     * A detached plugin is one that has any of the following characteristics:
+     * <ul>
+     *     <li>
+     *         Was an existing plugin that at some time previously bundled with the Jenkins war file.
+     *     </li>
+     *     <li>
+     *         Was previous code in jenkins core that was split to a separate-plugin (but may not have
+     *         ever been bundled in a jenkins war file - i.e. it gets split after this 2.0 update).
+     *     </li>
+     * </ul>
      */
+    @Restricted(NoExternalUse.class)
     public static final class DetachedPlugin {
         private final String shortName;
         /**