remove nextPageToken repetition check

isoos · isoos · commit 644eeb0fd494 · 2025-01-16T17:30:26.000+01:00
diff --git a/app/lib/service/youtube/backend.dart b/app/lib/service/youtube/backend.dart
@@ -121,7 +121,6 @@ class _PkgOfWeekVideoFetcher {
     final youtube = YouTubeApi(apiClient);
 
     try {
-      final pageTokensVisited = <String>{};
       String? nextPageToken;
 
       final videos = <PkgOfWeekVideo>[];
@@ -171,11 +170,9 @@ class _PkgOfWeekVideoFetcher {
           }
         }
 
-        pageTokensVisited.add(nextPageToken ?? '');
         // advance to next page token
         nextPageToken = rs.nextPageToken;
-        if (nextPageToken == null ||
-            pageTokensVisited.contains(nextPageToken)) {
+        if (nextPageToken == null) {
           break;
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,6 @@ class _PkgOfWeekVideoFetcher {`
`121`	`121`	`final youtube = YouTubeApi(apiClient);`
`122`	`122`
`123`	`123`	`try {`
`124`		`- final pageTokensVisited = <String>{};`
`125`	`124`	`String? nextPageToken;`
`126`	`125`
`127`	`126`	`final videos = <PkgOfWeekVideo>[];`
`@@ -171,11 +170,9 @@ class _PkgOfWeekVideoFetcher {`
`171`	`170`	`}`
`172`	`171`	`}`
`173`	`172`
`174`		`- pageTokensVisited.add(nextPageToken ?? '');`
`175`	`173`	`// advance to next page token`
`176`	`174`	`nextPageToken = rs.nextPageToken;`
`177`		`- if (nextPageToken == null \|\|`
`178`		`- pageTokensVisited.contains(nextPageToken)) {`
	`175`	`+ if (nextPageToken == null) {`
`179`	`176`	`break;`
`180`	`177`	`}`
`181`	`178`	`}`