louisdevie/mustangproject
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

whitespace corrections+switch to ioutils

Browse Source
This commit is contained in:
jstaerk
2024-07-10 11:06:12 +02:00
parent acbfa2c836
commit c88ec268f5
3 changed files with 165 additions and 169 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
History.md
View File

@@ -5,6 +5,7 @@
- Fix #389: ClassCastException: ZUGFeRDExporterFromA3 - Fix #389: ClassCastException: ZUGFeRDExporterFromA3
- jakarta support #372 - jakarta support #372
- Upgrade to PDFBox 3 #373 - Upgrade to PDFBox 3 #373
- Requires Java 11
- #397 - #397
- #392 CLI: action combine: --ignorefileextension to ignore PDF/A input file errors dosen't work - #392 CLI: action combine: --ignorefileextension to ignore PDF/A input file errors dosen't work
- for CLI combine, fx is now default - for CLI combine, fx is now default

17
library/src/main/java/org/mustangproject/ZUGFeRD/ZUGFeRDImporter.java
View File

@@ -14,10 +14,7 @@ package org.mustangproject.ZUGFeRD;
* @author jstaerk * @author jstaerk
*/ */
import java.io.BufferedInputStream; import java.io.*;
import java.io.ByteArrayInputStream;
import java.io.IOException;
import java.io.InputStream;
import java.math.BigDecimal; import java.math.BigDecimal;
import java.nio.charset.StandardCharsets; import java.nio.charset.StandardCharsets;
import java.nio.file.Files; import java.nio.file.Files;
@@ -43,8 +40,8 @@ import javax.xml.xpath.XPathExpression;
import javax.xml.xpath.XPathExpressionException; import javax.xml.xpath.XPathExpressionException;
import javax.xml.xpath.XPathFactory; import javax.xml.xpath.XPathFactory;
import org.apache.commons.io.IOUtils;
import org.apache.pdfbox.Loader; import org.apache.pdfbox.Loader;
import org.apache.pdfbox.io.IOUtils;
import org.apache.pdfbox.pdmodel.PDDocument; import org.apache.pdfbox.pdmodel.PDDocument;
import org.apache.pdfbox.pdmodel.PDDocumentNameDictionary; import org.apache.pdfbox.pdmodel.PDDocumentNameDictionary;
import org.apache.pdfbox.pdmodel.PDEmbeddedFilesNameTreeNode; import org.apache.pdfbox.pdmodel.PDEmbeddedFilesNameTreeNode;
@@ -786,11 +783,11 @@ public class ZUGFeRDImporter {
static String convertStreamToString(java.io.InputStream is) { static String convertStreamToString(java.io.InputStream is) {
// TODO wouldn't we use IOUtils.toByteArray nowadays??? try {
// source https://stackoverflow.com/questions/309424/how-do-i-read-convert-an-inputstream-into-a-string-in-java referring to return IOUtils.toString(is, StandardCharsets.UTF_8);
// https://community.oracle.com/blogs/pat/2004/10/23/stupid-scanner-tricks } catch (IOException e) {
final Scanner s = new Scanner(is, StandardCharsets.UTF_8).useDelimiter("\\A"); throw new UncheckedIOException(e);
return s.hasNext() ? s.next() : ""; }
} }
/** /**

48
validator/src/main/java/org/mustangproject/validator/ZUGFeRDValidator.java
View File

@@ -76,20 +76,20 @@ public class ZUGFeRDValidator {
} }
private String internalValidate (String contextFilename, InputStream inputStream, long inputLength) { private String internalValidate(String contextFilename, InputStream inputStream, long inputLength) {
context.clear(); context.clear();
StringBuilder finalStringResult = new StringBuilder(); StringBuilder finalStringResult = new StringBuilder();
SimpleDateFormat isoDF = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); SimpleDateFormat isoDF = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
Date date = new Date(); Date date = new Date();
startTime = Calendar.getInstance().getTimeInMillis(); startTime = Calendar.getInstance().getTimeInMillis();
context.setFilename(contextFilename);// fallback to provided name context.setFilename(contextFilename);// fallback to provided name
finalStringResult.append("<validation filename='").append (contextFilename).append ("' datetime='").append (isoDF.format(date)).append ("'>"); finalStringResult.append("<validation filename='").append(contextFilename).append("' datetime='").append(isoDF.format(date)).append("'>");
boolean isPDF = false; boolean isPDF = false;
byte[] content = null; byte[] content = null;
try { try {
if (contextFilename == null || contextFilename.isEmpty ()) { if (contextFilename == null || contextFilename.isEmpty()) {
optionsRecognized = false; optionsRecognized = false;
context.addResultItem(new ValidationResultItem(ESeverity.fatal, "Filename not specified").setSection(10) context.addResultItem(new ValidationResultItem(ESeverity.fatal, "Filename not specified").setSection(10)
.setPart(EPart.pdf)); .setPart(EPart.pdf));
@@ -114,7 +114,7 @@ public class ZUGFeRDValidator {
if (disableNotices) { if (disableNotices) {
xv.disableNotices(); xv.disableNotices();
} }
isPDF = ByteArraySearcher.startsWith(content, new byte[] {'%', 'P', 'D', 'F'}); isPDF = ByteArraySearcher.startsWith(content, new byte[]{'%', 'P', 'D', 'F'});
if (isPDF) { if (isPDF) {
// Avoid reading again from file // Avoid reading again from file
pdfv.setFilenameAndContents(contextFilename, content); pdfv.setFilenameAndContents(contextFilename, content);
@@ -160,7 +160,7 @@ public class ZUGFeRDValidator {
if (isXML) { if (isXML) {
pdfValidity = true; pdfValidity = true;
optionsRecognized = true; optionsRecognized = true;
xv.setStringContent (xmlAsString); xv.setStringContent(xmlAsString);
xv.setAutoload(false); xv.setAutoload(false);
xv.setFilename(contextFilename); xv.setFilename(contextFilename);
sha1Checksum = calcSHA1(content); sha1Checksum = calcSHA1(content);
@@ -179,7 +179,7 @@ public class ZUGFeRDValidator {
try { try {
xv.validate(); xv.validate();
} catch (IrrecoverableValidationError irx) { } catch (IrrecoverableValidationError irx) {
LOGGER.info("The hell"); LOGGER.error("XML validation threw an exception ", irx);
} }
finalStringResult.append(xv.getXMLResult()); finalStringResult.append(xv.getXMLResult());
finalStringResult.append("</xml>"); finalStringResult.append("</xml>");
@@ -193,7 +193,7 @@ public class ZUGFeRDValidator {
} }
} catch (IrrecoverableValidationError | IOException irx) { } catch (IrrecoverableValidationError | IOException irx) {
LOGGER.info(irx.getMessage()); LOGGER.info(irx.getMessage());
context.setInvalid (); context.setInvalid();
} finally { } finally {
finalStringResult.append(context.getXMLResult()); finalStringResult.append(context.getXMLResult());
finalStringResult.append("</validation>"); finalStringResult.append("</validation>");
@@ -221,13 +221,13 @@ public class ZUGFeRDValidator {
} else { } else {
File file = new File(filename); File file = new File(filename);
// set filename without path // set filename without path
contextFilename = file.getName (); contextFilename = file.getName();
if (file.isFile ()) { if (file.isFile()) {
try { try {
inputStream = new FileInputStream (file); inputStream = new FileInputStream(file);
inputLength = Files.size (file.toPath ()); inputLength = Files.size(file.toPath());
} catch (IOException ex) { } catch (IOException ex) {
throw new UncheckedIOException (ex); throw new UncheckedIOException(ex);
} }
} else { } else {
// Non-existing or Directory // Non-existing or Directory
@@ -236,33 +236,31 @@ public class ZUGFeRDValidator {
} }
} }
try { try {
return internalValidate (contextFilename, inputStream, inputLength); return internalValidate(contextFilename, inputStream, inputLength);
} finally { } finally {
StreamHelper.close (inputStream); StreamHelper.close(inputStream);
} }
} }
public String validate(InputStream inputStream, String fileNameOfInputStream) { public String validate(InputStream inputStream, String fileNameOfInputStream) {
long inputLength; long inputLength;
try { try {
inputLength = inputStream == null ? 0 : inputStream.available (); inputLength = inputStream == null ? 0 : inputStream.available();
} } catch (IOException ex) {
catch (IOException ex) { throw new UncheckedIOException(ex);
throw new UncheckedIOException (ex);
} }
try { try {
return internalValidate (fileNameOfInputStream, inputStream, inputLength); return internalValidate(fileNameOfInputStream, inputStream, inputLength);
} finally { } finally {
StreamHelper.close (inputStream); StreamHelper.close(inputStream);
} }
} }
public String validate(byte[] bytes, String fileNameOfInputStream) { public String validate(byte[] bytes, String fileNameOfInputStream) {
try(ByteArrayInputStream bais = new ByteArrayInputStream (bytes)) { try (ByteArrayInputStream bais = new ByteArrayInputStream(bytes)) {
return internalValidate (fileNameOfInputStream, bais, bytes.length); return internalValidate(fileNameOfInputStream, bais, bytes.length);
} } catch (IOException ex) {
catch (IOException ex) { throw new UncheckedIOException(ex);
throw new UncheckedIOException (ex);
} }
} }