Merge pull request #489 from jfrog/nehamjfrog-patch-1

nehamjfrog · web-flow · commit f08531112ac3 · 2025-03-27T12:07:59.000+05:30
Convert size property access within artifactCleanup.groovy to method
diff --git a/cleanup/artifactCleanup/artifactCleanup.groovy b/cleanup/artifactCleanup/artifactCleanup.groovy
@@ -276,16 +276,16 @@ private def getSkippedPaths(String[] repos) {
         // Simplify list to have only parent paths
         def paths = []
         for (path in pathsTmp.sort{ it }) {
-            if (paths.size == 0 || ! path.startsWith(paths[-1])) {
+            if (paths.size() == 0 || ! path.startsWith(paths[-1])) {
                 if (log.isTraceEnabled()){
                     log.trace "skip added for " + repoKey + ":" + path
                 }
                 paths.add(path)
             }
         }
 
-        if (paths.size > 0){
-            skip[repoKey] = paths.toArray(new String[paths.size])
+        if (paths.size() > 0){
+            skip[repoKey] = paths.toArray(new String[paths.size()])
         }
     }
     def timeStop = new Date()

Original file line number	Diff line number	Diff line change
`@@ -276,16 +276,16 @@ private def getSkippedPaths(String[] repos) {`
`276`	`276`	`// Simplify list to have only parent paths`
`277`	`277`	`def paths = []`
`278`	`278`	`for (path in pathsTmp.sort{ it }) {`
`279`		`- if (paths.size == 0 \|\| ! path.startsWith(paths[-1])) {`
	`279`	`+ if (paths.size() == 0 \|\| ! path.startsWith(paths[-1])) {`
`280`	`280`	`if (log.isTraceEnabled()){`
`281`	`281`	`log.trace "skip added for " + repoKey + ":" + path`
`282`	`282`	`}`
`283`	`283`	`paths.add(path)`
`284`	`284`	`}`
`285`	`285`	`}`
`286`	`286`
`287`		`- if (paths.size > 0){`
`288`		`- skip[repoKey] = paths.toArray(new String[paths.size])`
	`287`	`+ if (paths.size() > 0){`
	`288`	`+ skip[repoKey] = paths.toArray(new String[paths.size()])`
`289`	`289`	`}`
`290`	`290`	`}`
`291`	`291`	`def timeStop = new Date()`