Close all input streams after parsing templates.
[Sone.git] / src / main / java / net / pterodactylus / sone / core / SoneInserter.java
index a511d14..76903f0 100644 (file)
@@ -21,6 +21,7 @@ import static com.google.common.base.Preconditions.checkArgument;
 import static java.lang.System.currentTimeMillis;
 import static net.pterodactylus.sone.data.Album.NOT_EMPTY;
 
+import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.StringWriter;
 import java.nio.charset.Charset;
@@ -94,12 +95,11 @@ public class SoneInserter extends AbstractService {
        /** The Freenet interface. */
        private final FreenetInterface freenetInterface;
 
+       private final SoneModificationDetector soneModificationDetector;
+
        /** The Sone to insert. */
        private volatile Sone sone;
 
-       /** The fingerprint of the last insert. */
-       private volatile String lastInsertFingerprint;
-
        /**
         * Creates a new Sone inserter.
         *
@@ -118,6 +118,7 @@ public class SoneInserter extends AbstractService {
                this.eventBus = eventBus;
                this.freenetInterface = freenetInterface;
                this.sone = sone;
+               this.soneModificationDetector = new SoneModificationDetector(core, sone, insertionDelay);
        }
 
        //
@@ -154,7 +155,7 @@ public class SoneInserter extends AbstractService {
         * @return The fingerprint of the last insert
         */
        public String getLastInsertFingerprint() {
-               return lastInsertFingerprint;
+               return soneModificationDetector.getOriginalFingerprint();
        }
 
        /**
@@ -164,7 +165,7 @@ public class SoneInserter extends AbstractService {
         *            The fingerprint of the last insert
         */
        public void setLastInsertFingerprint(String lastInsertFingerprint) {
-               this.lastInsertFingerprint = lastInsertFingerprint;
+               soneModificationDetector.setFingerprint(lastInsertFingerprint);
        }
 
        /**
@@ -175,7 +176,7 @@ public class SoneInserter extends AbstractService {
         *         otherwise
         */
        public boolean isModified() {
-               return !lastInsertFingerprint.equals(sone.getFingerprint());
+               return soneModificationDetector.isModified();
        }
 
        //
@@ -187,7 +188,6 @@ public class SoneInserter extends AbstractService {
         */
        @Override
        protected void serviceRun() {
-               SoneModificationDetector soneModificationDetector = new SoneModificationDetector(core, sone, insertionDelay);
                while (!shouldStop()) {
                        try {
                                /* check every second. */
@@ -231,7 +231,6 @@ public class SoneInserter extends AbstractService {
                                                        if (insertInformation.getFingerprint().equals(sone.getFingerprint())) {
                                                                logger.log(Level.FINE, String.format("Sone “%s” was not modified further, resetting counter…", sone));
                                                                soneModificationDetector.setFingerprint(insertInformation.getFingerprint());
-                                                               lastInsertFingerprint = insertInformation.getFingerprint();
                                                                core.touchConfiguration();
                                                        }
                                                }
@@ -344,15 +343,18 @@ public class SoneInserter extends AbstractService {
                @SuppressWarnings("synthetic-access")
                private ManifestElement createManifestElement(String name, String contentType, String templateName) {
                        InputStreamReader templateInputStreamReader = null;
+                       InputStream templateInputStream = null;
                        Template template;
                        try {
-                               templateInputStreamReader = new InputStreamReader(getClass().getResourceAsStream(templateName), utf8Charset);
+                               templateInputStream = getClass().getResourceAsStream(templateName);
+                               templateInputStreamReader = new InputStreamReader(templateInputStream, utf8Charset);
                                template = TemplateParser.parse(templateInputStreamReader);
                        } catch (TemplateException te1) {
                                logger.log(Level.SEVERE, String.format("Could not parse template “%s”!", templateName), te1);
                                return null;
                        } finally {
                                Closer.close(templateInputStreamReader);
+                               Closer.close(templateInputStream);
                        }
 
                        TemplateContext templateContext = templateContextFactory.createTemplateContext();