From 7795039e33961ce250db9729a9e1c2b7a8313c92 Mon Sep 17 00:00:00 2001
From: John Pederzolli <jpederzolli@shutterfly.com>
Date: Tue, 18 Aug 2015 08:41:15 -0700
Subject: [PATCH] additional comments

---
 core/src/main/java/jenkins/model/RunIdMigrator.java | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/jenkins/model/RunIdMigrator.java b/core/src/main/java/jenkins/model/RunIdMigrator.java
index 0315eb5fb1..88dc17b900 100644
--- a/core/src/main/java/jenkins/model/RunIdMigrator.java
+++ b/core/src/main/java/jenkins/model/RunIdMigrator.java
@@ -352,8 +352,10 @@ public final class RunIdMigrator {
         for (File job : jobDirs) {
 
             if (job.getName().equals("builds")) {
+                // Might be maven modules, matrix builds, etc. which are direct children of job
                 unmigrateBuildsDir(job);
             }
+
             File[] kids = job.listFiles();
             if (kids == null) {
                 continue;
@@ -365,7 +367,8 @@ public final class RunIdMigrator {
                 if (kid.getName().equals("builds")) {
                     unmigrateBuildsDir(kid);
                 } else {
-                    // Might be jobs, modules, promotions, etc.; we assume an ItemGroup.getRootDirFor implementation returns grandchildren.
+                    // Might be jobs, modules, promotions, etc.; we assume an ItemGroup.getRootDirFor implementation
+                    // returns grandchildren, unmigrateJobsDir(job) call above handles children.
                     unmigrateJobsDir(kid);
                 }
             }
-- 
GitLab