Skip to content

Commit de9d720

Browse files
committed
fix: Handle empty list of announcements
1 parent 96dcf90 commit de9d720

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

extensions/youtube/src/main/java/app/revanced/extension/youtube/patches/announcements/AnnouncementsPatch.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,11 @@ private static boolean isLatestAlready() throws IOException {
5959
int id = Settings.ANNOUNCEMENT_LAST_ID.defaultValue;
6060
try {
6161
final var announcementIds = new JSONArray(jsonString);
62+
if (announcementIds.length() == 0) return true;
63+
6264
id = announcementIds.getJSONObject(0).getInt("id");
63-
6465
} catch (Throwable ex) {
65-
Logger.printException(() -> "Failed to parse announcement IDs", ex);
66+
Logger.printException(() -> "Failed to parse announcement ID", ex);
6667
}
6768

6869
// Do not show the announcement, if the last announcement id is the same as the current one.

extensions/youtube/src/main/java/app/revanced/extension/youtube/patches/announcements/requests/AnnouncementsRoutes.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010

1111
public class AnnouncementsRoutes {
1212
private static final String ANNOUNCEMENTS_PROVIDER = "https://api.revanced.app/v4";
13-
public static final Route GET_LATEST_ANNOUNCEMENT_IDS = new Route(GET, "/announcements/latest/id?tag=youtube");
14-
public static final Route GET_LATEST_ANNOUNCEMENTS = new Route(GET, "/announcements/latest?tag=youtube");
13+
public static final Route GET_LATEST_ANNOUNCEMENT_IDS = new Route(GET, "/announcements/latest/id?tag=\uD83C\uDF9E\uFE0F YouTube");
14+
public static final Route GET_LATEST_ANNOUNCEMENTS = new Route(GET, "/announcements/latest?tag=\uD83C\uDF9E\uFE0F YouTube");
1515

1616
private AnnouncementsRoutes() {
1717
}

0 commit comments

Comments
 (0)