Always try at least for number of displays

#154 (comment)
ifl0w · Feb 11, 2024 · 4be2fdf · 4be2fdf
1 parent 46f041b
commit 4be2fdf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/adapter/genericJson.ts b/src/adapter/genericJson.ts
@@ -45,7 +45,7 @@ class GenericJsonAdapter extends BaseAdapter {
         const authorNameJSONPath = this._settings.getString('author-name-path');
         const authorUrlJSONPath = this._settings.getString('author-url-path');
 
-        for (let i = 0; i < MAX_ARRAY_RETRIES && wallpaperResult.length < count; i++) {
+        for (let i = 0; i < MAX_ARRAY_RETRIES + count && wallpaperResult.length < count; i++) {
             const [returnObject, resolvedPath] = JSONPath.getTarget(response_body, imageJSONPath);
             if (!returnObject || (typeof returnObject !== 'string' && typeof returnObject !== 'number') || returnObject === '')
                 throw new Error('Unexpected json member found');
@@ -108,7 +108,7 @@ class GenericJsonAdapter extends BaseAdapter {
     async requestRandomImage(count: number) {
         const wallpaperResult: HistoryEntry[] = [];
 
-        for (let i = 0; i < MAX_SERVICE_RETRIES && wallpaperResult.length < count; i++) {
+        for (let i = 0; i < MAX_SERVICE_RETRIES + count && wallpaperResult.length < count; i++) {
             try {
                 // This should run sequentially
                 // eslint-disable-next-line no-await-in-loop

diff --git a/src/adapter/unsplash.ts b/src/adapter/unsplash.ts
@@ -70,7 +70,7 @@ class UnsplashAdapter extends BaseAdapter {
     async requestRandomImage(count: number) {
         const wallpaperResult: HistoryEntry[] = [];
 
-        for (let i = 0; i < MAX_SERVICE_RETRIES && wallpaperResult.length < count; i++) {
+        for (let i = 0; i < MAX_SERVICE_RETRIES + count && wallpaperResult.length < count; i++) {
             try {
                 // This should run sequentially
                 // eslint-disable-next-line no-await-in-loop