BXC-4771 DRY up code and update limit

UNC-Libraries · Nov 18, 2024 · e08ebc5 · e08ebc5
1 parent 8fa0303
commit e08ebc5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/operations/src/main/java/edu/unc/lib/boxc/operations/impl/download/DownloadBulkService.java b/operations/src/main/java/edu/unc/lib/boxc/operations/impl/download/DownloadBulkService.java
@@ -62,8 +62,9 @@ private void zipFiles(WorkObject workObject, AccessGroupSet agentPrincipals, Pat
 
             Map<String, Integer> duplicates = new HashMap<>();
             int count = 0;
+            var loopLimit = getLoopLimit(memberObjects.size());
             for (ContentObject memberObject : memberObjects ) {
-                if (count == getLoopLimit(memberObjects.size())) {
+                if (count == loopLimit) {
                     break;
                 }
                 if (!(memberObject instanceof FileObject)) {

diff --git a/web-services-app/src/main/webapp/WEB-INF/service-context.xml b/web-services-app/src/main/webapp/WEB-INF/service-context.xml
@@ -498,7 +498,7 @@
         <property name="aclService" ref="aclService"/>
         <property name="repoObjLoader" ref="repositoryObjectLoader"/>
         <property name="basePath" value="${data.dir}/zip/"/>
-        <property name="fileLimit" value="500" />
+        <property name="fileLimit" value="100" />
     </bean>
 
     <bean class="org.springframework.beans.factory.config.MethodInvokingFactoryBean">