Fix #7059: Don't show Brave News reload button on RSS-only feed

brave · Mar 8, 2023 · 64c81e7 · 64c81e7
1 parent 6a9f2ba
commit 64c81e7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Sources/Brave/WebFilters/AdblockRustEngine.swift b/Sources/Brave/WebFilters/AdblockRustEngine.swift
@@ -30,7 +30,6 @@ extension AdblockEngine {
     }
 
     guard let requestHost = requestURL.host, let sourceHost = sourceURL.host else {
-      assertionFailure("You must provide absolute paths for `requestURL` and `sourceURL`")
       return false
     }
 

diff --git a/Sources/BraveNews/Composer/FeedDataSource.swift b/Sources/BraveNews/Composer/FeedDataSource.swift
@@ -513,7 +513,8 @@ public class FeedDataSource: ObservableObject {
 
   /// Whether or not the feed content is currently expired and needs to be reloaded
   public var isFeedContentExpired: Bool {
-    return followedLocales.allSatisfy({ isResourceExpired(.feed, localeIdentifier: $0) })
+    let locales = followedLocales
+    return !locales.isEmpty && locales.allSatisfy({ isResourceExpired(.feed, localeIdentifier: $0) })
   }
 
   /// Whether or not the sources are currently expired and needs to be reloaded
-Original file line number
+Diff line change
@@ Expand Up / @@ -30,7 +30,6 @@ extension AdblockEngine { @@
         }
         guard let requestHost = requestURL.host, let sourceHost = sourceURL.host else {
-          assertionFailure("You must provide absolute paths for `requestURL` and `sourceURL`")
           return false
         }
@@ Expand Down @@