Author: fanningpj
Date: Thu Dec 30 06:18:09 2021
New Revision: 1896524
URL: http://svn.apache.org/viewvc?rev=1896524&view=rev
Log:
add isEmpty() method
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/ooxml/extractor/POIXMLExtractorFactory.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/OPCPackage.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackagePart.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFSlideShow.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/openxml4j/opc/TestPackageThumbnail.java
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/ooxml/extractor/POIXMLExtractorFactory.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/ooxml/extractor/POIXMLExtractorFactory.java?rev=1896524&r1=1896523&r2=1896524&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/ooxml/extractor/POIXMLExtractorFactory.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/ooxml/extractor/POIXMLExtractorFactory.java
Thu Dec 30 06:18:09 2021
@@ -187,11 +187,11 @@ public final class POIXMLExtractorFactor
core = pkg.getRelationshipsByType(CORE_DOCUMENT_REL);
// If nothing was found, try some of the other OOXML-based core
types
- if (core.size() == 0) {
+ if (core.isEmpty()) {
// Could it be an OOXML-Strict one?
core = pkg.getRelationshipsByType(STRICT_DOCUMENT_REL);
}
- if (core.size() == 0) {
+ if (core.isEmpty()) {
// Could it be a visio one?
core = pkg.getRelationshipsByType(VISIO_DOCUMENT_REL);
if (core.size() == 1) {
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/OPCPackage.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/OPCPackage.java?rev=1896524&r1=1896523&r2=1896524&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/OPCPackage.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/OPCPackage.java
Thu Dec 30 06:18:09 2021
@@ -1351,7 +1351,7 @@ public abstract class OPCPackage impleme
@Override
public boolean hasRelationships() {
- return (relationships.size() > 0);
+ return !relationships.isEmpty();
}
@Override
@@ -1644,7 +1644,7 @@ public abstract class OPCPackage impleme
public boolean isStrictOoxmlFormat() {
PackageRelationshipCollection coreDocRelationships =
getRelationshipsByType(
PackageRelationshipTypes.STRICT_CORE_DOCUMENT);
- return coreDocRelationships.size() > 0;
+ return !coreDocRelationships.isEmpty();
}
/**
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackagePart.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackagePart.java?rev=1896524&r1=1896523&r2=1896524&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackagePart.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackagePart.java
Thu Dec 30 06:18:09 2021
@@ -442,8 +442,7 @@ public abstract class PackagePart implem
*/
@Override
public boolean hasRelationships() {
- return (!this._isRelationshipPart && (_relationships != null &&
_relationships
- .size() > 0));
+ return (!this._isRelationshipPart && (_relationships != null &&
!_relationships.isEmpty()));
}
/**
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFSlideShow.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFSlideShow.java?rev=1896524&r1=1896523&r2=1896524&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFSlideShow.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFSlideShow.java
Thu Dec 30 06:18:09 2021
@@ -195,7 +195,7 @@ public class XSLFSlideShow extends POIXM
throw new IllegalStateException(e);
}
- if(notes.size() == 0) {
+ if(notes.isEmpty()) {
// No notes for this slide
return null;
}
@@ -239,7 +239,7 @@ public class XSLFSlideShow extends POIXM
throw new IllegalStateException(e);
}
- if(commentRels.size() == 0) {
+ if(commentRels.isEmpty()) {
// No comments for this slide
return null;
}
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java?rev=1896524&r1=1896523&r2=1896524&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
Thu Dec 30 06:18:09 2021
@@ -158,7 +158,7 @@ public class XSSFBReader extends XSSFRea
try {
PackageRelationshipCollection commentsList =
sheetPkg.getRelationshipsByType(XSSFRelation.SHEET_COMMENTS.getRelation());
- if (commentsList.size() > 0) {
+ if (!commentsList.isEmpty()) {
PackageRelationship comments =
commentsList.getRelationship(0);
if (comments == null || comments.getTargetURI() == null) {
return null;
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java?rev=1896524&r1=1896523&r2=1896524&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
Thu Dec 30 06:18:09 2021
@@ -373,7 +373,7 @@ public class XSSFReader {
try {
PackageRelationshipCollection commentsList =
sheetPkg.getRelationshipsByType(XSSFRelation.SHEET_COMMENTS.getRelation());
- if (commentsList.size() > 0) {
+ if (!commentsList.isEmpty()) {
PackageRelationship comments =
commentsList.getRelationship(0);
PackagePartName commentsName =
PackagingURIHelper.createPartName(comments.getTargetURI());
PackagePart commentsPart =
sheetPkg.getPackage().getPart(commentsName);
@@ -401,7 +401,8 @@ public class XSSFReader {
// Do we have a shapes relationship? (Only ever one if so)
try {
PackageRelationshipCollection drawingsList =
sheetPkg.getRelationshipsByType(XSSFRelation.DRAWINGS.getRelation());
- for (int i = 0; i < drawingsList.size(); i++) {
+ int drawingsSize = drawingsList.size();
+ for (int i = 0; i < drawingsSize; i++) {
PackageRelationship drawings =
drawingsList.getRelationship(i);
PackagePartName drawingsName =
PackagingURIHelper.createPartName(drawings.getTargetURI());
PackagePart drawingsPart =
sheetPkg.getPackage().getPart(drawingsName);
Modified:
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/openxml4j/opc/TestPackageThumbnail.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/test/java/org/apache/poi/openxml4j/opc/TestPackageThumbnail.java?rev=1896524&r1=1896523&r2=1896524&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/openxml4j/opc/TestPackageThumbnail.java
(original)
+++
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/openxml4j/opc/TestPackageThumbnail.java
Thu Dec 30 06:18:09 2021
@@ -17,6 +17,7 @@
package org.apache.poi.openxml4j.opc;
+import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotEquals;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static org.junit.jupiter.api.Assertions.fail;
@@ -50,7 +51,7 @@ public final class TestPackageThumbnail
// Open the newly created file to check core properties saved
values.
try (OPCPackage p2 = OPCPackage.open(outputFile.getAbsolutePath(),
PackageAccess.READ)) {
- assertNotEquals(0,
p2.getRelationshipsByType(PackageRelationshipTypes.THUMBNAIL).size(),
+
assertFalse(p2.getRelationshipsByType(PackageRelationshipTypes.THUMBNAIL).isEmpty(),
"Thumbnail not added to the package !");
p2.revert();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
