Check if feed is XML first

jocmp · Dec 11, 2023 · f9a2725 · f9a2725
1 parent 2a49a6e
commit f9a2725
Show file tree

Hide file tree

Showing 8 changed files with 40 additions and 37 deletions.
diff --git a/basil/src/main/java/com/jocmp/basil/Account.kt b/basil/src/main/java/com/jocmp/basil/Account.kt
@@ -7,6 +7,7 @@ import com.jocmp.feedfinder.FeedFinder
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
 import java.net.URI
+import java.net.URL
 import java.util.UUID
 
 data class Account(
@@ -49,15 +50,17 @@ data class Account(
 
         val feed = Feed(
             id = UUID.randomUUID().toString(),
-            name = entry.name,
-            feedURL = found.feedURL.toString()
+            name = entryNameOrDefault(entry, found.name),
+            feedURL = found.feedURL.toString(),
+            siteURL = entrySiteURL(found.siteURL)
         )
 
         if (entry.folderTitles.isEmpty()) {
             feeds.add(feed)
         } else {
             entry.folderTitles.forEach { folderTitle ->
-                val folder = folders.find { folder -> folder.title == folderTitle } ?: Folder(title = folderTitle)
+                val folder = folders.find { folder -> folder.title == folderTitle }
+                    ?: Folder(title = folderTitle)
 
                 folder.feeds.add(feed)
 
@@ -74,6 +77,18 @@ data class Account(
         return feed
     }
 
+    private fun entrySiteURL(url: URL?): String {
+        return url?.toString() ?: ""
+    }
+
+    private fun entryNameOrDefault(entry: FeedFormEntry, feedName: String): String {
+        if (entry.name.isBlank()) {
+            return feedName
+        }
+
+        return entry.name
+    }
+
     private suspend fun saveOPMLFile() = withContext(Dispatchers.IO) {
         opmlFile.save()
     }
@@ -95,7 +110,7 @@ fun Account.asOPML(): String {
         opml += feed.asOPML(indentLevel = 2)
     }
 
-    folders.sortedBy { it.title } .forEach { folder ->
+    folders.sortedBy { it.title }.forEach { folder ->
         opml += folder.asOPML(indentLevel = 2)
     }
 

diff --git a/basil/src/main/java/com/jocmp/basil/Feed.kt b/basil/src/main/java/com/jocmp/basil/Feed.kt
@@ -6,7 +6,7 @@ data class Feed(
     val id: String,
     val name: String,
     val feedURL: String,
-    val homepageURL: String = ""
+    val siteURL: String = ""
 ) {
     override fun equals(other: Any?): Boolean {
         if (other is Feed) {
@@ -21,6 +21,6 @@ data class Feed(
 }
 
 fun Feed.asOPML(indentLevel: Int): String {
-    val opml = "<outline text=\"${name}\" title=\"${name}\" description=\"\" type=\"rss\" version=\"RSS\" htmlUrl=\"\" xmlUrl=\"${feedURL}\"/>\n"
+    val opml = "<outline text=\"${name}\" title=\"${name}\" description=\"\" type=\"rss\" version=\"RSS\" htmlUrl=\"${siteURL}\" xmlUrl=\"${feedURL}\"/>\n"
     return opml.prepending(tabCount = indentLevel)
 }
diff --git a/feedfinder/src/main/java/com/jocmp/feedfinder/FeedFinder.kt b/feedfinder/src/main/java/com/jocmp/feedfinder/FeedFinder.kt
@@ -25,10 +25,6 @@ class FeedFinder internal constructor(
     val url: String,
     private val request: Request = DefaultRequest()
 ) {
-    // Convert URL to HTTPS if missing
-    // 1. Download the request using a Java HTTP connection
-    // 2. If the response is an XML Feed itself, return
-    // 3. If the response is HTML and th
     internal suspend fun find(): Result = withContext(Dispatchers.IO) {
         try {
             val parsedURL = URI(url.withProtocol).toURL()
@@ -52,6 +48,7 @@ class FeedFinder internal constructor(
 
     private fun sources(response: Response): List<Source> {
         return listOf(
+            XMLSource(response),
             MetaLinkSource(response = response, request = request),
         )
     }

diff --git a/feedfinder/src/main/java/com/jocmp/feedfinder/parser/FakeFeed.kt b/feedfinder/src/main/java/com/jocmp/feedfinder/parser/FakeFeed.kt
@@ -7,6 +7,12 @@ class FakeFeed: Feed {
         return false
     }
 
+    override val name: String
+        get() = "Ars Technica"
+
+    override val siteURL: URL
+        get() = URL("https://arstechnica.com/")
+
     override val feedURL: URL
         get() = URL("https://arstechnica.com/feed")
 }
diff --git a/feedfinder/src/main/java/com/jocmp/feedfinder/parser/Feed.kt b/feedfinder/src/main/java/com/jocmp/feedfinder/parser/Feed.kt
@@ -5,5 +5,9 @@ import java.net.URL
 interface Feed {
     fun isValid(): Boolean
 
+    val name: String
+
     val feedURL: URL
+
+    val siteURL: URL?
 }
diff --git a/feedfinder/src/main/java/com/jocmp/feedfinder/parser/XMLFeed.kt b/feedfinder/src/main/java/com/jocmp/feedfinder/parser/XMLFeed.kt
@@ -16,6 +16,14 @@ internal class XMLFeed(
                 hasEntries()
     }
 
+    override val name: String
+        get() = channel!!.title!!
+
+    override val siteURL: URL?
+        get() = channel?.link?.let {
+            URL(it)
+        }
+
     private fun hasEntries(): Boolean {
         return channel != null &&
                 channel.items.isNotEmpty()

diff --git a/feedfinder/src/main/java/com/jocmp/feedfinder/sources/BodyLinkSource.kt b/feedfinder/src/main/java/com/jocmp/feedfinder/sources/BodyLinkSource.kt
diff --git a/feedfinder/src/main/java/com/jocmp/feedfinder/sources/Guess.kt b/feedfinder/src/main/java/com/jocmp/feedfinder/sources/Guess.kt