diff --git a/docs-web/src/main/java/com/sismics/docs/rest/resource/DocumentResource.java b/docs-web/src/main/java/com/sismics/docs/rest/resource/DocumentResource.java index 30234b71..ad5d230b 100644 --- a/docs-web/src/main/java/com/sismics/docs/rest/resource/DocumentResource.java +++ b/docs-web/src/main/java/com/sismics/docs/rest/resource/DocumentResource.java @@ -497,14 +497,14 @@ public class DocumentResource extends BaseResource { .add("color", tagDto.getColor())); } - long filesCount; + Long filesCount; Collection filesOfDocument = null; if (Boolean.TRUE == files) { // Find files matching the document filesOfDocument = CollectionUtils.select(filesList, file -> file.getDocumentId().equals(documentDto.getId())); - filesCount = filesOfDocument.size(); + filesCount = (long) filesOfDocument.size(); } else { - filesCount = filesCountByDocument.get(documentDto.getId()); + filesCount = filesCountByDocument.getOrDefault(documentDto.getId(), 0L); } JsonObjectBuilder documentObjectBuilder = Json.createObjectBuilder() diff --git a/docs-web/src/test/java/com/sismics/docs/rest/TestDocumentResource.java b/docs-web/src/test/java/com/sismics/docs/rest/TestDocumentResource.java index 5061ca5c..1f1a7450 100644 --- a/docs-web/src/test/java/com/sismics/docs/rest/TestDocumentResource.java +++ b/docs-web/src/test/java/com/sismics/docs/rest/TestDocumentResource.java @@ -4,13 +4,6 @@ import com.google.common.io.ByteStreams; import com.google.common.io.Resources; import com.sismics.docs.core.util.DirectoryUtil; import com.sismics.util.filter.TokenBasedSecurityFilter; -import org.glassfish.jersey.media.multipart.FormDataMultiPart; -import org.glassfish.jersey.media.multipart.MultiPartFeature; -import org.glassfish.jersey.media.multipart.file.StreamDataBodyPart; -import org.joda.time.format.DateTimeFormat; -import org.junit.Assert; -import org.junit.Test; - import jakarta.json.JsonArray; import jakarta.json.JsonObject; import jakarta.ws.rs.client.Entity; @@ -18,6 +11,13 @@ import jakarta.ws.rs.core.Form; import jakarta.ws.rs.core.MediaType; import jakarta.ws.rs.core.Response; import jakarta.ws.rs.core.Response.Status; +import org.glassfish.jersey.media.multipart.FormDataMultiPart; +import org.glassfish.jersey.media.multipart.MultiPartFeature; +import org.glassfish.jersey.media.multipart.file.StreamDataBodyPart; +import org.joda.time.format.DateTimeFormat; +import org.junit.Assert; +import org.junit.Test; + import java.io.InputStream; import java.util.Date; @@ -261,6 +261,7 @@ public class TestDocumentResource extends BaseJerseyTest { Assert.assertEquals(create1Date, json.getJsonNumber("create_date").longValue()); Assert.assertNotNull(json.get("update_date")); tags = json.getJsonArray("tags"); + Assert.assertEquals(1, json.getJsonNumber("file_count").intValue()); Assert.assertEquals(2, tags.size()); Assert.assertEquals(tag2Id, tags.getJsonObject(0).getString("id")); Assert.assertEquals(tag1Id, tags.getJsonObject(1).getString("id"));