X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fmain%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fcore%2FDefaultElementLoader.kt;h=3f33efe52711ba5bb3c857c3e566bd46cc821cf4;hb=7a70c5564da777276295d6ec29ab76446425bb36;hp=956d73be86b5011450a211126edb33e5d7852a48;hpb=6e91e9356231217afcecb12275882c0ebf0f34a8;p=Sone.git

diff --git a/src/main/kotlin/net/pterodactylus/sone/core/DefaultElementLoader.kt b/src/main/kotlin/net/pterodactylus/sone/core/DefaultElementLoader.kt
index 956d73b..3f33efe 100644
--- a/src/main/kotlin/net/pterodactylus/sone/core/DefaultElementLoader.kt
+++ b/src/main/kotlin/net/pterodactylus/sone/core/DefaultElementLoader.kt
@@ -1,40 +1,51 @@
 package net.pterodactylus.sone.core
 
+import com.google.common.base.Ticker
 import com.google.common.cache.CacheBuilder
 import freenet.keys.FreenetURI
 import java.io.ByteArrayInputStream
+import java.util.concurrent.TimeUnit.MINUTES
 import javax.imageio.ImageIO
 import javax.inject.Inject
 
 /**
  * [ElementLoader] implementation that uses a simple Guava [com.google.common.cache.Cache].
  */
-class DefaultElementLoader @Inject constructor(private val freenetInterface: FreenetInterface) : ElementLoader {
+class DefaultElementLoader(private val freenetInterface: FreenetInterface, ticker: Ticker) : ElementLoader {
+
+	@Inject constructor(freenetInterface: FreenetInterface) : this(freenetInterface, Ticker.systemTicker())
 
 	private val loadingLinks = CacheBuilder.newBuilder().build<String, Boolean>()
-	private val imageCache = CacheBuilder.newBuilder().build<String, LinkedImage>()
+	private val failureCache = CacheBuilder.newBuilder().ticker(ticker).expireAfterWrite(30, MINUTES).build<String, Boolean>()
+	private val imageCache = CacheBuilder.newBuilder().build<String, LinkedElement>()
 	private val callback = object : FreenetInterface.BackgroundFetchCallback {
 		override fun cancelForMimeType(uri: FreenetURI, mimeType: String): Boolean {
+			println("Got MIME Type “$mimeType” for $uri.")
 			return !mimeType.startsWith("image/")
 		}
 
 		override fun loaded(uri: FreenetURI, mimeType: String, data: ByteArray) {
+			println("Got ${data.size} Bytes with MIME Type $mimeType for $uri.")
 			if (!mimeType.startsWith("image/")) {
 				return
 			}
 			ByteArrayInputStream(data).use {
 				ImageIO.read(it)
 			}?.let {
-				imageCache.get(uri.toString()) { LinkedImage(uri.toString()) }
+				println("Successfully parsed images from $uri.")
+				imageCache.get(uri.toString()) { LinkedElement(uri.toString()) }
 			}
 			removeLoadingLink(uri)
 		}
 
 		override fun failed(uri: FreenetURI) {
+			println("Failed to load $uri.")
+			failureCache.put(uri.toString(), true)
 			removeLoadingLink(uri)
 		}
 
 		private fun removeLoadingLink(uri: FreenetURI) {
+			println("Not loading anymore: $uri.")
 			synchronized(loadingLinks) {
 				loadingLinks.invalidate(uri.toString())
 			}
@@ -42,19 +53,24 @@ class DefaultElementLoader @Inject constructor(private val freenetInterface: Fre
 	}
 
 	override fun loadElement(link: String): LinkedElement {
+		println("Checking for $link...")
 		synchronized(loadingLinks) {
 			imageCache.getIfPresent(link)?.run {
+				println("In the Image Cache: $link")
 				return this
 			}
+			failureCache.getIfPresent(link)?.run {
+				println("In the Failure Cache: $link")
+				return LinkedElement(link, failed = true)
+			}
 			if (loadingLinks.getIfPresent(link) == null) {
+				println("Not loading: $link")
 				loadingLinks.put(link, true)
 				freenetInterface.startFetch(FreenetURI(link), callback)
 			}
 		}
-		return object : LinkedElement {
-			override val link = link
-			override val loading = true
-		}
+		println("Returning loading element: $link")
+		return LinkedElement(link, loading = true)
 	}
 
 }